robot is right

Author: odeimaiz

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

@@ -41,7 +41,7 @@ qx.Class.define("osparc.data.model.Function", {
       lastChangeDate: functionData.lastChangeDate ? new Date(functionData.lastChangeDate) : this.getLastChangeDate(),
       thumbnail: functionData.thumbnail || this.getThumbnail(),
       workbenchData: functionData.workbench || this.getWorkbenchData(),
-      functionUIData: functionData.ui || this.functionUIData(),
+      functionUIData: functionData.ui || this.getFunctionUIData(),
     });
   },
 
@@ -148,5 +148,33 @@ qx.Class.define("osparc.data.model.Function", {
       }
       return canWrite;
     },
+  },
+
+  members: {
+    serialize: function(clean = true) {
+      let jsonObject = {};
+      const propertyKeys = this.self().getProperties();
+      propertyKeys.forEach(key => {
+        jsonObject[key] = this.get(key);
+      });
+      return jsonObject;
+    },
+
+    patchFunction: function(functionChanges) {
+      return osparc.store.Study.getInstance().patchStudy(this.getUuid(), functionChanges)
+        .then(() => {
+          Object.keys(functionChanges).forEach(fieldKey => {
+            const upKey = qx.lang.String.firstUp(fieldKey);
+            const setter = "set" + upKey;
+            this[setter](functionChanges[fieldKey]);
+          })
+          this.set({
+            lastChangeDate: new Date()
+          });
+          const functionData = this.serialize();
+          resolve(functionData);
+        })
+        .catch(err => reject(err));
+    },
   }
 });

services/static-webserver/client/source/class/osparc/info/FunctionLarge.js

@@ -168,7 +168,7 @@ qx.Class.define("osparc.info.FunctionLarge", {
 
     __openDescriptionEditor: function() {
       const title = this.tr("Edit Description");
-      const textEditor = new osparc.editor.MarkdownEditor(this.getStudy().getDescription());
+      const textEditor = new osparc.editor.MarkdownEditor(this.getFunction().getDescription());
       textEditor.setMaxHeight(570);
       const win = osparc.ui.window.Window.popUpInWindow(textEditor, title, 400, 300);
       textEditor.addListener("textChanged", e => {
@@ -182,11 +182,10 @@ qx.Class.define("osparc.info.FunctionLarge", {
     },
 
     __patchFunction: function(fieldKey, value) {
-      this.getStudy().patchStudy({[fieldKey]: value})
-        .then(studyData => {
-          studyData["resourceType"] = this.getStudy().getTemplateType() ? "template" : "study";
-          this.fireDataEvent("updateStudy", studyData);
-          qx.event.message.Bus.getInstance().dispatchByName("updateStudy", studyData);
+      this.getFunction().patchFunction({[fieldKey]: value})
+        .then(functionData => {
+          this.fireDataEvent("updateFunction", functionData);
+          qx.event.message.Bus.getInstance().dispatchByName("updateFunction", functionData);
         })
         .catch(err => {
           const msg = this.tr("An issue occurred while updating the information.");

services/static-webserver/client/source/class/osparc/store/Functions.js

@@ -84,7 +84,7 @@ qx.Class.define("osparc.store.Functions", {
     },
 
     registerFunction: function(templateData, name, description, defaultInputs, exposedInputs, exposedOutputs) {
-      const functionData = this.self().__createFunctionData(templateData, name, description, defaultInputs, exposedInputs, exposedOutputs);
+      const functionData = this.__createFunctionData(templateData, name, description, defaultInputs, exposedInputs, exposedOutputs);
       const params = {
         data: functionData,
       };
@@ -122,7 +122,7 @@ qx.Class.define("osparc.store.Functions", {
           "functionId": functionId
         }
       };
-      return osparc.data.Resources.fetch("functions", "getOne", params, options)
+      return osparc.data.Resources.fetch("functions", "getOne", params)
         .then(func => {
           func["resourceType"] = "function";
           return func;