From 9e7e32b60ce80363c5865eb10bdf44d2938b3721 Mon Sep 17 00:00:00 2001
From: kalykun <984757534@qq.com>
Date: Thu, 4 Aug 2022 11:16:24 +0800
Subject: [PATCH] fix(Table): onChange `sorter` param types (#36710)

* fix(types): fixes `sorter` param types

* chore: add test case

* Update type.test.tsx

* Update type.test.tsx
---
 components/table/Table.tsx               |  6 +++---
 components/table/__tests__/type.test.tsx |  6 ++++++
 components/table/hooks/useSorter.tsx     | 12 ++++--------
 3 files changed, 13 insertions(+), 11 deletions(-)

diff --git a/components/table/Table.tsx b/components/table/Table.tsx
index 212536d5b28e..a8c524f22e34 100644
--- a/components/table/Table.tsx
+++ b/components/table/Table.tsx
@@ -61,7 +61,7 @@ interface ChangeEventInfo<RecordType> {
     total?: number;
   };
   filters: Record<string, FilterValue | null>;
-  sorter: SorterResult<RecordType> | SorterResult<RecordType>[];
+  sorter: SorterResult<RecordType> | SorterResult<RecordType[]>;
 
   filterStates: FilterState<RecordType>[];
   sorterStates: SortState<RecordType>[];
@@ -92,7 +92,7 @@ export interface TableProps<RecordType>
   onChange?: (
     pagination: TablePaginationConfig,
     filters: Record<string, FilterValue | null>,
-    sorter: SorterResult<RecordType> | SorterResult<RecordType>[],
+    sorter: SorterResult<RecordType> | SorterResult<RecordType[]>,
     extra: TableCurrentDataSource<RecordType>,
   ) => void;
   rowSelection?: TableRowSelection<RecordType>;
@@ -263,7 +263,7 @@ function InternalTable<RecordType extends object = any>(
 
   // ============================ Sorter =============================
   const onSorterChange = (
-    sorter: SorterResult<RecordType> | SorterResult<RecordType>[],
+    sorter: SorterResult<RecordType> | SorterResult<RecordType[]>,
     sorterStates: SortState<RecordType>[],
   ) => {
     triggerOnChange(
diff --git a/components/table/__tests__/type.test.tsx b/components/table/__tests__/type.test.tsx
index a064b0f69fe1..e33dad2f8f66 100644
--- a/components/table/__tests__/type.test.tsx
+++ b/components/table/__tests__/type.test.tsx
@@ -36,6 +36,12 @@ describe('Table.typescript', () => {
     const table = <Table<RecordType> dataSource={[{ key: 'Bamboo' }]} />;
     expect(table).toBeTruthy();
   });
+
+  it('Sorter types', () => {
+    const table = <Table onChange={(_pagination, _filters, sorter) => sorter.field} />;
+
+    expect(table).toBeTruthy();
+  });
 });
 
 describe('Table.typescript types', () => {
diff --git a/components/table/hooks/useSorter.tsx b/components/table/hooks/useSorter.tsx
index 2c08a33ab642..9784f92ded0c 100644
--- a/components/table/hooks/useSorter.tsx
+++ b/components/table/hooks/useSorter.tsx
@@ -247,7 +247,7 @@ function stateToInfo<RecordType>(sorterStates: SortState<RecordType>) {
 
 function generateSorterInfo<RecordType>(
   sorterStates: SortState<RecordType>[],
-): SorterResult<RecordType> | SorterResult<RecordType>[] {
+): SorterResult<RecordType> | SorterResult<RecordType[]> {
   const list = sorterStates.filter(({ sortOrder }) => sortOrder).map(stateToInfo);
 
   // =========== Legacy compatible support ===========
@@ -259,11 +259,7 @@ function generateSorterInfo<RecordType>(
     };
   }
 
-  if (list.length <= 1) {
-    return list[0] || {};
-  }
-
-  return list;
+  return list[0] || {};
 }
 
 export function getSortData<RecordType>(
@@ -324,7 +320,7 @@ interface SorterConfig<RecordType> {
   prefixCls: string;
   mergedColumns: ColumnsType<RecordType>;
   onSorterChange: (
-    sorterResult: SorterResult<RecordType> | SorterResult<RecordType>[],
+    sorterResult: SorterResult<RecordType> | SorterResult<RecordType[]>,
     sortStates: SortState<RecordType>[],
   ) => void;
   sortDirections: SortOrder[];
@@ -343,7 +339,7 @@ export default function useFilterSorter<RecordType>({
   TransformColumns<RecordType>,
   SortState<RecordType>[],
   ColumnTitleProps<RecordType>,
-  () => SorterResult<RecordType> | SorterResult<RecordType>[],
+  () => SorterResult<RecordType> | SorterResult<RecordType[]>,
 ] {
   const [sortStates, setSortStates] = React.useState<SortState<RecordType>[]>(
     collectSortStates(mergedColumns, true),