@redsift/table 11.8.2-muiv5 → 11.8.3-muiv5

package/_internal/ControlledPagination.js

@@ -2,7 +2,7 @@ import { a as _extends$1 } from './_rollupPluginBabelHelpers.js';
 import * as React from 'react';
 import React__default, { Children, isValidElement, cloneElement, createElement } from 'react';
 import { Flexbox, Text, LinkButton } from '@redsift/design-system';
-import { gridPaginatedVisibleSortedGridRowEntriesSelector, gridPaginatedVisibleSortedGridRowIdsSelector, gridFilteredSortedRowEntriesSelector, gridFilteredSortedRowIdsSelector } from '@mui/x-data-grid-pro';
+import { gridFilteredSortedRowEntriesSelector, gridFilteredSortedRowIdsSelector } from '@mui/x-data-grid-pro';
 import { t as chainPropTypes, P as PropTypes, p as ownerDocument, w as useEnhancedEffect, u as useTheme$1, z as defaultTheme, T as THEME_ID, v as useForkRef, e as exactProp, j as generateUtilityClass, h as generateUtilityClasses, _ as _objectWithoutPropertiesLoose, a as _extends, x as useSlotProps, q as Portal, H as HTMLElementType, n as composeClasses, y as useClassNamesOverride, k as styled, l as useThemeProps, c as clsx, A as isHostComponent, o as refType, m as capitalize, B as formatMuiErrorMessage, r as rootShouldForwardProp, D as deepmerge, E as reactIsExports, F as resolveComponentProps, G as alpha, I as slotShouldForwardProp, J as lighten, K as darken } from './Portal.js';
 import { j as jsxRuntimeExports } from './jsx-runtime.js';
 import { Global, keyframes } from '@emotion/react';
@@ -10321,19 +10321,22 @@ process.env.NODE_ENV !== "production" ? TablePagination.propTypes /* remove-prop
 var TablePagination$1 = TablePagination;
 
 const onServerSideSelectionStatusChange = (newSelectionModel, apiRef, selectionStatus, isRowSelectable, page, pageSize) => {
-  const selectableRowsInPage = isRowSelectable ? gridPaginatedVisibleSortedGridRowEntriesSelector(apiRef).filter(_ref => {
-    let {
-      model
-    } = _ref;
-    return isRowSelectable({
-      row: model
-    });
-  }).map(_ref2 => {
+  // We can't rely on the gridPaginatedVisibleSortedGridRowEntriesSelector(apiRef) function to fetch the list of visible rows
+  // as it doesn't work properly when all rows are selected on a page via checkbox
+  // and then navigate to another page.
+  // So instead we fetch the visible rows directly from the apiRef state, since the logic in `onServerSideSelectionStatusChange` applies
+  // to server-side pagination only and visible rows are all loaded rows.
+  // The bug may have been latent for several MUI releases now, as it seems to have been fixed in MUI v7.
+  // See https://github.com/mui/mui-x/pull/14083
+  const rowsInPage = Array.from(apiRef.current.getVisibleRowModels().values());
+  const selectableRowsInPage = isRowSelectable ? rowsInPage.filter(model => isRowSelectable({
+    row: model
+  })).map(_ref => {
     let {
       id
-    } = _ref2;
+    } = _ref;
     return id;
-  }) : gridPaginatedVisibleSortedGridRowIdsSelector(apiRef);
+  }) : rowsInPage;
   const numberOfSelectableRowsInPage = selectableRowsInPage.length;
   const numberOfSelectedRows = newSelectionModel.length;
   const selectedRowsInPage = selectableRowsInPage.filter(rowId => newSelectionModel.includes(rowId));
@@ -10375,7 +10378,7 @@ const onServerSideSelectionStatusChange = (newSelectionModel, apiRef, selectionS
     };
   }
 };
-const ServerSideControlledPagination = _ref3 => {
+const ServerSideControlledPagination = _ref2 => {
   let {
     selectionStatus,
     displaySelection,
@@ -10388,7 +10391,7 @@ const ServerSideControlledPagination = _ref3 => {
     rowsPerPageOptions,
     paginationProps,
     rowCount
-  } = _ref3;
+  } = _ref2;
   const totalNumberOfRowsInTable = rowCount;
   const totalRowsLabel = `${selectionStatus.numberOfSelectedRows} row${selectionStatus.numberOfSelectedRows > 1 ? 's' : ''} selected`;
   const pageRowsLabel = `All ${selectionStatus.numberOfSelectedRowsInPage} selectable rows on this page are selected${selectionStatus.numberOfSelectedRows != selectionStatus.numberOfSelectedRowsInPage ? ` (${selectionStatus.numberOfSelectedRows} row${selectionStatus.numberOfSelectedRows > 1 ? 's' : ''} selected in total)` : ''}.`;