update isUpdatable

Author: odeimaiz

services/static-webserver/client/source/class/osparc/metadata/ServicesInStudyUpdate.js

@@ -207,8 +207,8 @@ qx.Class.define("osparc.metadata.ServicesInStudyUpdate", {
           column: this.self().GRID_POS.COMPATIBLE_VERSION
         });
 
-        const isUpdatable = osparc.service.Utils.isUpdatable(node);
         if (latestCompatible && canIWrite) {
+          const isUpdatable = osparc.service.Utils.isUpdatable(metadata);
           const updateButton = new osparc.ui.form.FetchButton(null, "@MaterialIcons/update/14");
           updateButton.set({
             enabled: isUpdatable

services/static-webserver/client/source/class/osparc/service/Utils.js

@@ -162,8 +162,14 @@ qx.Class.define("osparc.service.Utils", {
     RETIRED_AUTOUPDATABLE_INSTRUCTIONS: qx.locale.Manager.tr("Please Update the Service"),
 
     isUpdatable: function(metadata) {
-      const latestCompatible = this.getLatestCompatible(metadata["key"], metadata["version"]);
-      return latestCompatible && (metadata["key"] !== latestCompatible["key"] || metadata["version"] !== latestCompatible["version"]);
+      if (metadata["history"]) {
+        const found = metadata["history"].find(history => history["version"] === metadata["version"]);
+        if (found && found["compatibility"] && found["compatibility"]["canUpdateTo"]) {
+          const latestCompatible = found["compatibility"]["canUpdateTo"];
+          return latestCompatible && (metadata["key"] !== latestCompatible["key"] || metadata["version"] !== latestCompatible["version"]);
+        }
+      }
+      return false;
     },
 
     isDeprecated: function(metadata) {

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

@@ -317,29 +317,12 @@ qx.Class.define("osparc.store.Services", {
         this.__servicesCached[key] = {};
       }
       this.__servicesCached[key][version] = service;
-
-      if ("history" in service) {
-        this.__servicesCached[key][version]["cached"] = true;
-        service["history"].forEach(historyEntry => {
-          const hVersion = historyEntry.version;
-          if (!(hVersion in this.__servicesCached[key])) {
-            this.__servicesCached[key][hVersion] = {};
-            this.__servicesCached[key][hVersion]["cached"] = false;
-          }
-          // merge history data into current metadata
-          this.__servicesCached[key][hVersion] = {
-            ...this.__servicesCached[key][hVersion],
-            ...historyEntry
-          };
-        });
-      }
     },
 
     __isInCache: function(key, version) {
       return (
         key in this.__servicesCached &&
-        version in this.__servicesCached[key] &&
-        this.__servicesCached[key][version]["cached"]
+        version in this.__servicesCached[key]
       );
     },