unsubscribe from Redux when AngularJS app is destroyed

angular-redux · Apr 18, 2022 · 8bf7818 · 8bf7818
1 parent 243ec51
commit 8bf7818
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/components/ngRedux.js b/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;
   };

diff --git a/src/components/storeWrapper.js b/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)
 }