@tanstack/table-core 8.2.1 → 8.2.4

package/build/types/index.d.ts

@@ -333,6 +333,7 @@ declare type SortingColumn<TData extends RowData> = {
     getAutoSortingFn: () => SortingFn<TData>;
     getAutoSortDir: () => SortDirection;
     getSortingFn: () => SortingFn<TData>;
+    getFirstSortDir: () => SortDirection;
     getNextSortingOrder: () => SortDirection | false;
     getCanSort: () => boolean;
     getCanMultiSort: () => boolean;

package/build/umd/index.development.js

@@ -1983,6 +1983,10 @@
 
             if (value) {
               preGroupedFlatRows.forEach(row => {
+                if (!row.getCanSelect()) {
+                  return;
+                }
+
                 rowSelection[row.id] = true;
               });
             } else {
@@ -2246,8 +2250,8 @@
   const mutateRowIsSelected = (selectedRowIds, id, value, table) => {
     var _row$subRows;
 
-    const row = table.getRow(id);
-    row.getIsGrouped(); // if ( // TODO: enforce grouping row selection rules
+    const row = table.getRow(id); // const isGrouped = row.getIsGrouped()
+    // if ( // TODO: enforce grouping row selection rules
     //   !isGrouped ||
     //   (isGrouped && table.options.enableGroupingRowSelection)
     // ) {
@@ -2257,7 +2261,9 @@
         Object.keys(selectedRowIds).forEach(key => delete selectedRowIds[key]);
       }
 
-      selectedRowIds[id] = true;
+      if (row.getCanSelect()) {
+        selectedRowIds[id] = true;
+      }
     } else {
       delete selectedRowIds[id];
     } // }
@@ -2353,7 +2359,12 @@
   };
 
   const datetime = (rowA, rowB, columnId) => {
-    return compareBasic(rowA.getValue(columnId).getTime(), rowB.getValue(columnId).getTime());
+    const a = rowA.getValue(columnId);
+    const b = rowB.getValue(columnId); // Can handle nullish values
+    // Use > and < because == (and ===) doesn't work with
+    // Date objects (would require calling getTime()).
+
+    return a > b ? 1 : a < b ? -1 : 0;
   };
 
   const basic = (rowA, rowB, columnId) => {
@@ -2514,18 +2525,17 @@
           // }
           // this needs to be outside of table.setSorting to be in sync with rerender
           const nextSortingOrder = column.getNextSortingOrder();
+          const hasManualValue = typeof desc !== 'undefined' && desc !== null;
           table.setSorting(old => {
-            var _table$options$enable, _table$options$enable2;
-
             // Find any existing sorting for this column
             const existingSorting = old == null ? void 0 : old.find(d => d.id === column.id);
             const existingIndex = old == null ? void 0 : old.findIndex(d => d.id === column.id);
-            const hasDescDefined = typeof desc !== 'undefined' && desc !== null;
             let newSorting = []; // What should we do with this sort action?
 
             let sortAction;
+            let nextDesc = hasManualValue ? desc : nextSortingOrder === 'desc'; // Multi-mode
 
-            if (column.getCanMultiSort() && multi) {
+            if (old != null && old.length && column.getCanMultiSort() && multi) {
               if (existingSorting) {
                 sortAction = 'toggle';
               } else {
@@ -2543,63 +2553,71 @@
             } // Handle toggle states that will remove the sorting
 
 
-            if (sortAction === 'toggle' && ( // Must be toggling
-            (_table$options$enable = table.options.enableSortingRemoval) != null ? _table$options$enable : true) && // If enableSortRemove, enable in general
-            !hasDescDefined && ( // Must not be setting desc
-            multi ? (_table$options$enable2 = table.options.enableMultiRemove) != null ? _table$options$enable2 : true : true) && // If multi, don't allow if enableMultiRemove
-            !nextSortingOrder // Finally, detect if it should indeed be removed
-            ) {
-              sortAction = 'remove';
+            if (sortAction === 'toggle') {
+              // If we are "actually" toggling (not a manual set value), should we remove the sorting?
+              if (!hasManualValue) {
+                // Is our intention to remove?
+                if (!nextSortingOrder) {
+                  sortAction = 'remove';
+                }
+              }
             }
 
-            if (sortAction === 'replace') {
-              newSorting = [{
-                id: column.id,
-                desc: hasDescDefined ? desc : nextSortingOrder === 'desc'
-              }];
-            } else if (sortAction === 'add' && old != null && old.length) {
+            if (sortAction === 'add') {
               var _table$options$maxMul;
 
               newSorting = [...old, {
                 id: column.id,
-                desc: hasDescDefined ? desc : nextSortingOrder === 'desc'
+                desc: nextDesc
               }]; // Take latest n columns
 
               newSorting.splice(0, newSorting.length - ((_table$options$maxMul = table.options.maxMultiSortColCount) != null ? _table$options$maxMul : Number.MAX_SAFE_INTEGER));
-            } else if (sortAction === 'toggle' && old != null && old.length) {
+            } else if (sortAction === 'toggle') {
               // This flips (or sets) the
               newSorting = old.map(d => {
                 if (d.id === column.id) {
                   return { ...d,
-                    desc: hasDescDefined ? desc : nextSortingOrder === 'desc'
+                    desc: nextDesc
                   };
                 }
 
                 return d;
               });
-            } else if (sortAction === 'remove' && old != null && old.length) {
+            } else if (sortAction === 'remove') {
               newSorting = old.filter(d => d.id !== column.id);
+            } else {
+              newSorting = [{
+                id: column.id,
+                desc: nextDesc
+              }];
             }
 
             return newSorting;
           });
         },
-        getNextSortingOrder: () => {
+        getFirstSortDir: () => {
           var _ref, _column$columnDef$sor;
 
           const sortDescFirst = (_ref = (_column$columnDef$sor = column.columnDef.sortDescFirst) != null ? _column$columnDef$sor : table.options.sortDescFirst) != null ? _ref : column.getAutoSortDir() === 'desc';
-          const firstSortDirection = sortDescFirst ? 'desc' : 'asc';
+          return sortDescFirst ? 'desc' : 'asc';
+        },
+        getNextSortingOrder: multi => {
+          var _table$options$enable, _table$options$enable2;
+
+          const firstSortDirection = column.getFirstSortDir();
           const isSorted = column.getIsSorted();
 
           if (!isSorted) {
             return firstSortDirection;
           }
 
-          if (isSorted === firstSortDirection) {
-            return isSorted === 'desc' ? 'asc' : 'desc';
-          } else {
+          if (isSorted !== firstSortDirection && ((_table$options$enable = table.options.enableSortingRemoval) != null ? _table$options$enable : true) && ( // If enableSortRemove, enable in general
+          multi ? (_table$options$enable2 = table.options.enableMultiRemove) != null ? _table$options$enable2 : true : true) // If multi, don't allow if enableMultiRemove))
+          ) {
             return false;
           }
+
+          return isSorted === 'desc' ? 'asc' : 'desc';
         },
         getCanSort: () => {
           var _column$columnDef$ena, _table$options$enable3;
@@ -3546,9 +3564,20 @@
           depth = 0;
         }
 
-        // This is the last level, just return the rows
-        if (depth === existingGrouping.length) {
-          return rows;
+        // Grouping depth has been been met
+        // Stop grouping and simply rewrite thd depth and row relationships
+        if (depth >= existingGrouping.length) {
+          return rows.map(row => {
+            row.depth = depth;
+            groupedFlatRows.push(row);
+            groupedRowsById[row.id] = row;
+
+            if (row.subRows) {
+              row.subRows = groupUpRecursively(row.subRows, depth + 1);
+            }
+
+            return row;
+          });
         }
 
         const columnId = existingGrouping[depth]; // Group the rows together for this level