minor fixes

Author: odeimaiz

services/static-webserver/client/source/class/osparc/conversation/Conversation.js

@@ -217,7 +217,7 @@ qx.Class.define("osparc.conversation.Conversation", {
           const messages = resp["data"];
           messages.forEach(message => this.addMessage(message));
           this.__nextRequestParams = resp["_links"]["next"];
-          if (this.__nextRequestParams === null) {
+          if (this.__nextRequestParams === null && this.__loadMoreMessages) {
             this.__loadMoreMessages.exclude();
           }
         })

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

@@ -718,7 +718,7 @@ qx.Class.define("osparc.data.model.Node", {
         });
       } else {
         this.fireDataEvent("projectDocumentChanged", {
-          "op": "delete",
+          "op": "remove",
           "path": `/ui/workbench/${this.getNodeId()}/marker`,
           "osparc-resource": "ui",
         });

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

@@ -573,19 +573,21 @@ qx.Class.define("osparc.data.model.Study", {
       // Do not listen to output related backend updates if the node is a frontend node.
       // The frontend controls its output values, progress and states.
       // If a File Picker is uploading a file, the backend could override the current state with some older state.
-      if (node && nodeData && !osparc.data.model.Node.isFrontend(node.getMetaData())) {
-        node.setOutputData(nodeData.outputs);
-        if ("progress" in nodeData) {
-          const progress = Number.parseInt(nodeData["progress"]);
-          node.getStatus().setProgress(progress);
+      if (node) {
+        if (nodeData && !osparc.data.model.Node.isFrontend(node.getMetaData())) {
+          node.setOutputData(nodeData.outputs);
+          if ("progress" in nodeData) {
+            const progress = Number.parseInt(nodeData["progress"]);
+            node.getStatus().setProgress(progress);
+          }
+          node.populateStates(nodeData);
+        }
+        if ("errors" in nodeUpdatedData) {
+          const errors = nodeUpdatedData["errors"];
+          node.setErrors(errors);
+        } else {
+          node.setErrors([]);
         }
-        node.populateStates(nodeData);
-      }
-      if (node && "errors" in nodeUpdatedData) {
-        const errors = nodeUpdatedData["errors"];
-        node.setErrors(errors);
-      } else {
-        node.setErrors([]);
       }
     },
 

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

@@ -143,7 +143,7 @@ qx.Class.define("osparc.data.model.StudyUI", {
       if (annotationId in this.getAnnotations()) {
         const annotation = this.getAnnotations()[annotationId]
         this.fireDataEvent("projectDocumentChanged", {
-          "op": "delete",
+          "op": "remove",
           "path": `/ui/annotations/${annotation.getId()}`,
           "osparc-resource": "study-ui",
         });

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

@@ -320,7 +320,7 @@ qx.Class.define("osparc.data.model.Workbench", {
         this.__addNode(node);
 
         node.populateNodeData();
-        this.giveUniqueNameToNode(node, node.getLabel());
+        this.__giveUniqueNameToNode(node, node.getLabel());
         node.checkState();
 
         return node;
@@ -653,15 +653,15 @@ qx.Class.define("osparc.data.model.Workbench", {
       return false;
     },
 
-    giveUniqueNameToNode: function(node, label, suffix = 2) {
+    __giveUniqueNameToNode: function(node, label, suffix = 2) {
       const newLabel = label + "_" + suffix;
       const allModels = this.getNodes();
       const nodes = Object.values(allModels);
       for (const node2 of nodes) {
         if (node2.getNodeId() !== node.getNodeId() &&
             node2.getLabel().localeCompare(node.getLabel()) === 0) {
           node.setLabel(newLabel);
-          this.giveUniqueNameToNode(node, label, suffix+1);
+          this.__giveUniqueNameToNode(node, label, suffix+1);
         }
       }
     },
@@ -720,7 +720,7 @@ qx.Class.define("osparc.data.model.Workbench", {
 
           nodeIds.forEach(nodeId => {
             const node = this.getNode(nodeId);
-            this.giveUniqueNameToNode(node, node.getLabel());
+            this.__giveUniqueNameToNode(node, node.getLabel());
           });
         });
     },

services/static-webserver/client/source/class/osparc/workbench/Annotation.js

@@ -276,14 +276,8 @@ qx.Class.define("osparc.workbench.Annotation", {
       const serializeData = {
         type: this.getType(),
         attributes: this.getAttributes(),
+        color: this.getColor(), // TYPES.NOTE and TYPES.CONVERSATION do not need a color but backend expects it
       };
-      if ([
-        this.self().TYPES.RECT,
-        this.self().TYPES.TEXT,
-        this.self().TYPES.NOTE,
-      ].includes(this.getType())) {
-        serializeData.color = this.getColor();
-      }
       return serializeData;
     }
   }

services/static-webserver/client/source/class/osparc/workbench/WorkbenchUI.js

@@ -346,7 +346,7 @@ qx.Class.define("osparc.workbench.WorkbenchUI", {
 
     __addNode: async function(service, pos) {
       // render temporary node
-      let tempNodeUI = this.__createTemporaryNodeUI(pos);
+      let dashedNodeUI = this.__createTemporaryNodeUI(pos);
 
       let nodeUI = null;
       try {
@@ -359,7 +359,7 @@ qx.Class.define("osparc.workbench.WorkbenchUI", {
         console.error(err);
       } finally {
         // remove temporary node
-        this.__removeTemporaryNodeUI(tempNodeUI);
+        this.__removeTemporaryNodeUI(dashedNodeUI);
       }
       return nodeUI;
     },