Add client-side rollback during snapshot import (#492)

b310-digital · Oct 21, 2024 · 9058c58 · 9058c58
1 parent c127abb
commit 9058c58
Showing 1 changed file with 11 additions and 3 deletions.
diff --git a/teammapper-frontend/mmp/src/map/handlers/history.ts b/teammapper-frontend/mmp/src/map/handlers/history.ts
@@ -64,8 +64,16 @@ export default class History {
 
             this.map.zoom.center('position', 0)
 
-            this.save()
-            if (notifyWithEvent) this.map.events.call(Event.create, this.map.dom, { previousMap: previousData })
+            // If the amount of nodes is == 0, automatically rollback to the last clean snapshot and display a toast
+            if (this.map.nodes.getNodes().length === 0) {
+                if (previousData.length > 0) {
+                    this.redraw(previousData)
+                }
+                Log.error('There was an error importing the map; changes have been rolled back.')
+            } else {
+                this.save()
+                if (notifyWithEvent) this.map.events.call(Event.create, this.map.dom, { previousMap: previousData })
+            }
         } else {
             Log.error('The snapshot is not correct')
         }
@@ -121,7 +129,7 @@ export default class History {
      */
     private redraw(snapshot: MapSnapshot) {
         this.map.nodes.clear()
-
+        
         snapshot.forEach((property: ExportNodeProperties) => {
             // in case the data model changes this makes sure all properties are at least present using defaults
             const mergedProperty = { ...DefaultNodeValues, ...property } as ExportNodeProperties