Merge pull request #246 from AprilArcus/cleanup-subscription

unsubscribe from provided store when AngularJS app is destroyed

Author: oliviernt

src/components/ngRedux.js

@@ -107,7 +107,7 @@ export default function ngReduxProvider() {
 
     const mergedStore = assign({}, store, { connect: Connector(store) });
 
-    if (_providedStore) wrapStore(_providedStore, mergedStore);
+    if (_providedStore) wrapStore(_providedStore, mergedStore, injector.get('$rootScope'));
 
     return mergedStore;
   };

src/components/storeWrapper.js

@@ -1,10 +1,11 @@
-export default function wrapStore(providedStore, ngReduxStore) {
-  providedStore.subscribe(() => {
+export default function wrapStore(providedStore, ngReduxStore, $rootScope) {
+  const unsubscribe = providedStore.subscribe(() => {
     let newState = providedStore.getState();
     ngReduxStore.dispatch({
       type: '@@NGREDUX_PASSTHROUGH',
       payload: newState
     });
   });
   providedStore.dispatch({ type: '@@NGREDUX_PASSTHROUGH_INIT' })
+  $rootScope.$on('$destroy', unsubscribe)
 }