@limetech/lime-elements 37.55.1 → 37.55.2

package/CHANGELOG.md

@@ -1,3 +1,11 @@
+## [37.55.2](https://github.com/Lundalogik/lime-elements/compare/v37.55.1...v37.55.2) (2024-07-29)
+
+
+### Bug Fixes
+
+
+* **table:** make sorting work again ([2834ccd](https://github.com/Lundalogik/lime-elements/commit/2834ccd04df027cb440f2bb5f60df8dd535013fe))
+
 ## [37.55.1](https://github.com/Lundalogik/lime-elements/compare/v37.55.0...v37.55.1) (2024-07-29)
 
 

package/dist/cjs/limel-table.cjs.entry.js

@@ -24260,18 +24260,18 @@ const Table = class {
     this.formatRows();
   }
   updateData(newData = [], oldData = []) {
-    const newIds = new Set(newData.map((item) => { var _a; return (_a = item.id) !== null && _a !== void 0 ? _a : item; }));
-    if (oldData.every((item) => { var _a; return newIds.has((_a = item.id) !== null && _a !== void 0 ? _a : item); })) {
-      this.tabulator.updateOrAddData(newData);
-      return;
-    }
     this.pool.releaseAll();
+    const shouldReplace = this.shouldReplaceData(newData, oldData);
     setTimeout(() => {
       if (!this.tabulator) {
         return;
       }
-      this.tabulator.replaceData(this.data);
-      this.setSelection();
+      if (shouldReplace) {
+        this.tabulator.replaceData(newData);
+        this.setSelection();
+        return;
+      }
+      this.tabulator.updateOrAddData(newData);
     });
   }
   updateColumns(newColumns, oldColumns) {
@@ -24327,6 +24327,21 @@ const Table = class {
     }
     this.tabulator.setSort(newSorting);
   }
+  shouldReplaceData(newData, oldData) {
+    const newIds = newData.map((item) => { var _a; return (_a = item.id) !== null && _a !== void 0 ? _a : item; });
+    const oldIds = oldData.map((item) => { var _a; return (_a = item.id) !== null && _a !== void 0 ? _a : item; });
+    return (!this.areEqualIds(newIds, oldIds) ||
+      !this.isSameOrder(newIds, oldIds));
+  }
+  areEqualIds(newIds, oldIds) {
+    const newIdSet = new Set(newIds);
+    const oldIdSet = new Set(oldIds);
+    return (newIdSet.size === oldIdSet.size &&
+      newIds.every((id) => oldIdSet.has(id)));
+  }
+  isSameOrder(newIds, oldIds) {
+    return newIds.every((id, index) => id === oldIds[index]);
+  }
   areSameColumns(newColumns, oldColumns) {
     return (newColumns.length === oldColumns.length &&
       newColumns.every((column) => oldColumns.includes(column)));