From 24445c0c9c589922a91dec014e5fdd39f111b2b2 Mon Sep 17 00:00:00 2001
From: Odei Maiz <33152403+odeimaiz@users.noreply.github.com>
Date: Mon, 4 Nov 2024 14:55:37 +0100
Subject: [PATCH] =?UTF-8?q?=F0=9F=90=9B=20[Frontend]=20Fix=20Service=20bro?=
 =?UTF-8?q?wser=20(#6659)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../client/source/class/osparc/store/Services.js       |  5 +++--
 .../client/source/class/osparc/utils/Utils.js          | 10 ++++++----
 2 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/services/static-webserver/client/source/class/osparc/store/Services.js b/services/static-webserver/client/source/class/osparc/store/Services.js
index 569f5ce08e4..f6851b3aa43 100644
--- a/services/static-webserver/client/source/class/osparc/store/Services.js
+++ b/services/static-webserver/client/source/class/osparc/store/Services.js
@@ -59,7 +59,7 @@ qx.Class.define("osparc.store.Services", {
               let serviceLatest = servicesLatest[key];
               if (excludeFrontend && key.includes("simcore/services/frontend/")) {
                 // do not add frontend services
-                return;
+                continue;
               }
               if (excludeDeprecated && serviceLatest["retired"]) {
                 // first check if a previous version of this service isn't retired
@@ -68,13 +68,14 @@ qx.Class.define("osparc.store.Services", {
                 for (let j=0; j<versions.length; j++) {
                   const version = versions[j];
                   if (!this.servicesCached[key][version]["retired"]) {
+                    // one older non retired version found
                     serviceLatest = await this.getService(key, version);
                     break;
                   }
                 }
                 if (serviceLatest["retired"]) {
                   // do not add retired services
-                  return;
+                  continue;
                 }
               }
               servicesList.push(serviceLatest);
diff --git a/services/static-webserver/client/source/class/osparc/utils/Utils.js b/services/static-webserver/client/source/class/osparc/utils/Utils.js
index ead285bccab..629468c1312 100644
--- a/services/static-webserver/client/source/class/osparc/utils/Utils.js
+++ b/services/static-webserver/client/source/class/osparc/utils/Utils.js
@@ -720,12 +720,14 @@ qx.Class.define("osparc.utils.Utils", {
       if (!+bytes) {
         return "0 Bytes";
       }
+
       const k = 1000;
-      const sizes = ["Bytes", "KB", "MB", "GB", "TB"];
       const dm = decimals < 0 ? 0 : decimals;
-
-      const i = Math.floor(Math.log(bytes) / Math.log(k))
-      return `${isDecimalCollapsed ? parseFloat((bytes / Math.pow(k, i)).toFixed(dm)) : (bytes / Math.pow(k, i)).toFixed(dm)} ${sizes[i]}`
+      const i = Math.floor(Math.log(bytes) / Math.log(k));
+      const value = (bytes / Math.pow(k, i)).toFixed(dm);
+      const metrics = ["Bytes", "kB", "MB", "GB", "TB"];
+      const metric = i < metrics.length ? metrics[i] : "";
+      return `${isDecimalCollapsed ? parseFloat(value) : value} ${metric}`
     },
 
     bytesToGB: function(bytes) {