From 06ba3a8fe9f5971e06b0b21ad20104a93d69607f Mon Sep 17 00:00:00 2001
From: Max Richter <jim-x@web.de>
Date: Thu, 18 Apr 2024 13:25:05 +0200
Subject: [PATCH] fix: issue where updates didnt work

---
 app/src/lib/graph-interface/graph-manager.ts | 17 ++++++++++-------
 app/src/lib/graph-interface/helpers/index.ts |  2 ++
 2 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/app/src/lib/graph-interface/graph-manager.ts b/app/src/lib/graph-interface/graph-manager.ts
index e098f92..a2730df 100644
--- a/app/src/lib/graph-interface/graph-manager.ts
+++ b/app/src/lib/graph-interface/graph-manager.ts
@@ -53,7 +53,7 @@ export class GraphManager extends EventEmitter<{ "save": Graph, "result": any }>
   }
 
   serialize(): Graph {
-    logger.log("serializing graph")
+    logger.group("serializing graph")
     const nodes = Array.from(this._nodes.values()).map(node => ({
       id: node.id,
       position: [...node.position],
@@ -62,7 +62,11 @@ export class GraphManager extends EventEmitter<{ "save": Graph, "result": any }>
     })) as Node[];
     const edges = this._edges.map(edge => [edge[0].id, edge[1], edge[2].id, edge[3]]) as Graph["edges"];
     const settings = get(this.settings);
-    return { id: this.graph.id, settings, nodes, edges };
+    const serialized = { id: this.graph.id, settings, nodes, edges };
+    console.log(serialized);
+    logger.groupEnd();
+
+    return serialized;
   }
 
   execute() { }
@@ -366,13 +370,11 @@ export class GraphManager extends EventEmitter<{ "save": Graph, "result": any }>
 
     const edgeToBeReplaced = this._edges.find(e => e[2].id === to.id && e[3] === toSocket);
     if (edgeToBeReplaced) {
-      this.removeEdge(edgeToBeReplaced, { applyDeletion: applyUpdate });
+      this.removeEdge(edgeToBeReplaced, { applyDeletion: false });
     }
 
     if (applyUpdate) {
-      this.edges.update((edges) => {
-        return [...edges, [from, fromSocket, to, toSocket]];
-      });
+      this._edges.push([from, fromSocket, to, toSocket]);
     } else {
       this._edges.push([from, fromSocket, to, toSocket]);
     }
@@ -385,10 +387,11 @@ export class GraphManager extends EventEmitter<{ "save": Graph, "result": any }>
     to.tmp.parents = to.tmp.parents || [];
     to.tmp.parents.push(from);
 
-    this.execute();
     if (applyUpdate) {
+      this.edges.set(this._edges);
       this.save();
     }
+    this.execute();
   }
 
   undo() {
diff --git a/app/src/lib/graph-interface/helpers/index.ts b/app/src/lib/graph-interface/helpers/index.ts
index 9d60856..57f2d19 100644
--- a/app/src/lib/graph-interface/helpers/index.ts
+++ b/app/src/lib/graph-interface/helpers/index.ts
@@ -80,6 +80,8 @@ export const createLogger = (() => {
     let muted = false;
     return {
       log: (...args: any[]) => !muted && console.log(`[%c${scope.padEnd(maxLength, " ")}]:`, "color: #888", ...args),
+      group: (...args: any[]) => !muted && console.groupCollapsed(`[%c${scope.padEnd(maxLength, " ")}]:`, "color: #888", ...args),
+      groupEnd: () => !muted && console.groupEnd(),
       info: (...args: any[]) => !muted && console.info(`[%c${scope.padEnd(maxLength, " ")}]:`, "color: #888", ...args),
       warn: (...args: any[]) => !muted && console.warn(`[%c${scope.padEnd(maxLength, " ")}]:`, "color: #888", ...args),
       error: (...args: any[]) => console.error(`[%c${scope.padEnd(maxLength, " ")}]:`, "color: #f88", ...args),