fix(app): URL parsing when using @link for @inputs

vogloblinsky · vogloblinsky · commit 95e4945f647f · 2021-10-12T09:59:24.000+02:00
fix #1125
diff --git a/src/app/compiler/angular/deps/helpers/class-helper.ts b/src/app/compiler/angular/deps/helpers/class-helper.ts
@@ -1412,11 +1412,13 @@ export class ClassHelper {
                         _return.jsdoctags = markedtags(jsdoctags[0].tags);
                     }
                     if (typeof property.jsDoc[0].comment !== 'undefined') {
-                        const rawDescription = this.jsdocParserUtil.parseJSDocNode(
-                            property.jsDoc[0]
+                        const comment = this.jsdocParserUtil.getMainCommentOfNode(
+                            property,
+                            sourceFile
                         );
-                        _return.rawdescription = rawDescription;
-                        _return.description = marked(rawDescription);
+                        const cleanedDescription = this.jsdocParserUtil.parseComment(comment);
+                        _return.rawdescription = cleanedDescription;
+                        _return.description = marked(cleanedDescription);
                     }
                 }
             }
diff --git a/test/fixtures/todomvc-ng2/src/app/header/header.component.ts b/test/fixtures/todomvc-ng2/src/app/header/header.component.ts
@@ -49,7 +49,7 @@ export class HeaderComponent {
     }
 
     /**
-     * Setter of _fullName
+     * Setter of _fullName {@link https://compodoc.app/}
      * @param  {string} newName The new name
      */
     @Input()
diff --git a/test/src/cli/cli-generation-big-app.spec.ts b/test/src/cli/cli-generation-big-app.spec.ts
@@ -870,4 +870,9 @@ describe('CLI simple generation - big app', () => {
         expect(file).to.contain('or link to <a href="../classes/Todo.html">Todo');
         expect(file).to.contain('ore link to <a href="../classes/Todo.html">Todo');
     });
+
+    it('should support JSDoc @link for inputs', () => {
+        let file = read(distFolder + '/components/HeaderComponent.html');
+        expect(file).to.contain('_fullName <a href="https://compodoc.app/">https://compodoc.app/');
+    });
 });

Original file line number	Diff line number	Diff line change
`@@ -1412,11 +1412,13 @@ export class ClassHelper {`
`1412`	`1412`	`_return.jsdoctags = markedtags(jsdoctags[0].tags);`
`1413`	`1413`	`}`
`1414`	`1414`	`if (typeof property.jsDoc[0].comment !== 'undefined') {`
`1415`		`- const rawDescription = this.jsdocParserUtil.parseJSDocNode(`
`1416`		`- property.jsDoc[0]`
	`1415`	`+ const comment = this.jsdocParserUtil.getMainCommentOfNode(`
	`1416`	`+ property,`
	`1417`	`+ sourceFile`
`1417`	`1418`	`);`
`1418`		`- _return.rawdescription = rawDescription;`
`1419`		`- _return.description = marked(rawDescription);`
	`1419`	`+ const cleanedDescription = this.jsdocParserUtil.parseComment(comment);`
	`1420`	`+ _return.rawdescription = cleanedDescription;`
	`1421`	`+ _return.description = marked(cleanedDescription);`
`1420`	`1422`	`}`
`1421`	`1423`	`}`
`1422`	`1424`	`}`
Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ export class HeaderComponent {`
`49`	`49`	`}`
`50`	`50`
`51`	`51`	`/**`
`52`		`- * Setter of _fullName`
	`52`	`+ * Setter of _fullName {@link https://compodoc.app/}`
`53`	`53`	`* @param {string} newName The new name`
`54`	`54`	`*/`
`55`	`55`	`@Input()`