Merge pull request #25 from shiftyp/bug/#24-nested-state

#24: Fixed nested state updates by creating new pausers for each tree node
shiftyp · Nov 6, 2016 · b8abdff · b8abdff
2 parents 37f46d0 + 449436d
commit b8abdff
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 7 deletions.
diff --git a/lib/state/tree.js b/lib/state/tree.js
@@ -151,15 +151,14 @@ export const createFinalTreeFromProvisionalNode = ({
 
 const createInitialTree = ({
   initialState,
-  pauser,
+  rootPauser,
   hookMap,
   createNode,
   provisional,
   provisionalNode
 }) => {
-  if (!pauser) {
-    pauser = new Rx.BehaviorSubject(true);
-  }
+  const pauser = new Rx.BehaviorSubject(true);
+  if (rootPauser) rootPauser.subscribe(pauser.onNext.bind(pauser));
   if (!hookMap) {
     hookMap = new WeakMap();
   }

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "redurx",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "description": "Redux'ish Functional State Management using RxJS",
   "keywords": [
     "redurx",

diff --git a/test/state/hooks.js b/test/state/hooks.js
@@ -8,11 +8,15 @@ test('should be able to hook into tree node', t => {
   const testVal = 42;
   const initialVal = {
     a: 1,
-    b: 2
+    b: {
+      val: 2
+    }
   };
   const finalVal = {
     a: 43,
-    b: 44
+    b: {
+      val: 53
+    }
   };
   const state = createState();
   const action = new Rx.Subject();