refactor(external_link): run exclude array once a page

hexojs · Nov 6, 2019 · 1c48f6b · 1c48f6b
1 parent 60acb95
commit 1c48f6b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/lib/plugins/filter/after_post_render/external_link.js b/lib/plugins/filter/after_post_render/external_link.js
@@ -11,8 +11,7 @@ const urlObj = (str) => {
 };
 
 const isExternal = (url, config) => {
-  const exclude = Array.isArray(config.external_link.exclude) ? config.external_link.exclude
-    : [config.external_link.exclude];
+  const { exclude } = config.external_link;
   const data = urlObj(url);
   const sitehost = urlObj(config.url).hostname || config.url;
 
@@ -33,6 +32,8 @@ const isExternal = (url, config) => {
 
 function externalLinkFilter(data) {
   const { config } = this;
+  config.external_link.exclude = Array.isArray(config.external_link.exclude) ? config.external_link.exclude
+    : [config.external_link.exclude];
 
   if (typeof config.external_link === 'undefined' || typeof config.external_link === 'object'
     || config.external_link === true) {

diff --git a/lib/plugins/filter/after_render/external_link.js b/lib/plugins/filter/after_render/external_link.js
@@ -17,8 +17,7 @@ const urlObj = (str) => {
  * @returns {Boolean} True if the link doesn't have protocol or link has same host with config.url
  */
 const isExternal = (url, config) => {
-  const exclude = Array.isArray(config.external_link.exclude) ? config.external_link.exclude
-    : [config.external_link.exclude];
+  const { exclude } = config.external_link;
   const data = urlObj(url);
   const sitehost = urlObj(config.url).hostname || config.url;
 
@@ -39,6 +38,8 @@ const isExternal = (url, config) => {
 
 function externalLinkFilter(data) {
   const { config } = this;
+  config.external_link.exclude = Array.isArray(config.external_link.exclude) ? config.external_link.exclude
+    : [config.external_link.exclude];
 
   if (typeof config.external_link === 'undefined' || typeof config.external_link === 'object'
     || config.external_link === true) {