feat($state): allow disabling of transition notifications

Closes #401

Author: nateabele

src/state.js

@@ -264,7 +264,7 @@ function $StateProvider(   $urlRouterProvider,   $urlMatcherFactory,           $
     $state.transitionTo = function transitionTo(to, toParams, options) {
       if (!isDefined(options)) options = (options === true || options === false) ? { location: options } : {};
       toParams = toParams || {};
-      options = extend({ location: true, inherit: false, relative: null, $retry: false }, options);
+      options = extend({ location: true, inherit: false, relative: null, notify: true, $retry: false }, options);
 
       var from = $state.$current, fromParams = $state.params, fromPath = from.path;
 
@@ -331,8 +331,10 @@ function $StateProvider(   $urlRouterProvider,   $urlMatcherFactory,           $
       toParams = normalize(to.params, toParams || {});
 
       // Broadcast start event and cancel the transition if requested
-      evt = $rootScope.$broadcast('$stateChangeStart', to.self, toParams, from.self, fromParams);
-      if (evt.defaultPrevented) return TransitionPrevented;
+      if (options.notify) {
+        evt = $rootScope.$broadcast('$stateChangeStart', to.self, toParams, from.self, fromParams);
+        if (evt.defaultPrevented) return TransitionPrevented;
+      }
 
       // Resolve locals for the remaining states, but don't update any global state just
       // yet -- if anything fails to resolve the current state needs to remain untouched.
@@ -394,7 +396,9 @@ function $StateProvider(   $urlRouterProvider,   $urlMatcherFactory,           $
           }
         }
 
-        $rootScope.$broadcast('$stateChangeSuccess', to.self, toParams, from.self, fromParams);
+        if (options.notify) {
+          $rootScope.$broadcast('$stateChangeSuccess', to.self, toParams, from.self, fromParams);
+        }
 
         return $state.current;
       }, function (error) {

test/stateSpec.js

@@ -278,6 +278,36 @@ describe('state', function () {
       expect($state.current).toBe(D);
     }));
 
+    it('does not trigger $stateChangeSuccess when suppressed, but changes state', inject(function ($state, $q, $rootScope) {
+      initStateTo(E, { i: 'iii' });
+      var called;
+
+      $rootScope.$on('$stateChangeSuccess', function (ev, to, toParams, from, fromParams) {
+        called = true;
+      });
+
+      $state.transitionTo(D, { x: '1', y: '2' }, { notify: false });
+      $q.flush();
+
+      expect(called).toBeFalsy();
+      expect($state.current).toBe(D);
+    }));
+
+    it('does not trigger $stateChangeSuccess when suppressed, but updates params', inject(function ($state, $q, $rootScope) {
+      initStateTo(E, { x: 'iii' });
+      var called;
+
+      $rootScope.$on('$stateChangeSuccess', function (ev, to, toParams, from, fromParams) {
+        called = true;
+      });
+      $state.transitionTo(E, { i: '1', y: '2' }, { notify: false });
+      $q.flush();
+
+      expect(called).toBeFalsy();
+      expect($state.params.i).toBe('1');
+      expect($state.current).toBe(E);
+    }));
+
     it('is a no-op when passing the current state and identical parameters', inject(function ($state, $q) {
       initStateTo(A);
       var trans = $state.transitionTo(A, {}); // no-op