mirror/sunmao-ui
2
0
Fork 0
mirror of https://github.com/smartxworks/sunmao-ui.git synced 2025-03-31 21:30:22 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(Table): fix where the state was not updated correctly in some cases

Browse Source
This commit is contained in:
xzdry 2023-05-15 11:27:16 +08:00
parent b8c3a3c259
commit dd3d034463
1 changed files with 11 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
packages/arco-lib/src/components/Table/Table.tsx
View File

@ -215,7 +215,7 @@ export const Table = implementRuntimeComponent({
} = pagination;
const rowSelectionType = rowSelectionTypeMap[cProps.rowSelectionType];
const currentChecked = useRef<Record<string, unknown>[]>([]);
const currentChecked = useRef<(string | number)[]>([]);
const currentClickedRow = useRef<(string | number)[] | undefined>(undefined);
const [currentPage, setCurrentPage] = useStateValue<number>(
@ -274,18 +274,22 @@ export const Table = implementRuntimeComponent({
// reset selectedRows state when data changed
useEffect(() => {
const currentCheckedRowKeys = currentChecked.current.map(
row => row[rowKey] as string
// under server-side paging and with checkCrossPage enabled, the previous data should be cached without having to update it
if (useCustomPagination && checkCrossPage) {
return;
}
const selectedRows = currentPageData.filter(d =>
currentChecked.current.includes(d[rowKey])
);
// TODO: Save clickedRow state when rowkey changes, save the UI of clickedRow when turning the page
const clickedRow = currentPageData.find(d => d[rowKey] === currentClickedRow.current);
if (!clickedRow) currentClickedRow.current = undefined;
mergeState({
selectedRowKeys: currentCheckedRowKeys,
selectedRows: currentChecked.current,
selectedRowKeys: selectedRows.map(r => r[rowKey]),
selectedRows,
clickedRow,
});
}, [currentPageData, mergeState, rowKey]);
}, [checkCrossPage, currentPageData, mergeState, rowKey, useCustomPagination]);
// If there is less data to display than the current page, reset to the first page
useEffect(() => {
@ -573,7 +577,7 @@ export const Table = implementRuntimeComponent({
// This option is required to achieve multi-selection across pages when customizing paging
preserveSelectedRowKeys: useCustomPagination ? checkCrossPage : undefined,
onChange(selectedRowKeys, selectedRows) {
currentChecked.current = selectedRows;
currentChecked.current = selectedRowKeys;
mergeState({
selectedRowKeys: selectedRowKeys as string[],
selectedRows,