fix(router): generate links for router-link with baseHref

modules/angular2/src/router/location.js

@@ -27,6 +27,13 @@ export class Location {
     return this._stripBaseHref(stripIndexHtml(url));
   }
 
+  normalizeAbsolutely(url) {
+    if (url[0] != '/') {
+      url = '/' + url;
+    }
+    return this._addBaseHref(url);
+  }
+
   _stripBaseHref(url) {
     if (this._baseHref.length > 0 && StringWrapper.startsWith(url, this._baseHref)) {
       return StringWrapper.substring(url, this._baseHref.length);
@@ -34,8 +41,15 @@ export class Location {
     return url;
   }
 
+  _addBaseHref(url) {
+    if (!StringWrapper.startsWith(url, this._baseHref)) {
+      return this._baseHref + url;
+    }
+    return url;
+  }
+
   go(url:string) {
-    var finalUrl = url[0] == '/' ? url : this._baseHref + '/' + url;
+    var finalUrl = this.normalizeAbsolutely(url);
     this._browserLocation.pushState(null, '', finalUrl);
   }
 

modules/angular2/src/router/router_link.js

@@ -6,6 +6,7 @@ import {isPresent} from 'angular2/src/facade/lang';
 import {DOM} from 'angular2/src/dom/dom_adapter';
 
 import {Router} from './router';
+import {Location} from './location';
 
 /**
  * The RouterLink directive lets you link to specific parts of your app.
@@ -41,11 +42,13 @@ export class RouterLink {
   _route:string;
   _params:any;
   _router:Router;
+  _location:Location;
   _href:string;
 
-  constructor(elementRef:ElementRef, router:Router) {
+  constructor(elementRef:ElementRef, router:Router, location:Location) {
     this._domEl = elementRef.domElement;
     this._router = router;
+    this._location = location;
     this._params = StringMapWrapper.create();
     DOM.on(this._domEl, 'click', (evt) => {
       evt.preventDefault();
@@ -64,10 +67,10 @@ export class RouterLink {
   onAllChangesDone() {
     if (isPresent(this._route) && isPresent(this._params)) {
       var newHref = this._router.generate(this._route, this._params);
-      this._href = newHref;
+      this._href = this._location.normalizeAbsolutely(newHref);
       // Keeping the link on the element to support contextual menu `copy link`
       // and other in-browser affordances.
-      DOM.setAttribute(this._domEl, 'href', newHref);
+      DOM.setAttribute(this._domEl, 'href', this._href);
     }
   }
 }