refactored $location service so that it correctly updates under all conditions

5 files changed, 63 insertions, 47 deletions

diff --git a/src/Angular.js b/src/Angular.js
index 32e3ccf7..80acddf0 100644
--- a/src/Angular.js
+++ b/src/Angular.js
@@ -300,10 +300,10 @@ function bind(_this, _function) {
       } :
       function() {
         return _function.apply(_this, curryArgs.concat(slice.call(arguments, 0, arguments.length)));
-      }
+      };
   } else {
-    // in IE, native methonds ore not functions and so they can not be bound (but they don't need to be)
-    return function(a, b, c, d, e){ return _function(a, b, c, d, e); };
+    // in IE, native methods ore not functions and so they can not be bound (but they don't need to be)
+    return _function;
   }
 }
 
diff --git a/src/angular-bootstrap.js b/src/angular-bootstrap.js
index 90e1104e..e055371a 100644
--- a/src/angular-bootstrap.js
+++ b/src/angular-bootstrap.js
@@ -22,16 +22,14 @@
  * THE SOFTWARE.
  */
 (function(previousOnLoad){
-  var filename = /(.*)\/angular-(.*).js(#(.*))?/,
+  var filename = /(.*)\/angular-(.*).js(#.*)?/,
       scripts = document.getElementsByTagName("SCRIPT"),
       serverPath,
-      config,
       match;
   for(var j = 0; j < scripts.length; j++) {
     match = (scripts[j].src || "").match(filename);
     if (match) {
       serverPath = match[1];
-      config = match[4];
     }
   }
 
@@ -63,7 +61,7 @@
     try {
       if (previousOnLoad) previousOnLoad();
     } catch(e) {}
-    angularInit(parseKeyValue(config));
+    angularInit(parseKeyValue(angularJsConfig(document)));
   };
 
 })(window.onload);
diff --git a/src/formatters.js b/src/formatters.js
index 40462cf3..ca1ce83e 100644
--- a/src/formatters.js
+++ b/src/formatters.js
@@ -5,6 +5,7 @@ var NUMBER = /^\s*[-+]?\d*(\.\d*)?\s*$/;
 
 extend(angularFormatter, {
   'noop':formatter(identity, identity),
+  'json':formatter(toJson, fromJson),
   'boolean':formatter(toString, toBoolean),
   'number':formatter(toString,
       function(obj){
diff --git a/src/jqLite.js b/src/jqLite.js
index 04682754..22b3c070 100644
--- a/src/jqLite.js
+++ b/src/jqLite.js
@@ -107,7 +107,7 @@ JQLite.prototype = {
           if (!event.preventDefault) {
             event.preventDefault = function(){
               event.returnValue = false;
-            }
+            };
           }
           foreach(eventHandler.fns, function(fn){
             fn.call(self, event);
diff --git a/src/services.js b/src/services.js
index 106f8954..3dd7df09 100644
--- a/src/services.js
+++ b/src/services.js
@@ -7,61 +7,78 @@ var URL_MATCH = /^(file|ftp|http|https):\/\/(\w+:{0,1}\w*@)?([\w\.-]*)(:([0-9]+)
 var HASH_MATCH = /^([^\?]*)?(\?([^\?]*))?$/;
 var DEFAULT_PORTS = {'http': 80, 'https': 443, 'ftp':21};
 angularService("$location", function(browser){
-  var scope = this, location = {parse:parseUrl, toString:toString};
-  var lastHash, lastUrl;
+  var scope = this,
+      location = {parse:parseUrl, toString:toString, update:update},
+      lastLocation = {};
+
+  browser.watchUrl(function(url){
+    update(url);
+    scope.$root.$eval();
+  });
+  this.$onEval(PRIORITY_FIRST, update);
+  this.$onEval(PRIORITY_LAST, update);
+  update(browser.getUrl());
+  return location;
+
+  function update(href){
+    if (href) {
+      parseUrl(href);
+    } else {
+      href = check('href') || check('protocol', '://', 'host', ':', 'port', '', 'path', '?', 'search');
+      var hash = check('hash');
+      if (isUndefined(hash)) hash = check('hashPath', '?', 'hashSearch');
+      if (isDefined(hash)) {
+        href = (href || location.href).split('#')[0];
+        href+= '#' + hash;
+      }
+      if (isDefined(href)) {
+        parseUrl(href);
+        browser.setUrl(href);
+      }
+    }
+  }
+
+  function check() {
+    var i = -1,
+        length=arguments.length,
+        name, seperator, parts = [],
+        value, same = true;
+    for(; i<length; i = i+2) {
+      parts.push(seperator = (arguments[i] || ''));
+      name = arguments[i + 1];
+      value=location[name];
+      parts.push(typeof value == 'object' ? toKeyValue(value) : value);
+      same = same && equals(lastLocation[name], value);
+    }
+    return same ? undefined : parts.join('');
+  }
+
   function parseUrl(url){
     if (isDefined(url)) {
       var match = URL_MATCH.exec(url);
       if (match) {
-        location.href = url;
+        location.href = url.replace('#$', '');
         location.protocol = match[1];
         location.host = match[3] || '';
-        location.port = match[5] || DEFAULT_PORTS[location.href] || null;
+        location.port = match[5] || DEFAULT_PORTS[location.protocol] || null;
         location.path = match[6];
         location.search = parseKeyValue(match[8]);
         location.hash = match[9] || '';
         if (location.hash)
           location.hash = location.hash.substr(1);
-        parseHash(location.hash);
+        match = HASH_MATCH.exec(location.hash);
+        location.hashPath = match[1] || '';
+        location.hashSearch = parseKeyValue(match[3]);
+
+        copy(location, lastLocation);
       }
     }
   }
-  function parseHash(hash) {
-    var match = HASH_MATCH.exec(hash);
-    location.hashPath = match[1] || '';
-    location.hashSearch = parseKeyValue(match[3]);
-    lastHash = hash;
-  }
+
   function toString() {
-    if (lastHash === location.hash) {
-      var hashKeyValue = toKeyValue(location.hashSearch),
-          hash = (location.hashPath ? location.hashPath : '') + (hashKeyValue ? '?' + hashKeyValue : ''),
-          url = location.href.split('#')[0] + '#' + (hash ? hash : '');
-      if (url !== location.href) parseUrl(url);
-      return url;
-    } else {
-      parseUrl(location.href.split('#')[0] + '#' + location.hash);
-      return toString();
-    }
+    update();
+    return location.href;
   }
-  browser.watchUrl(function(url){
-    parseUrl(url);
-    scope.$root.$eval();
-  });
-  parseUrl(browser.getUrl());
-  this.$onEval(PRIORITY_FIRST, function(){
-    if (location.hash != lastHash) {
-      parseHash(location.hash);
-    }
-  });
-  this.$onEval(PRIORITY_LAST, function(){
-    var url = toString();
-    if (lastUrl != url) {
-      browser.setUrl(url);
-      lastUrl = url;
-    }
-  });
-  return location;
 }, {inject: ['$browser']});
 
 angularService("$log", function($window){