From 1e3460be0b6a697b92598f43c8e1ef49347f8962 Mon Sep 17 00:00:00 2001 From: Pawel Kozlowski Date: Fri, 31 Aug 2018 16:52:13 +0200 Subject: [PATCH] refactor(ivy): remove obsolete types (#25767) In the past factories could return an array with content queries but we no longer manage queries in factory functions. PR Close #25767 --- packages/core/src/render3/definition.ts | 2 +- packages/core/src/render3/interfaces/definition.ts | 5 +---- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/packages/core/src/render3/definition.ts b/packages/core/src/render3/definition.ts index 5f64e5672a..41546a186b 100644 --- a/packages/core/src/render3/definition.ts +++ b/packages/core/src/render3/definition.ts @@ -515,7 +515,7 @@ export const defineDirective = defineComponent as any as(directiveDefinition: /** * Factory method used to create an instance of directive. */ - factory: () => T | ({0: T} & any[]); /* trying to say T | [T, ...any] */ + factory: () => T; /** * Static attributes to set on host element. diff --git a/packages/core/src/render3/interfaces/definition.ts b/packages/core/src/render3/interfaces/definition.ts index 8c377fd56b..af19a7c161 100644 --- a/packages/core/src/render3/interfaces/definition.ts +++ b/packages/core/src/render3/interfaces/definition.ts @@ -127,11 +127,8 @@ export interface DirectiveDef extends BaseDef { /** * Factory function used to create a new directive instance. - * - * Usually returns the directive instance, but if the directive has a content query, - * it instead returns an array that contains the instance as well as content query data. */ - factory(): T|[T]; + factory(): T; /** * Function to create instances of content queries associated with a given directive.