wip: allow to mark an operation as not undoable

Co-authored-by: David Larlet <david@larlet.fr>

umap/static/umap/js/modules/data/layer.js

@@ -1098,7 +1098,11 @@ export class DataLayer extends ServerStored {
 
   setReferenceVersion({ response, sync }) {
     this._referenceVersion = response.headers.get('X-Datalayer-Version')
-    if (sync) this.sync.update('_referenceVersion', this._referenceVersion)
+    if (sync) {
+      this.sync.update('_referenceVersion', this._referenceVersion, null, {
+        undo: false,
+      })
+    }
   }
 
   async save() {

umap/static/umap/js/modules/sync/engine.js

@@ -169,7 +169,7 @@ export class SyncEngine {
     this._send(operation)
   }
 
-  update(subject, metadata, key, value, oldValue) {
+  update(subject, metadata, key, value, oldValue, { undo } = { undo: true }) {
     const operation = {
       verb: 'update',
       subject,
@@ -186,7 +186,7 @@ export class SyncEngine {
       this._batch.push(stage)
       return
     }
-    this._undoManager.add(stage)
+    if (undo) this._undoManager.add(stage)
     this._send(operation)
   }
 
@@ -211,7 +211,7 @@ export class SyncEngine {
   async save() {
     const needSave = new Map()
     if (!this._umap.id) {
-      // There is no operation for fist map save
+      // There is no operation for first map save
       needSave.set(this._umap, [])
     }
     for (const operation of this._operations.sorted()) {

umap/static/umap/js/modules/sync/undo.js

@@ -35,13 +35,10 @@ export class UndoManager {
   }
 
   add(stage) {
-    // FIXME make it more generic
-    if (!stage.operation || stage.operation.key !== '_referenceVersion') {
-      stage.operation.dirty = true
-      this._redoStack = []
-      this._undoStack.push(stage)
-      this.toggleState()
-    }
+    stage.operation.dirty = true
+    this._redoStack = []
+    this._undoStack.push(stage)
+    this.toggleState()
   }
 
   copyOperation(stage, redo) {