@controleonline/ui-default 1.0.276 → 1.0.277

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@controleonline/ui-default",
-  "version": "1.0.276",
+  "version": "1.0.277",
   "author": "ControleOnline",
   "description": "ControleOnline Default Quasar UI Component",
   "license": "MIT",

package/src/react/components/table/DefaultTable.js

@@ -404,7 +404,7 @@ const DefaultTable = ({
         showTotalItemsInFooter,
         storedViewMode,
         tableFiltersVisible,
-        sortedDataLength: sortedData.length,
+        sortedData: stableSerialize(sortedData),
         toolbarActionsLength: Array.isArray(toolbarActions) ? toolbarActions.length : 0,
       }),
     [
@@ -442,7 +442,7 @@ const DefaultTable = ({
       showTotalItemsInFooter,
       storedViewMode,
       tableFiltersVisible,
-      sortedData.length,
+      sortedData,
       toolbarActions,
     ],
   );

package/src/tests/react/components/table/useDefaultTableStoreSync.test.js

@@ -25,13 +25,13 @@ const {
   useDefaultTableStoreSync,
 } = require('../../../../react/components/table/useDefaultTableStoreSync');
 
-function Harness({data, store, storeFilters, columnsForTable, tablePreferenceScope}) {
+function Harness({data, store, storeFilters, columnsForTable, tablePreferenceScope, configsSignature = 'sig', configs = {}}) {
   useDefaultTableStoreSync({
     columns: [],
     columnsForTable: columnsForTable || [],
     data,
-    defaultTableConfigs: {},
-    defaultTableConfigsSignature: 'sig',
+    defaultTableConfigs: configs,
+    defaultTableConfigsSignature: configsSignature,
     store,
     storeColumnsLength: 0,
     storeFilters: storeFilters || {},
@@ -158,4 +158,40 @@ describe('useDefaultTableStoreSync', () => {
     expect(areTableFiltersEqual(a, {alterDate: {shortcut: 'all'}})).toBe(false);
     expect(areTableFiltersEqual({}, {})).toBe(true);
   });
+
+  it('publishes changed same-count table configs but ignores equivalent snapshots', () => {
+    const setConfigs = jest.fn();
+    const store = {actions: {setConfigs, setItems}, getters: {}};
+    const first = [{id: 'device:396', configs: [{id: 487}]}];
+    const second = [{id: 'device:403', configs: [{id: 510}]}];
+    let tree;
+    renderer.act(() => {
+      tree = renderer.create(React.createElement(Harness, {
+        data: first,
+        store,
+        configs: {sortedData: first},
+        configsSignature: JSON.stringify(first),
+      }));
+    });
+    expect(setConfigs).toHaveBeenCalledTimes(1);
+    renderer.act(() => {
+      tree.update(React.createElement(Harness, {
+        data: second,
+        store,
+        configs: {sortedData: second},
+        configsSignature: JSON.stringify(second),
+      }));
+    });
+    expect(setConfigs).toHaveBeenCalledTimes(2);
+    renderer.act(() => {
+      tree.update(React.createElement(Harness, {
+        data: [{id: 'device:403', configs: [{id: 510}]}],
+        store: {actions: {setConfigs, setItems}, getters: {}},
+        configs: {sortedData: second},
+        configsSignature: JSON.stringify(second),
+      }));
+    });
+    expect(setConfigs).toHaveBeenCalledTimes(2);
+    tree.unmount();
+  });
 });