Merge pull request hexojs#1406 from liuhongjiang/add_root_to_urlForHe…

…lper Prepend root path to urlForHelper
uiolee · Aug 2, 2015 · 14d2b9b · 14d2b9b
2 parents b1faffd + 5257363
commit 14d2b9b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/lib/plugins/helper/url_for.js b/lib/plugins/helper/url_for.js
@@ -26,13 +26,9 @@ function urlForHelper(path, options){
   }
 
   // Prepend root path
-  if(path === '/') {
-    return root;
-  } else if (path[0] !== '/'){
-    return root + path;
-  }
+  path = root + path;
 
-  return path;
+  return path.replace(/\/{2,}/g, '/');
 }
 
 module.exports = urlForHelper;
diff --git a/test/scripts/helpers/url_for.js b/test/scripts/helpers/url_for.js
@@ -14,10 +14,12 @@ describe('url_for', function(){
     ctx.config.root = '/';
     urlFor('index.html').should.eql('/index.html');
     urlFor('/').should.eql('/');
+    urlFor('/index.html').should.eql('/index.html');
 
     ctx.config.root = '/blog/';
     urlFor('index.html').should.eql('/blog/index.html');
     urlFor('/').should.eql('/blog/');
+    urlFor('/index.html').should.eql('/blog/index.html');
   });
 
   it('internal url (relative on)', function(){