0.2.8

steveruizok · Aug 2, 2022 · f765ef9 · f765ef9
1 parent 0bfb641
commit f765ef9
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 23 deletions.
diff --git a/packages/liquorstore/CHANGELOG.md b/packages/liquorstore/CHANGELOG.md
@@ -1,5 +1,11 @@
 # trashly-react
 
+## 0.2.8
+
+### Patch Changes
+
+- Fix returns on undo / redo
+
 ## 0.2.7
 
 ### Patch Changes

diff --git a/packages/liquorstore/package.json b/packages/liquorstore/package.json
@@ -1,6 +1,6 @@
 {
   "name": "liquorstore",
-  "version": "0.2.7",
+  "version": "0.2.8",
   "description": "A reactive store for React.",
   "license": "MIT",
   "author": {

diff --git a/packages/liquorstore/src/lib/LiquorStore.ts b/packages/liquorstore/src/lib/LiquorStore.ts
@@ -181,6 +181,7 @@ export class LiquorStore<T extends Record<string, any>> extends EventEmitter {
     this.willChange()
     this.current = next
     this.didChange(patch)
+    return this
   }
 
   /**
@@ -193,17 +194,11 @@ export class LiquorStore<T extends Record<string, any>> extends EventEmitter {
    */
   update = (fn: (state: T) => void) => {
     this.willChange()
-
     const tNext = Object.assign({}, this.current)
-
     fn(tNext)
-
     this.current = this.processState(tNext)
-
     const patch = diff(this.prev, this.current)
-
     this.didChange(patch)
-
     return this
   }
 
@@ -261,16 +256,16 @@ export class LiquorStore<T extends Record<string, any>> extends EventEmitter {
       this.isPaused = false
     }
 
-    if (!this.canUndo) return
+    if (this.canUndo) {
+      const patch = this.history[this.pointer]
 
-    const patch = this.history[this.pointer]
-
-    this.pointer--
-    this.prev = this.current
-    this.current = this.applyPatch(patch, true)
+      this.pointer--
+      this.prev = this.current
+      this.current = this.applyPatch(patch, true)
 
-    this.emit("undo")
-    this.notifySubscriptions(patch)
+      this.emit("undo")
+      this.notifySubscriptions(patch)
+    }
 
     return this
   }
@@ -294,17 +289,17 @@ export class LiquorStore<T extends Record<string, any>> extends EventEmitter {
       this.isPaused = false
     }
 
-    if (!this.canRedo) return
-
-    this.pointer++
+    if (this.canRedo) {
+      this.pointer++
 
-    const patch = this.history[this.pointer]
+      const patch = this.history[this.pointer]
 
-    this.prev = this.current
-    this.current = this.applyPatch(patch)
+      this.prev = this.current
+      this.current = this.applyPatch(patch)
 
-    this.emit("redo")
-    this.notifySubscriptions(patch)
+      this.emit("redo")
+      this.notifySubscriptions(patch)
+    }
 
     return this
   }