refactor

Author: odeimaiz

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

@@ -528,7 +528,8 @@ qx.Class.define("osparc.data.model.Node", {
           this.setLabel(nodeData.label);
         }
         this.__populateInputOutputData(nodeData);
-        this.populateStates(nodeData);
+        this.populateProgress(nodeData);
+        this.populateState(nodeData);
         if (nodeData.bootOptions) {
           this.setBootOptions(nodeData.bootOptions);
         }
@@ -570,7 +571,7 @@ qx.Class.define("osparc.data.model.Node", {
       this.setInputsRequired(nodeData.inputsRequired || []);
     },
 
-    populateStates: function(nodeData) {
+    populateProgress: function(nodeData) {
       if ("progress" in nodeData) {
         const progress = Number.parseInt(nodeData["progress"]);
         const oldProgress = this.getStatus().getProgress();
@@ -581,6 +582,9 @@ qx.Class.define("osparc.data.model.Node", {
           this.getStatus().setProgress(progress);
         }
       }
+    },
+
+    populateState: function(nodeData) {
       if ("state" in nodeData) {
         this.getStatus().setState(nodeData.state);
       }

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

@@ -576,11 +576,8 @@ qx.Class.define("osparc.data.model.Study", {
       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);
+          node.populateProgress(nodeData);
+          node.populateState(nodeData);
         }
         if ("errors" in nodeUpdatedData) {
           const errors = nodeUpdatedData["errors"];