serialize nodeUI

Author: odeimaiz

services/static-webserver/client/source/class/osparc/data/model/Node.js

@@ -1307,6 +1307,19 @@ qx.Class.define("osparc.data.model.Node", {
       }
 
       return filteredNodeEntry;
-    }
+    },
+
+    serializeUI: function() {
+      const uiInfo = {
+        "position": this.getPosition(),
+      };
+      const marker = this.getMarker();
+      if (marker) {
+        uiInfo["marker"] = {
+          "color": marker.getColor()
+        };
+      }
+      return uiInfo;
+    },
   }
 });

services/static-webserver/client/source/class/osparc/data/model/Workbench.js

@@ -743,9 +743,8 @@ qx.Class.define("osparc.data.model.Workbench", {
         // workbench is not initialized
         return this.__workbenchInitData;
       }
-      let workbench = {};
-      const allModels = this.getNodes();
-      const nodes = Object.values(allModels);
+      const workbench = {};
+      const nodes = Object.values(this.getNodes());
       for (const node of nodes) {
         const data = node.serialize(clean);
         if (data) {
@@ -760,17 +759,12 @@ qx.Class.define("osparc.data.model.Workbench", {
         // workbenchUI is not initialized
         return this.__workbenchUIInitData;
       }
-      let workbenchUI = {};
-      const nodes = this.getNodes();
-      for (const nodeId in nodes) {
-        const node = nodes[nodeId];
-        workbenchUI[nodeId] = {};
-        workbenchUI[nodeId]["position"] = node.getPosition();
-        const marker = node.getMarker();
-        if (marker) {
-          workbenchUI[nodeId]["marker"] = {
-            color: marker.getColor()
-          };
+      const workbenchUI = {};
+      const nodes = Object.values(this.getNodes());
+      for (const node of nodes) {
+        const data = node.serializeUI(clean);
+        if (data) {
+          workbench[node.getNodeId()] = data;
         }
       }
       return workbenchUI;