v13.0.0-53 -- distribution files

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "yjs",
-  "version": "13.0.0-54",
+  "version": "13.0.0-53",
   "lockfileVersion": 1,
   "requires": true,
   "dependencies": {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "yjs",
-  "version": "13.0.0-54",
+  "version": "13.0.0-53",
   "description": "A framework for real-time p2p shared editing on any data",
   "main": "./y.node.js",
   "browser": "./y.js",

src/Struct/Item.js

@@ -77,19 +77,18 @@ export default class Item {
     if (parent._redone !== null) {
       parent = parent._redone
       // find next cloned items
-      while (left !== null) {
+      while (left !== null && left._redone === null) {
-        if (left._redone !== null && left._redone._parent === parent) {
-          left = left._redone
-          break
-        }
         left = left._left
       }
-      while (right !== null) {
+      if (left !== null) {
-        if (right._redone !== null && right._redone._parent === parent) {
+        left = left._redone
-          right = right._redone
+      }
-        }
+      while (right !== null && right._redone === null) {
         right = right._right
       }
+      if (right !== null) {
+        right = right._redone
+      }
     }
     struct._origin = left
     struct._left = left

src/Util/UndoManager.js

@@ -73,7 +73,6 @@ export default class UndoManager {
     this._scope = scope
     this._undoing = false
     this._redoing = false
-    this._lastTransactionWasUndo = false
     const y = scope._y
     this.y = y
     y.on('afterTransaction', (y, transaction, remote) => {
@@ -81,12 +80,7 @@ export default class UndoManager {
         let reverseOperation = new ReverseOperation(y, transaction)
         if (!this._undoing) {
           let lastUndoOp = this._undoBuffer.length > 0 ? this._undoBuffer[this._undoBuffer.length - 1] : null
-          if (
+          if (lastUndoOp !== null && reverseOperation.created - lastUndoOp.created <= options.captureTimeout) {
-            this._redoing === false &&
-            this._lastTransactionWasUndo === false &&
-            lastUndoOp !== null &&
-            reverseOperation.created - lastUndoOp.created <= options.captureTimeout
-          ) {
             lastUndoOp.created = reverseOperation.created
             if (reverseOperation.toState !== null) {
               lastUndoOp.toState = reverseOperation.toState
@@ -96,14 +90,12 @@ export default class UndoManager {
             }
             reverseOperation.deletedStructs.forEach(lastUndoOp.deletedStructs.add, lastUndoOp.deletedStructs)
           } else {
-            this._lastTransactionWasUndo = false
             this._undoBuffer.push(reverseOperation)
           }
           if (!this._redoing) {
             this._redoBuffer = []
           }
         } else {
-          this._lastTransactionWasUndo = true
           this._redoBuffer.push(reverseOperation)
         }
       }

y.node.js

@@ -1,7 +1,7 @@
 
 /**
  * yjs - A framework for real-time p2p shared editing on any data
- * @version v13.0.0-54
+ * @version v13.0.0-53
  * @license MIT
  */
 
@@ -1366,19 +1366,18 @@ class Item {
     if (parent._redone !== null) {
       parent = parent._redone;
       // find next cloned items
-      while (left !== null) {
+      while (left !== null && left._redone === null) {
-        if (left._redone !== null && left._redone._parent === parent) {
-          left = left._redone;
-          break
-        }
         left = left._left;
       }
-      while (right !== null) {
+      if (left !== null) {
-        if (right._redone !== null && right._redone._parent === parent) {
+        left = left._redone;
-          right = right._redone;
+      }
-        }
+      while (right !== null && right._redone === null) {
         right = right._right;
       }
+      if (right !== null) {
+        right = right._redone;
+      }
     }
     struct._origin = left;
     struct._left = left;
@@ -3746,7 +3745,6 @@ class UndoManager {
     this._scope = scope;
     this._undoing = false;
     this._redoing = false;
-    this._lastTransactionWasUndo = false;
     const y = scope._y;
     this.y = y;
     y.on('afterTransaction', (y, transaction, remote) => {
@@ -3754,12 +3752,7 @@ class UndoManager {
         let reverseOperation = new ReverseOperation(y, transaction);
         if (!this._undoing) {
           let lastUndoOp = this._undoBuffer.length > 0 ? this._undoBuffer[this._undoBuffer.length - 1] : null;
-          if (
+          if (lastUndoOp !== null && reverseOperation.created - lastUndoOp.created <= options.captureTimeout) {
-            this._redoing === false &&
-            this._lastTransactionWasUndo === false &&
-            lastUndoOp !== null &&
-            reverseOperation.created - lastUndoOp.created <= options.captureTimeout
-          ) {
             lastUndoOp.created = reverseOperation.created;
             if (reverseOperation.toState !== null) {
               lastUndoOp.toState = reverseOperation.toState;
@@ -3769,14 +3762,12 @@ class UndoManager {
             }
             reverseOperation.deletedStructs.forEach(lastUndoOp.deletedStructs.add, lastUndoOp.deletedStructs);
           } else {
-            this._lastTransactionWasUndo = false;
             this._undoBuffer.push(reverseOperation);
           }
           if (!this._redoing) {
             this._redoBuffer = [];
           }
         } else {
-          this._lastTransactionWasUndo = true;
           this._redoBuffer.push(reverseOperation);
         }
       }