fix(url_for): absolute path is processed by relative_url (#262)

* fix(url_for): absolute path is processed by relative_url

* test: absolute urls are processed by relative_url

Author: zkqiang

lib/url_for.js

@@ -9,6 +9,8 @@ const Cache = require('./cache');
 const cache = new Cache();
 
 function urlForHelper(path = '/', options) {
+  if (/^(#|\/\/|http(s)?:)/.test(path)) return path;
+
   const { config } = this;
 
   options = Object.assign({
@@ -28,8 +30,6 @@ function urlForHelper(path = '/', options) {
 
   // cacheId is designed to works across different hexo.config & options
   return cache.apply(`${config.url}-${root}-${prettyUrlsOptions.trailing_index}-${prettyUrlsOptions.trailing_html}-${path}`, () => {
-    if (/^(#|\/\/|http(s)?:)/.test(path)) return path;
-
     const sitehost = parse(config.url).hostname || config.url;
     const data = new URL(path, `http://${sitehost}`);
 

test/url_for.spec.js

@@ -128,6 +128,20 @@ describe('url_for', () => {
     });
   });
 
+  it('absolute url - should not be processed by relative_url', () => {
+    ctx.config.relative_link = true;
+    [
+      'https://hexo.io/',
+      '//google.com/',
+      'https://hexo.io/docs/index.html',
+      'http://example.com/foo/bar/',
+      'https://example.com/foo/bar/'
+    ].forEach(url => {
+      urlFor(url).should.eql(url);
+    });
+    ctx.config.relative_link = false;
+  });
+
   it('only hash', () => {
     urlFor('#test').should.eql('#test');
   });