refactor

Author: odeimaiz

services/static-webserver/client/source/class/osparc/dashboard/ResourceBrowserBase.js

@@ -790,7 +790,7 @@ qx.Class.define("osparc.dashboard.ResourceBrowserBase", {
                 if (selectedPricingUnitId) {
                   const nodeId = nodesIdsListed[idx];
                   const pricingPlanId = nodePricingUnits.getPricingPlanId();
-                  promises.push(osparc.study.NodePricingUnits.patchPricingUnitSelection(studyId, nodeId, pricingPlanId, selectedPricingUnitId));
+                  promises.push(osparc.store.Study.updateSelectedPricingUnit(studyId, nodeId, pricingPlanId, selectedPricingUnitId));
                 }
               });
 

services/static-webserver/client/source/class/osparc/node/TierSelectionView.js

@@ -96,7 +96,7 @@ qx.Class.define("osparc.node.TierSelectionView", {
                   if (selection.length) {
                     tierBox.setEnabled(false);
                     const selectedUnitId = selection[0].getModel();
-                    osparc.study.NodePricingUnits.patchPricingUnitSelection(studyId, nodeId, pricingPlans["pricingPlanId"], selectedUnitId)
+                    osparc.store.Study.updateSelectedPricingUnit(studyId, nodeId, pricingPlans["pricingPlanId"], selectedUnitId)
                       .finally(() => {
                         tierBox.setEnabled(true);
                         showSelectedTier(selectedUnitId);

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

@@ -244,7 +244,21 @@ qx.Class.define("osparc.store.Study", {
           pricingUnitId: selectedUnitId
         }
       };
-      return osparc.data.Resources.fetch("studies", "putPricingUnit", params);
+      return osparc.data.Resources.fetch("studies", "putPricingUnit", params)
+        .then(() => {
+          // update the cache
+          if (!(studyId in this.__nodePricingUnit)) {
+            this.__nodePricingUnit[studyId] = {};
+          }
+          const selectedPricingUnit = osparc.store.Pricing.getInstance().getPricingUnit(planId, selectedUnitId);
+          if (selectedPricingUnit) {
+            this.__nodePricingUnit[studyId][nodeId] = selectedUnitId;
+          }
+        })
+        .catch(err => {
+          console.error("Failed to update selected pricing unit:", err);
+          throw err;
+        });
     },
   }
 });

services/static-webserver/client/source/class/osparc/study/NodePricingUnits.js

@@ -72,12 +72,6 @@ qx.Class.define("osparc.study.NodePricingUnits", {
     },
   },
 
-  statics: {
-    patchPricingUnitSelection: function(studyId, nodeId, planId, selectedUnitId) {
-      return osparc.store.Study.updateSelectedPricingUnit(studyId, nodeId, planId, selectedUnitId);
-    }
-  },
-
   members: {
     __nodeKey: null,
     __nodeVersion: null,
@@ -115,7 +109,7 @@ qx.Class.define("osparc.study.NodePricingUnits", {
                       if (this.isPatchNode()) {
                         pricingUnitTiers.setEnabled(false);
                         const pricingPlanId = this.getPricingPlanId();
-                        this.self().patchPricingUnitSelection(studyId, nodeId, pricingPlanId, selectedPricingUnitId)
+                        osparc.store.Study.updateSelectedPricingUnit(studyId, nodeId, pricingPlanId, selectedPricingUnitId)
                           .then(() => pricingUnitTiers.setSelectedUnitId(selectedPricingUnitId))
                           .catch(err => osparc.FlashMessenger.logError(err, this.tr("Cannot change Tier")))
                           .finally(() => pricingUnitTiers.setEnabled(true));