hypertools last

odeimaiz · odeimaiz · commit 0b2aa6d4a575 · 2025-05-12T17:09:45.000+02:00
diff --git a/services/static-webserver/client/source/class/osparc/service/Utils.js b/services/static-webserver/client/source/class/osparc/service/Utils.js
@@ -98,11 +98,14 @@ qx.Class.define("osparc.service.Utils", {
       }
       servicesArray.sort((a, b) => {
         if (basedOn.sort === "hits") {
-          if (a[basedOn.sort] !== b[basedOn.sort]) {
+          const aHits = a["hits"] === undefined ? -1 : a["hits"]; // Treat undefined hits as -1
+          const bHits = b["hits"] === undefined ? -1 : b["hits"]; // Treat undefined hits as -1
+
+          if (aHits !== bHits) {
             if (basedOn.order === "down") {
-              return b[basedOn.sort] - a[basedOn.sort];
+              return bHits - aHits;
             }
-            return a[basedOn.sort] - b[basedOn.sort];
+            return aHits - bHits;
           }
           return a["name"].localeCompare(b["name"]);
         } else if (basedOn.sort === "name") {

Original file line number	Diff line number	Diff line change
`@@ -98,11 +98,14 @@ qx.Class.define("osparc.service.Utils", {`
`98`	`98`	`}`
`99`	`99`	`servicesArray.sort((a, b) => {`
`100`	`100`	`if (basedOn.sort === "hits") {`
`101`		`- if (a[basedOn.sort] !== b[basedOn.sort]) {`
	`101`	`+ const aHits = a["hits"] === undefined ? -1 : a["hits"]; // Treat undefined hits as -1`
	`102`	`+ const bHits = b["hits"] === undefined ? -1 : b["hits"]; // Treat undefined hits as -1`
	`103`	`+`
	`104`	`+ if (aHits !== bHits) {`
`102`	`105`	`if (basedOn.order === "down") {`
`103`		`- return b[basedOn.sort] - a[basedOn.sort];`
	`106`	`+ return bHits - aHits;`
`104`	`107`	`}`
`105`		`- return a[basedOn.sort] - b[basedOn.sort];`
	`108`	`+ return aHits - bHits;`
`106`	`109`	`}`
`107`	`110`	`return a["name"].localeCompare(b["name"]);`
`108`	`111`	`} else if (basedOn.sort === "name") {`