feat(ng-widget): notify old url on location change events

Closes #24

Author: ofirdagan

app/scripts/app.js

@@ -11,15 +11,18 @@ angular.module('angularWidget', ['angularWidgetInternal'])
     //an internal route changes, effecting only the router inside that view.
     $provide.decorator('$rootScope', function ($delegate, $injector) {
       var next, last, originalBroadcast = $delegate.$broadcast;
+      var lastAbsUrl = '';
 
       //sending $locationChangeSuccess will cause another $routeUpdate
       //so we need this ugly flag to prevent call stack overflow
       var suspendListener = false;
 
       function suspendedNotify(widgets, $location) {
         suspendListener = true;
-        widgets.notifyWidgets('$locationChangeStart', $location.absUrl(), '');
-        widgets.notifyWidgets('$locationChangeSuccess', $location.absUrl(), '');
+        var absUrl = $location.absUrl();
+        widgets.notifyWidgets('$locationChangeStart', absUrl, lastAbsUrl);
+        widgets.notifyWidgets('$locationChangeSuccess', absUrl, lastAbsUrl);
+        lastAbsUrl = absUrl;
         suspendListener = false;
       }
 
@@ -91,7 +94,7 @@ angular.module('angularWidgetOnly', [])
   .run(function ($rootScope, $location) {
     //widget - since $location is shared and is not going to be instantiated
     //by the new injector of this widget, we send the $locationChangeSuccess
-    //ourselves to kickoff ng-rounte and ui-router ($location usually does that
+    //ourselves to kickoff ng-route and ui-router ($location usually does that
     //itself during instantiation)
     $rootScope.$evalAsync(function () {
       var ev = $rootScope.$broadcast('$locationChangeStart', $location.absUrl(), '');

test/spec/app.spec.js

@@ -57,6 +57,18 @@ describe('Unit testing routing hacks', function () {
     expect(notifyWidgets).toHaveBeenCalledWith('$locationChangeSuccess', 'http://server/', '');
   }));
 
+  it('should notify location change with old url', inject(function ($rootScope, $location) {
+    $rootScope.$broadcast('$routeUpdate');
+    expect(notifyWidgets).toHaveBeenCalledWith('$locationChangeStart', 'http://server/', '');
+    expect(notifyWidgets).toHaveBeenCalledWith('$locationChangeSuccess', 'http://server/', '');
+
+    $location.path('/bla');
+    $rootScope.$broadcast('$routeUpdate');
+
+    expect(notifyWidgets).toHaveBeenCalledWith('$locationChangeStart', 'http://server/#/bla', 'http://server/');
+    expect(notifyWidgets).toHaveBeenCalledWith('$locationChangeSuccess', 'http://server/#/bla', 'http://server/');
+  }));
+
   it('should mute route change if widget did not change', inject(function ($rootScope) {
     var eventSpy = jasmine.createSpy('$routeChangeSuccess');
     var eventSpyMuted = jasmine.createSpy('$routeChangeMuted');