let post_link use original post title as title attribute

hexojs · May 18, 2022 · bc06aa2 · bc06aa2
1 parent afc4d7f
commit bc06aa2
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/lib/plugins/tag/post_link.js b/lib/plugins/tag/post_link.js
@@ -27,8 +27,8 @@ module.exports = ctx => {
     if (!post) return error;
 
     let title = args.length ? args.join(' ') : post.title;
-    const attrTitle = escapeHTML(title);
-    if (escape === 'true') title = attrTitle;
+    const attrTitle = escapeHTML(post.title);
+    if (escape === 'true') title = escapeHTML(title);
 
     const link = encodeURL(resolve(ctx.config.root, post.path));
 

diff --git a/test/scripts/tags/post_link.js b/test/scripts/tags/post_link.js
@@ -33,7 +33,7 @@ describe('post_link', () => {
   });
 
   it('title', () => {
-    postLink(['foo', 'test']).should.eql('<a href="/foo/" title="test">test</a>');
+    postLink(['foo', 'test']).should.eql('<a href="/foo/" title="Hello world">test</a>');
   });
 
   it('should escape tag in title by default', () => {
@@ -45,7 +45,7 @@ describe('post_link', () => {
   });
 
   it('should escape tag in custom title', () => {
-    postLink(['title-with-tag', '<test>', 'title', 'true']).should.eql('<a href="/title-with-tag/" title="&lt;test&gt; title">&lt;test&gt; title</a>');
+    postLink(['title-with-tag', '<test>', 'title', 'true']).should.eql('<a href="/title-with-tag/" title="&quot;Hello&quot; &lt;new world&gt;!">&lt;test&gt; title</a>');
   });
 
   it('should not escape tag in title', () => {
@@ -54,7 +54,7 @@ describe('post_link', () => {
 
   it('should not escape tag in custom title', () => {
     postLink(['title-with-tag', 'This is a <b>Bold</b> "statement"', 'false'])
-      .should.eql('<a href="/title-with-tag/" title="This is a &lt;b&gt;Bold&lt;&#x2F;b&gt; &quot;statement&quot;">This is a <b>Bold</b> "statement"</a>');
+      .should.eql('<a href="/title-with-tag/" title="&quot;Hello&quot; &lt;new world&gt;!">This is a <b>Bold</b> "statement"</a>');
   });
 
   it('no slug', () => {