Now use the correct properties in the error message and fixed linter warnings

PissedCapslock · PissedCapslock · commit d5bca9eb6064 · 2019-10-21T10:51:18.000+02:00
diff --git a/apps/api-extractor-model/src/mixins/ApiItemContainerMixin.ts b/apps/api-extractor-model/src/mixins/ApiItemContainerMixin.ts
@@ -154,7 +154,8 @@ export function ApiItemContainerMixin<TBaseClass extends IApiItemConstructor>(ba
 
     public addMember(member: ApiItem): void {
       if (this[_membersByContainerKey].has(member.containerKey)) {
-        throw new Error(`Another member has already been added with the same name (${member.name}) and containerKey (${member.containterKey})`);
+        throw new Error(`Another member has already been added with the same name (${member.displayName})` +
+          ` and containerKey (${member.containerKey})`);
       }
 
       const existingParent: ApiItem | undefined = member.parent;

Original file line number	Diff line number	Diff line change
`@@ -154,7 +154,8 @@ export function ApiItemContainerMixin<TBaseClass extends IApiItemConstructor>(ba`
`154`	`154`
`155`	`155`	`public addMember(member: ApiItem): void {`
`156`	`156`	`if (this[_membersByContainerKey].has(member.containerKey)) {`
`157`		- throw new Error(`Another member has already been added with the same name (${member.name}) and containerKey (${member.containterKey})`);
	`157`	+ throw new Error(`Another member has already been added with the same name (${member.displayName})` +
	`158`	+ ` and containerKey (${member.containerKey})`);
`158`	`159`	`}`
`159`	`160`
`160`	`161`	`const existingParent: ApiItem \| undefined = member.parent;`