Address feedback

Author: devversion

tools/dgeni/common/dgeni-definitions.ts

@@ -1,7 +1,6 @@
 import {ClassExportDoc} from 'dgeni-packages/typescript/api-doc-types/ClassExportDoc';
 import {ClassLikeExportDoc} from 'dgeni-packages/typescript/api-doc-types/ClassLikeExportDoc';
 import {PropertyMemberDoc} from 'dgeni-packages/typescript/api-doc-types/PropertyMemberDoc';
-import {DirectiveMetadata} from './directive-metadata';
 import {NormalizedMethodMemberDoc} from './normalize-method-parameters';
 
 /** Extended Dgeni class-like document that includes separated class members. */
@@ -18,7 +17,7 @@ export interface CategorizedClassDoc extends ClassExportDoc, CategorizedClassLik
   isNgModule: boolean;
   directiveExportAs?: string | null;
   directiveSelectors?: string[];
-  directiveMetadata: DirectiveMetadata | null;
+  directiveMetadata: Map<string, any> | null;
   extendedDoc: ClassLikeExportDoc | null;
 }
 

tools/dgeni/common/directive-metadata.ts

@@ -7,13 +7,12 @@ import {
   StringLiteral, SyntaxKind
 } from 'typescript';
 
-export type DirectiveMetadata = Map<string, any>;
-
 /**
  * Determines the component or directive metadata from the specified Dgeni class doc. The resolved
  * directive metadata will be stored in a Map.
  *
- * Currently only string literal assignments and array literal assignments are supported.
+ * Currently only string literal assignments and array literal assignments are supported. Other
+ * value types are not necessary because they are not needed for any user-facing documentation.
  *
  * ```ts
  * @Component({
@@ -23,7 +22,7 @@ export type DirectiveMetadata = Map<string, any>;
  * export class MyComponent {}
  * ```
  */
-export function getDirectiveMetadata(classDoc: CategorizedClassDoc): DirectiveMetadata | null {
+export function getDirectiveMetadata(classDoc: CategorizedClassDoc): Map<string, any> | null {
   const declaration = classDoc.symbol.valueDeclaration;
 
   if (!declaration || !declaration.decorators) {

tools/dgeni/common/property-bindings.ts

@@ -1,5 +1,4 @@
 import {PropertyMemberDoc} from 'dgeni-packages/typescript/api-doc-types/PropertyMemberDoc';
-import {DirectiveMetadata} from './directive-metadata';
 import {hasMemberDecorator} from './decorators';
 
 /** Interface that describes an Angular property binding. Can be either an input or output. */
@@ -12,7 +11,7 @@ export interface PropertyBinding {
  * Detects whether the specified property member is an input. If the property is an input, the
  * alias and input name will be returned.
  */
-export function getInputBindingData(doc: PropertyMemberDoc, metadata: DirectiveMetadata)
+export function getInputBindingData(doc: PropertyMemberDoc, metadata: Map<string, any>)
     : PropertyBinding | undefined {
   return getBindingPropertyData(doc, metadata, 'inputs', 'Input');
 }
@@ -21,7 +20,7 @@ export function getInputBindingData(doc: PropertyMemberDoc, metadata: DirectiveM
  * Detects whether the specified property member is an output. If the property is an output, the
  * alias and output name will be returned.
  */
-export function getOutputBindingData(doc: PropertyMemberDoc, metadata: DirectiveMetadata)
+export function getOutputBindingData(doc: PropertyMemberDoc, metadata: Map<string, any>)
     : PropertyBinding | undefined {
   return getBindingPropertyData(doc, metadata, 'outputs', 'Output');
 }
@@ -30,7 +29,7 @@ export function getOutputBindingData(doc: PropertyMemberDoc, metadata: Directive
  * Method that detects the specified type of property binding (either "output" or "input") from
  * the directive metadata or from the associated decorator on the property.
  */
-function getBindingPropertyData(doc: PropertyMemberDoc, metadata: DirectiveMetadata,
+function getBindingPropertyData(doc: PropertyMemberDoc, metadata: Map<string, any>,
                                 propertyName: string, decoratorName: string) {
 
   if (metadata) {