fix($location): don't call toString on null values

Author: jeffbcross

src/ng/location.js

@@ -416,7 +416,7 @@ LocationHashbangInHtml5Url.prototype =
    * @return {string} path
    */
   path: locationGetterSetter('$$path', function(path) {
-    path = path.toString();
+    path = path ? path.toString() : '';
     return path.charAt(0) == '/' ? path : '/' + path;
   }),
 
@@ -513,7 +513,7 @@ LocationHashbangInHtml5Url.prototype =
    * @return {string} hash
    */
   hash: locationGetterSetter('$$hash', function(hash) {
-    return hash.toString();
+    return hash ? hash.toString() : '';
   }),
 
   /**

test/ng/locationSpec.js

@@ -142,6 +142,14 @@ describe('$location', function() {
       expect(url.absUrl()).toBe('http://www.domain.com:9877/1?search=a&b=c&d#hash');
     });
 
+
+    it('path() should set to empty path on null value', function () {
+      url.path('/foo');
+      expect(url.path()).toBe('/foo');
+      url.path(null);
+      expect(url.path()).toBe('/');
+    });
+
     it('search() should accept string', function() {
       url.search('x=y&c');
       expect(url.search()).toEqual({x: 'y', c: true});
@@ -233,6 +241,13 @@ describe('$location', function() {
     });
 
 
+    it('hash() should accept null parameter', function() {
+      url.hash(null);
+      expect(url.hash()).toBe('');
+      expect(url.absUrl()).toBe('http://www.domain.com:9877/path/b?search=a&b=c&d');
+    });
+
+
     it('url() should change the path, search and hash', function() {
       url.url('/some/path?a=b&c=d#hhh');
       expect(url.url()).toBe('/some/path?a=b&c=d#hhh');