From 9e87974eb16e23f13cd418bf8b05c45d3315c4a1 Mon Sep 17 00:00:00 2001
From: Christoph Kluge <christoph.kluge@fau.de>
Date: Wed, 4 Jun 2025 17:02:02 +0200
Subject: [PATCH] Fix compareTable sorting

---
 web/frontend/src/generic/JobCompare.svelte | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/web/frontend/src/generic/JobCompare.svelte b/web/frontend/src/generic/JobCompare.svelte
index 484972d..2920eea 100644
--- a/web/frontend/src/generic/JobCompare.svelte
+++ b/web/frontend/src/generic/JobCompare.svelte
@@ -133,6 +133,7 @@
     }
     compareTableSorting = { ...compareTableSorting };
 
+    let pendingCompareData;
     if (key == 'resources') {
       let longField = "";
       switch (field) {
@@ -148,12 +149,13 @@
         default:
           console.log("Unknown Res Field", field)
       } 
-      compareTableData = compareTableData.sort((j1, j2) => {
+
+      pendingCompareData = compareTableData.sort((j1, j2) => {
         if (j1[longField] == null || j2[longField] == null) return -1;
         return s.dir != "up" ? j1[longField] - j2[longField] : j2[longField] - j1[longField];
       });
     } else if (key == 'meta') {
-      compareTableData = compareTableData.sort((j1, j2) => {
+      pendingCompareData = compareTableData.sort((j1, j2) => {
         if (j1[field] == null || j2[field] == null) return -1;
         if (field == 'cluster') {
           let c1 = `${j1.cluster} (${j1.subCluster})`
@@ -164,13 +166,15 @@
         }
       });
     } else {
-      compareTableData = compareTableData.sort((j1, j2) => {
+      pendingCompareData = compareTableData.sort((j1, j2) => {
         let s1 = j1.stats.find((m) => m.name == key)?.data;
         let s2 = j2.stats.find((m) => m.name == key)?.data;
         if (s1 == null || s2 == null) return -1;
         return s.dir != "up" ? s1[field] - s2[field] : s2[field] - s1[field];
       });
     }
+
+    compareTableData = [...pendingCompareData]
   }
 
   function jobs2uplot(jobs, metrics) {