fix($location): consider baseHref in relative link for legacy browsers

src/ng/location.js

@@ -668,6 +668,13 @@ function $LocationProvider(){
         if (href.indexOf('://') < 0) {         // Ignore absolute URLs
           var prefix = '#' + hashPrefix;
           if (href[0] == '/') {
+            // Account for base href already present in appBase
+            if (baseHref && href.indexOf(baseHref) === 0) {
+              href = href.substr(baseHref.length);
+              if (!href || href[0] != '/') {
+                href = '/' + href;
+              }
+            }
             // absolute path - replace old path
             absHref = appBase + prefix + href;
           } else if (href[0] == '#') {

test/ng/locationSpec.js

@@ -1330,6 +1330,36 @@ describe('$location', function() {
         }).not.toThrow();
       });
     });
+
+
+    it('should transform the url correctly when a base path is present and html5mode is enabled but not supported', function() {
+      var base;
+      module(function() {
+        return function($browser) {
+          base = 'http://server/foo';
+          $browser.url(base);
+          $browser.$$baseHref = '/foo';
+        };
+      });
+      inject(initService(true, '', false), function($rootScope, $compile, $browser, $rootElement, $document, $location) {
+        // we need to do this otherwise we can't simulate events
+        $document.find('body').append($rootElement);
+
+        var element = $compile('<a href="/foo/view1">v1</a><a href="/foo/bar/view2">v2</a>')($rootScope);
+        $rootElement.append(element);
+        var av1 = $rootElement.find('a').eq(0);
+        var av2 = $rootElement.find('a').eq(1);
+
+
+        browserTrigger(av1, 'click');
+        expect($browser.url()).toEqual(base + '#/view1');
+
+        browserTrigger(av2, 'click');
+        expect($browser.url()).toEqual(base + '#/bar/view2');
+
+        $rootElement.remove();
+      });
+    });
   });