Fix some bugs found during testing

pgonzal · pgonzal · commit f72932d703e4 · 2018-05-14T16:54:08.000-07:00
diff --git a/apps/api-documenter/src/yaml/YamlDocumenter.ts b/apps/api-documenter/src/yaml/YamlDocumenter.ts
@@ -312,7 +312,7 @@ export class YamlDocumenter {
       if (!yamlItem.remarks) {
         yamlItem.remarks = sealedMessage;
       } else {
-        yamlItem.remarks += sealedMessage + '\n\n' + yamlItem.remarks;
+        yamlItem.remarks = sealedMessage + '\n\n' + yamlItem.remarks;
       }
     }
   }
diff --git a/apps/api-extractor/src/aedoc/ApiDocumentation.ts b/apps/api-extractor/src/aedoc/ApiDocumentation.ts
@@ -184,6 +184,11 @@ export class ApiDocumentation {
     this.reportError = errorLogger;
     this.parameters = {};
     this.warnings = warnings;
+
+    this.isSealed = false;
+    this.isVirtual = false;
+    this.isOverride = false;
+
     this._parseDocs();
   }
 
diff --git a/apps/api-extractor/src/generators/ApiFileGenerator.ts b/apps/api-extractor/src/generators/ApiFileGenerator.ts
@@ -214,7 +214,7 @@ export class ApiFileGenerator extends AstItemVisitor {
         footerParts.push('@virtual');
       }
 
-      if (astItem.documentation.isSealed) {
+      if (astItem.documentation.isOverride) {
         footerParts.push('@override');
       }
 

Original file line number	Diff line number	Diff line change
`@@ -312,7 +312,7 @@ export class YamlDocumenter {`
`312`	`312`	`if (!yamlItem.remarks) {`
`313`	`313`	`yamlItem.remarks = sealedMessage;`
`314`	`314`	`} else {`
`315`		`- yamlItem.remarks += sealedMessage + '\n\n' + yamlItem.remarks;`
	`315`	`+ yamlItem.remarks = sealedMessage + '\n\n' + yamlItem.remarks;`
`316`	`316`	`}`
`317`	`317`	`}`
`318`	`318`	`}`
Original file line number	Diff line number	Diff line change
`@@ -214,7 +214,7 @@ export class ApiFileGenerator extends AstItemVisitor {`
`214`	`214`	`footerParts.push('@virtual');`
`215`	`215`	`}`
`216`	`216`
`217`		`- if (astItem.documentation.isSealed) {`
	`217`	`+ if (astItem.documentation.isOverride) {`
`218`	`218`	`footerParts.push('@override');`
`219`	`219`	`}`
`220`	`220`