refactor(core): renamed previousOrParent to currentTNode (#38707)

The previous name of `previousOrParent` was confusing. Changed the
terminology to `currentTNode`.

PR Close #38707
This commit is contained in:
Misko Hevery
2020-09-14 13:43:44 -07:00
parent 0fa208f624
commit 65d4e7a8af
26 changed files with 161 additions and 169 deletions

View File

@ -687,7 +687,7 @@ onlyInIvy('Ivy i18n logic').describe('runtime i18n', () => {
'(lView[0] as Element).appendChild(lView[2])',
'lView[4] = document.createTextNode("World")',
'(lView[2] as Element).appendChild(lView[4])',
'setPreviousOrParentTNode(tView.data[2] as TNode)',
'setCurrentTNode(tView.data[2] as TNode)',
'lView[5] = document.createTextNode("!")',
'(lView[0] as Element).appendChild(lView[5])',
]);

View File

@ -171,10 +171,10 @@
"name": "getConstant"
},
{
"name": "getFirstLContainer"
"name": "getCurrentTNode"
},
{
"name": "getIsParent"
"name": "getFirstLContainer"
},
{
"name": "getLView"
@ -200,9 +200,6 @@
{
"name": "getOrCreateTNode"
},
{
"name": "getPreviousOrParentTNode"
},
{
"name": "getSimpleChangesStore"
},
@ -245,6 +242,9 @@
{
"name": "isCssClassMatching"
},
{
"name": "isCurrentTNodeParent"
},
{
"name": "isInlineTemplate"
},
@ -323,6 +323,9 @@
{
"name": "setCurrentQueryIndex"
},
{
"name": "setCurrentTNode"
},
{
"name": "setDirectiveInputsWhichShadowsStyling"
},
@ -335,9 +338,6 @@
{
"name": "setInputsFromAttrs"
},
{
"name": "setPreviousOrParentTNode"
},
{
"name": "setSelectedIndex"
},

View File

@ -965,6 +965,9 @@
{
"name": "getConstant"
},
{
"name": "getCurrentTNode"
},
{
"name": "getDOM"
},
@ -986,9 +989,6 @@
{
"name": "getInjectorIndex"
},
{
"name": "getIsParent"
},
{
"name": "getLCleanup"
},
@ -1052,9 +1052,6 @@
{
"name": "getPreviousIndex"
},
{
"name": "getPreviousOrParentTNode"
},
{
"name": "getPromiseCtor"
},
@ -1172,6 +1169,9 @@
{
"name": "isCssClassMatching"
},
{
"name": "isCurrentTNodeParent"
},
{
"name": "isDirectiveHost"
},
@ -1499,6 +1499,9 @@
{
"name": "setCurrentQueryIndex"
},
{
"name": "setCurrentTNode"
},
{
"name": "setDirectiveInputsWhichShadowsStyling"
},
@ -1517,9 +1520,6 @@
{
"name": "setLocaleId"
},
{
"name": "setPreviousOrParentTNode"
},
{
"name": "setSelectedIndex"
},

View File

@ -119,6 +119,9 @@
{
"name": "getComponentLViewByIndex"
},
{
"name": "getCurrentTNode"
},
{
"name": "getFirstLContainer"
},
@ -134,9 +137,6 @@
{
"name": "getOrCreateTNode"
},
{
"name": "getPreviousOrParentTNode"
},
{
"name": "getSimpleChangesStore"
},
@ -206,15 +206,15 @@
{
"name": "setCurrentQueryIndex"
},
{
"name": "setCurrentTNode"
},
{
"name": "setIncludeViewProviders"
},
{
"name": "setInjectImplementation"
},
{
"name": "setPreviousOrParentTNode"
},
{
"name": "setSelectedIndex"
},

View File

@ -1280,6 +1280,9 @@
{
"name": "getCurrentQueryIndex"
},
{
"name": "getCurrentTNode"
},
{
"name": "getDOM"
},
@ -1310,9 +1313,6 @@
{
"name": "getInjectorIndex"
},
{
"name": "getIsParent"
},
{
"name": "getLCleanup"
},
@ -1388,9 +1388,6 @@
{
"name": "getPreviousIndex"
},
{
"name": "getPreviousOrParentTNode"
},
{
"name": "getPromiseCtor"
},
@ -1508,6 +1505,9 @@
{
"name": "isCssClassMatching"
},
{
"name": "isCurrentTNodeParent"
},
{
"name": "isDirectiveHost"
},
@ -1832,6 +1832,9 @@
{
"name": "setCurrentQueryIndex"
},
{
"name": "setCurrentTNode"
},
{
"name": "setDirectiveInputsWhichShadowsStyling"
},
@ -1850,9 +1853,6 @@
{
"name": "setLocaleId"
},
{
"name": "setPreviousOrParentTNode"
},
{
"name": "setRouterState"
},

View File

@ -344,6 +344,9 @@
{
"name": "getConstant"
},
{
"name": "getCurrentTNode"
},
{
"name": "getDebugContext"
},
@ -353,9 +356,6 @@
{
"name": "getInjectorIndex"
},
{
"name": "getIsParent"
},
{
"name": "getLCleanup"
},
@ -410,9 +410,6 @@
{
"name": "getPreviousIndex"
},
{
"name": "getPreviousOrParentTNode"
},
{
"name": "getSelectedIndex"
},
@ -482,6 +479,9 @@
{
"name": "isCssClassMatching"
},
{
"name": "isCurrentTNodeParent"
},
{
"name": "isDirectiveHost"
},
@ -650,6 +650,9 @@
{
"name": "setCurrentQueryIndex"
},
{
"name": "setCurrentTNode"
},
{
"name": "setDirectiveInputsWhichShadowsStyling"
},
@ -665,9 +668,6 @@
{
"name": "setInputsFromAttrs"
},
{
"name": "setPreviousOrParentTNode"
},
{
"name": "setSelectedIndex"
},

View File

@ -128,7 +128,7 @@ describe('i18n debug', () => {
it('should print ElementEnd', () => {
expect(i18nMutateOpCodesToString([
1 << I18nMutateOpCode.SHIFT_REF | I18nMutateOpCode.ElementEnd,
])).toEqual(['setPreviousOrParentTNode(tView.data[1] as TNode)']);
])).toEqual(['setCurrentTNode(tView.data[1] as TNode)']);
});
it('should print RemoveNestedIcu', () => {

View File

@ -112,13 +112,13 @@ describe('Runtime i18n', () => {
'(lView[1] as Element).appendChild(lView[2])',
'lView[5] = document.createTextNode("world")',
'(lView[2] as Element).appendChild(lView[5])',
'setPreviousOrParentTNode(tView.data[2] as TNode)',
'setCurrentTNode(tView.data[2] as TNode)',
'lView[6] = document.createTextNode(" and ")',
'(lView[1] as Element).appendChild(lView[6])',
'(lView[1] as Element).appendChild(lView[3])',
'lView[7] = document.createTextNode("universe")',
'(lView[3] as Element).appendChild(lView[7])',
'setPreviousOrParentTNode(tView.data[3] as TNode)',
'setCurrentTNode(tView.data[3] as TNode)',
'lView[8] = document.createTextNode("!")',
'(lView[1] as Element).appendChild(lView[8])',
]),
@ -221,8 +221,7 @@ describe('Runtime i18n', () => {
'(lView[1] as Element).appendChild(lView[3])',
'(lView[1] as Element).appendChild(lView[16381])',
'lView[4] = document.createTextNode("after")',
'(lView[1] as Element).appendChild(lView[4])',
'setPreviousOrParentTNode(tView.data[1] as TNode)'
'(lView[1] as Element).appendChild(lView[4])', 'setCurrentTNode(tView.data[1] as TNode)'
]),
update: [],
icus: null
@ -242,8 +241,7 @@ describe('Runtime i18n', () => {
create: debugMatch([
'(lView[0] as Element).appendChild(lView[1])',
'lView[2] = document.createTextNode("middle")',
'(lView[1] as Element).appendChild(lView[2])',
'setPreviousOrParentTNode(tView.data[1] as TNode)'
'(lView[1] as Element).appendChild(lView[2])', 'setCurrentTNode(tView.data[1] as TNode)'
]),
update: [],
icus: null

View File

@ -236,7 +236,7 @@ export function resetDOM() {
containerEl.setAttribute('host', '');
document.body.appendChild(containerEl);
hostView = null;
// TODO: assert that the global state is clean (e.g. ngData, previousOrParentNode, etc)
// TODO: assert that the global state is clean (e.g. ngData, currentTNode, etc)
}