refactor(ivy): remove inputsPropertyName (#22716)

Closes #22591

PR Close #22716

packages/core/src/render3/definition.ts

@@ -15,7 +15,7 @@ import {Type} from '../type';
 import {resolveRendererType2} from '../view/util';
 
 import {diPublic} from './di';
-import {ComponentDef, ComponentDefArgs, DirectiveDef, DirectiveDefArgs, PipeDef, PipeType} from './interfaces/definition';
+import {ComponentDef, ComponentDefArgs, DirectiveDef, DirectiveDefArgs, DirectiveDefFeature, PipeDef} from './interfaces/definition';
 
 
 
@@ -45,7 +45,6 @@ export function defineComponent<T>(componentDefinition: ComponentDefArgs<T>): Co
     h: componentDefinition.hostBindings || noop,
     attributes: componentDefinition.attributes || null,
     inputs: invertObject(componentDefinition.inputs),
-    inputsPropertyName: componentDefinition.inputsPropertyName || null,
     outputs: invertObject(componentDefinition.outputs),
     rendererType: resolveRendererType2(componentDefinition.rendererType) || null,
     exportAs: componentDefinition.exportAs,
@@ -72,15 +71,39 @@ type OnChangesExpando = OnChanges & {
   [key: string]: any;
 };
 
-export function NgOnChangesFeature(definition: DirectiveDef<any>): void {
+/**
+ * Creates an NgOnChangesFeature function for a component's features list.
+ *
+ * It accepts an optional map of minified input property names to original property names,
+ * if any input properties have a public alias.
+ *
+ * The NgOnChangesFeature function that is returned decorates a component with support for
+ * the ngOnChanges lifecycle hook, so it should be included in any component that implements
+ * that hook.
+ *
+ * Example usage:
+ *
+ * ```
+ * static ngComponentDef = defineComponent({
+ *   ...
+ *   inputs: {name: 'publicName'},
+ *   features: [NgOnChangesFeature({name: 'name'})]
+ * });
+ * ```
+ *
+ * @param inputPropertyNames Map of input property names, if they are aliased
+ * @returns DirectiveDefFeature
+ */
+export function NgOnChangesFeature(inputPropertyNames?: {[key: string]: string}):
+    DirectiveDefFeature {
+  return function(definition: DirectiveDef<any>): void {
     const inputs = definition.inputs;
     const proto = definition.type.prototype;
-  const inputsPropertyName = definition.inputsPropertyName;
     // Place where we will store SimpleChanges if there is a change
     Object.defineProperty(proto, PRIVATE_PREFIX, {value: undefined, writable: true});
     for (let pubKey in inputs) {
       const minKey = inputs[pubKey];
-    const propertyName = inputsPropertyName && inputsPropertyName[minKey] || pubKey;
+      const propertyName = inputPropertyNames && inputPropertyNames[minKey] || pubKey;
       const privateMinKey = PRIVATE_PREFIX + minKey;
       // Create a place where the actual value will be stored and make it non-enumerable
       Object.defineProperty(proto, privateMinKey, {value: undefined, writable: true});
@@ -114,6 +137,7 @@ export function NgOnChangesFeature(definition: DirectiveDef<any>): void {
     }
 
     definition.doCheck = onChangesWrapper(definition.doCheck);
+  };
 
   function onChangesWrapper(delegateHook: (() => void) | null) {
     return function(this: OnChangesExpando) {

packages/core/src/render3/interfaces/definition.ts

@@ -68,14 +68,6 @@ export interface DirectiveDef<T> {
    */
   readonly inputs: {[P in keyof T]: P};
 
-  /**
-   * A dictionary mapping the inputs' minified property names to the original unminified property
-   * names.
-   *
-   * An entry is added if and only if the alias is different from the property name.
-   */
-  readonly inputsPropertyName: {[P in keyof T]: P};
-
   /**
    * A dictionary mapping the outputs' minified property names to their public API names, which
    * are their aliases if any, or their original unminified property names
@@ -240,11 +232,6 @@ export interface DirectiveDefArgs<T> {
    */
   inputs?: {[P in keyof T]?: string};
 
-  /**
-   * TODO: Remove per https://github.com/angular/angular/issues/22591
-   */
-  inputsPropertyName?: {[P in keyof T]?: string};
-
   /**
    * A map of output names.
    *

packages/core/test/render3/common_with_def.ts

@@ -19,7 +19,7 @@ NgForOf.ngDirectiveDef = defineDirective({
   factory: () => new NgForOfDef(
                injectViewContainerRef(), injectTemplateRef(),
                directiveInject(IterableDiffers, InjectFlags.Default, defaultIterableDiffers)),
-  features: [NgOnChangesFeature],
+  features: [NgOnChangesFeature()],
   inputs: {
     ngForOf: 'ngForOf',
     ngForTrackBy: 'ngForTrackBy',

packages/core/test/render3/compiler_canonical/life_cycle_spec.ts

@@ -45,8 +45,7 @@ describe('lifecycle hooks', () => {
       factory: function LifecycleComp_Factory() { return new LifecycleComp(); },
       template: function LifecycleComp_Template(ctx: $LifecycleComp$, cm: $boolean$) {},
       inputs: {nameMin: 'name'},
-      inputsPropertyName: {nameMin: 'nameMin'},
+      features: [$r3$.ɵNgOnChangesFeature({nameMin: 'nameMin'})]
-      features: [$r3$.ɵNgOnChangesFeature]
     });
     // /NORMATIVE
   }

packages/core/test/render3/define_spec.ts

@@ -30,7 +30,7 @@ describe('define', () => {
           static ngDirectiveDef = defineDirective({
             type: MyDirective,
             factory: () => new MyDirective(),
-            features: [NgOnChangesFeature],
+            features: [NgOnChangesFeature()],
             inputs: {valA: 'valA', valB: 'valB'}
           });
         }

packages/core/test/render3/di_spec.ts

@@ -420,7 +420,7 @@ describe('di', () => {
           type: IfDirective,
           factory: () => new IfDirective(injectTemplateRef(), injectViewContainerRef()),
           inputs: {myIf: 'myIf'},
-          features: [PublicFeature, NgOnChangesFeature]
+          features: [PublicFeature, NgOnChangesFeature()]
         });
       }
 

packages/core/test/render3/lifecycle_spec.ts

@@ -1987,9 +1987,8 @@ describe('lifecycles', () => {
           type: Component,
           tag: name,
           factory: () => new Component(),
-          features: [NgOnChangesFeature],
+          features: [NgOnChangesFeature({b: 'val2'})],
-          inputs: {a: 'val1', b: 'publicName'},
+          inputs: {a: 'val1', b: 'publicName'}, template
-          inputsPropertyName: {b: 'val2'}, template
         });
       };
     }
@@ -2007,9 +2006,8 @@ describe('lifecycles', () => {
       static ngDirectiveDef = defineDirective({
         type: Directive,
         factory: () => new Directive(),
-        features: [NgOnChangesFeature],
+        features: [NgOnChangesFeature({b: 'val2'})],
-        inputs: {a: 'val1', b: 'publicName'},
+        inputs: {a: 'val1', b: 'publicName'}
-        inputsPropertyName: {b: 'val2'}
       });
     }
 
@@ -2397,7 +2395,7 @@ describe('lifecycles', () => {
           tag: name,
           factory: () => new Component(),
           inputs: {val: 'val'}, template,
-          features: [NgOnChangesFeature]
+          features: [NgOnChangesFeature()]
         });
       };
     }