Skip to content

Commit f35d515

Browse files
authored
🐛 [Frontend] Fix: Do not list deprecated service versions (#7504)
1 parent 9bb210e commit f35d515

File tree

4 files changed

+16
-17
lines changed

4 files changed

+16
-17
lines changed

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

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -666,20 +666,13 @@ qx.Class.define("osparc.dashboard.CardBase", {
666666

667667
// Block card
668668
const cantReadServices = osparc.study.Utils.getCantExecuteServices(services);
669-
let inaccessibleServices = [];
670-
if (this.isResourceType("study") || this.isResourceType("template")) {
671-
inaccessibleServices = osparc.store.Services.getInaccessibleServices(this.getResourceData()["workbench"]);
672-
}
673-
if (cantReadServices.length || inaccessibleServices.length) {
669+
if (cantReadServices.length) {
674670
this.setBlocked("UNKNOWN_SERVICES");
675671
const image = "@FontAwesome5Solid/ban/";
676672
let toolTipText = this.tr("Inaccessible service(s):");
677673
cantReadServices.forEach(unSrv => {
678674
toolTipText += "<br>" + unSrv.key + ":" + osparc.service.Utils.extractVersionDisplay(unSrv.release);
679675
});
680-
inaccessibleServices.forEach(unSrv => {
681-
toolTipText += "<br>" + unSrv.key + ":" + unSrv.version;
682-
});
683676
this.__showBlockedCard(image, toolTipText);
684677
}
685678

services/static-webserver/client/source/class/osparc/desktop/wallets/WalletListItem.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ qx.Class.define("osparc.desktop.wallets.WalletListItem", {
2424
const creditsCol = 7;
2525
const layout = this._getLayout();
2626
layout.setSpacingX(10);
27+
layout.setColumnAlign(0, "left", "middle");
2728
layout.setColumnWidth(creditsCol, 110);
2829
layout.setColumnAlign(creditsCol, "right", "middle");
2930

services/static-webserver/client/source/class/osparc/share/NewCollaboratorsManager.js

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -35,12 +35,7 @@ qx.Class.define("osparc.share.NewCollaboratorsManager", {
3535

3636
this.__shareWithEmailEnabled = false;
3737
if (this.__resourceData["resourceType"] === "study") {
38-
osparc.utils.DisabledPlugins.isShareWithEmailEnabled()
39-
.then(isEnabled => {
40-
if (isEnabled) {
41-
this.__shareWithEmailEnabled = true;
42-
}
43-
});
38+
this.__shareWithEmailEnabled = osparc.utils.DisabledPlugins.isShareWithEmailEnabled();
4439
}
4540

4641
this.center();

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

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -204,7 +204,10 @@ qx.Class.define("osparc.store.Services", {
204204
continue;
205205
}
206206
if (excludeDeprecated) {
207-
if (osparc.service.Utils.isDeprecated(serviceLatest)) {
207+
if (
208+
osparc.service.Utils.isRetired(serviceLatest) ||
209+
osparc.service.Utils.isDeprecated(serviceLatest)
210+
) {
208211
// first check if a previous version of this service isn't deprecated
209212
// getService to get its history
210213
await this.getService(serviceLatest["key"], serviceLatest["version"]);
@@ -213,15 +216,21 @@ qx.Class.define("osparc.store.Services", {
213216
const historyEntry = serviceMetadata["history"][j];
214217
if (!historyEntry["retired"]) {
215218
// one older non retired version found
216-
const olderNonRetired = await this.getService(key, historyEntry["version"]);
219+
let olderNonRetired = await this.getService(key, historyEntry["version"]);
220+
if (!olderNonRetired) {
221+
olderNonRetired = await this.getService(key, historyEntry["version"]);
222+
}
217223
serviceLatest = osparc.utils.Utils.deepCloneObject(olderNonRetired);
218224
// make service metadata latest model like
219225
serviceLatest["release"] = osparc.service.Utils.extractVersionFromHistory(olderNonRetired);
220226
break;
221227
}
222228
}
223229
}
224-
if (osparc.service.Utils.isDeprecated(serviceLatest)) {
230+
if (
231+
osparc.service.Utils.isRetired(serviceLatest) ||
232+
osparc.service.Utils.isDeprecated(serviceLatest)
233+
) {
225234
// do not add retired services
226235
continue;
227236
}
@@ -361,6 +370,7 @@ qx.Class.define("osparc.store.Services", {
361370

362371
__isInCache: function(key, version) {
363372
return (
373+
this.__servicesCached &&
364374
key in this.__servicesCached &&
365375
version in this.__servicesCached[key]
366376
);

0 commit comments

Comments
 (0)