fix($location): retain original encoded path

Change location to retain the expected path encoding. The decode/encode cycle
for the location path loses data as %3B (a semi-colon in the path) and ; (a
sub-delim in the URI for path parameters) are considered to be different.

Author: brettporter

src/Angular.js

@@ -1065,7 +1065,6 @@ function toKeyValue(obj) {
  */
 function encodeUriSegment(val) {
   return encodeUriQuery(val, true).
-             replace(/%3B/gi, ';').
              replace(/%26/gi, '&').
              replace(/%3D/gi, '=').
              replace(/%2B/gi, '+');

src/ng/location.js

@@ -37,15 +37,17 @@ function parseAppUrl(relativeUrl, locationObj, appBase) {
     relativeUrl = '/' + relativeUrl;
   }
   var match = urlResolve(relativeUrl, appBase);
-  locationObj.$$path = decodeURIComponent(prefixed && match.pathname.charAt(0) === '/' ?
-      match.pathname.substring(1) : match.pathname);
-  locationObj.$$search = parseKeyValue(match.search);
-  locationObj.$$hash = decodeURIComponent(match.hash);
+  locationObj.pathEncoded = prefixed && match.pathname.charAt(0) === '/' ?
+      match.pathname.substring(1) : match.pathname;
 
   // make sure path starts with '/';
-  if (locationObj.$$path && locationObj.$$path.charAt(0) != '/') {
-    locationObj.$$path = '/' + locationObj.$$path;
+  if (locationObj.pathEncoded && locationObj.pathEncoded.charAt(0) != '/') {
+    locationObj.pathEncoded = '/' + locationObj.pathEncoded;
   }
+
+  locationObj.$$path = decodeURIComponent(locationObj.pathEncoded);
+  locationObj.$$search = parseKeyValue(match.search);
+  locationObj.$$hash = decodeURIComponent(match.hash);
 }
 
 
@@ -123,7 +125,7 @@ function LocationHtml5Url(appBase, basePrefix) {
     var search = toKeyValue(this.$$search),
         hash = this.$$hash ? '#' + encodeUriSegment(this.$$hash) : '';
 
-    this.$$url = encodePath(this.$$path) + (search ? '?' + search : '') + hash;
+    this.$$url = this.pathEncoded + (search ? '?' + search : '') + hash;
     this.$$absUrl = appBaseNoFile + this.$$url.substr(1); // first char is always '/'
   };
 
@@ -190,7 +192,7 @@ function LocationHashbangUrl(appBase, hashPrefix) {
     var search = toKeyValue(this.$$search),
         hash = this.$$hash ? '#' + encodeUriSegment(this.$$hash) : '';
 
-    this.$$url = encodePath(this.$$path) + (search ? '?' + search : '') + hash;
+    this.$$url = this.pathEncoded + (search ? '?' + search : '') + hash;
     this.$$absUrl = appBase + (this.$$url ? hashPrefix + this.$$url : '');
   };
 
@@ -350,8 +352,10 @@ LocationHashbangInHtml5Url.prototype =
    * @param {string=} path New path
    * @return {string} path
    */
-  path: locationGetterSetter('$$path', function(path) {
-    return path.charAt(0) == '/' ? path : '/' + path;
+  path: locationGetterSetter('$$path', function(path, locationObj) {
+    path = path.charAt(0) == '/' ? path : '/' + path;
+    locationObj.pathEncoded = encodePath(path);
+    return path;
   }),
 
   /**
@@ -446,7 +450,7 @@ function locationGetterSetter(property, preprocess) {
     if (isUndefined(value))
       return this[property];
 
-    this[property] = preprocess(value);
+    this[property] = preprocess(value, this);
     this.$$compose();
 
     return this;

test/AngularSpec.js

@@ -569,12 +569,12 @@ describe('angular', function() {
         toEqual("-_.!~*'()%20-_.!~*'()");
 
       //don't encode the rest of pchar'
-      expect(encodeUriSegment(':;@&=+$, :;@&=+$,')).
-        toEqual(':;@&=+$,%20:;@&=+$,');
+      expect(encodeUriSegment(':@&=+$, :@&=+$,')).
+        toEqual(':@&=+$,%20:@&=+$,');
 
-      //encode '/', and ' ''
-      expect(encodeUriSegment('/ /')).
-        toEqual('%2F%20%2F');
+      //encode '/', ';' and ' ''
+      expect(encodeUriSegment('/; /;')).
+        toEqual('%2F%3B%20%2F%3B');
     });
   });