fix: 删除重复声明

react-component · Nov 26, 2024 · 173d4de · 173d4de
1 parent a29a6e4
commit 173d4de
Show file tree

Hide file tree

Showing 7 changed files with 13 additions and 17 deletions.
diff --git a/examples/panel.tsx b/examples/panel.tsx
@@ -99,6 +99,7 @@ export default () => {
           setValue2(nextValue);
         }}
         disabled={disabled}
+        defaultActiveKey={['bj', 'haidian']}
       />
 
       <Cascader.Panel options={addressOptions} disabled={disabled} direction="rtl" />

diff --git a/src/Cascader.tsx b/src/Cascader.tsx
@@ -159,7 +159,7 @@ export interface CascaderProps<
     value: GetValueType<OptionType, ValueField, Multiple>,
     selectOptions: OptionType[],
   ) => void;
-  defaultActiveValueCells?: React.Key[];
+  defaultActiveKey?: React.Key[];
 }
 
 export type SingleValueType = (string | number)[];
@@ -176,7 +176,7 @@ export type InternalCascaderProps = Omit<CascaderProps, 'onChange' | 'value' | '
     value: InternalValueType,
     selectOptions: BaseOptionType[] | BaseOptionType[][],
   ) => void;
-  defaultActiveValueCells?: React.Key[];
+  defaultActiveKey?: React.Key[];
 };
 
 export type CascaderRef = Omit<BaseSelectRef, 'scrollTo'>;

diff --git a/src/OptionList/List.tsx b/src/OptionList/List.tsx
@@ -29,7 +29,7 @@ export type RawOptionListProps = Pick<
   | 'direction'
   | 'open'
   | 'disabled'
-> & { defaultActiveValueCells?: React.Key[]; };
+> & { defaultActiveKey?: React.Key[]; };
 
 const RawOptionList = React.forwardRef<RefOptionListProps, RawOptionListProps>((props, ref) => {
   const {
@@ -41,7 +41,7 @@ const RawOptionList = React.forwardRef<RefOptionListProps, RawOptionListProps>((
     direction,
     open,
     disabled,
-    defaultActiveValueCells,
+    defaultActiveKey,
   } = props;
 
   const containerRef = React.useRef<HTMLDivElement>(null);
@@ -106,7 +106,7 @@ const RawOptionList = React.forwardRef<RefOptionListProps, RawOptionListProps>((
   const halfCheckedSet = React.useMemo(() => new Set(toPathKeys(halfValues)), [halfValues]);
 
   // ====================== Accessibility =======================
-  const [activeValueCells, setActiveValueCells] = useActive(multiple, open);
+  const [activeValueCells, setActiveValueCells] = useActive(multiple, open, defaultActiveKey);
 
   // =========================== Path ===========================
   const onPathOpen = (nextValueCells: React.Key[]) => {
@@ -116,12 +116,6 @@ const RawOptionList = React.forwardRef<RefOptionListProps, RawOptionListProps>((
     internalLoadData(nextValueCells);
   };
 
-  React.useEffect(() => {
-    if (defaultActiveValueCells && defaultActiveValueCells?.length > 0) {
-        setActiveValueCells(defaultActiveValueCells)
-    }
-  }, [defaultActiveValueCells]);
-
   const isSelectable = (option: DefaultOptionType) => {
     if (disabled) {
       return false;

diff --git a/src/OptionList/useActive.ts b/src/OptionList/useActive.ts
@@ -7,14 +7,15 @@ import CascaderContext from '../context';
 const useActive = (
   multiple?: boolean,
   open?: boolean,
+  defaultActiveKey?: React.Key[],
 ): [React.Key[], (activeValueCells: React.Key[]) => void] => {
   const { values } = React.useContext(CascaderContext);
 
   const firstValueCells = values[0];
 
   // Record current dropdown active options
   // This also control the open status
-  const [activeValueCells, setActiveValueCells] = React.useState<React.Key[]>([]);
+  const [activeValueCells, setActiveValueCells] = React.useState<React.Key[]>(defaultActiveKey ?? []);
 
   React.useEffect(
     () => {

diff --git a/src/Panel.tsx b/src/Panel.tsx
@@ -71,7 +71,7 @@ export default function Panel<
     direction,
     notFoundContent = 'Not Found',
     disabled,
-    defaultActiveValueCells
+    defaultActiveKey,
   } = props as Pick<InternalCascaderProps, PickType>;
 
   // ======================== Multiple ========================

diff --git a/tests/Panel.spec.tsx b/tests/Panel.spec.tsx
@@ -79,14 +79,14 @@ describe('Cascader.Panel', () => {
     expect(onChange).toHaveBeenCalledWith([['bamboo', 'little']], expect.anything());
   });
 
-  it('multiple with defaultActiveValueCells', () => {
+  it('multiple with defaultActiveKey', () => {
     const onChange = jest.fn();
     const { container } = render(
       <Cascader.Panel
         checkable
         options={options}
         onChange={onChange}
-        defaultActiveValueCells={['bamboo', 'little']}
+        defaultActiveKey={['bamboo', 'little']}
       />,
     );
     expect(container.querySelectorAll('.rc-cascader-menu')).toHaveLength(2);

diff --git a/tests/selector.spec.tsx b/tests/selector.spec.tsx
@@ -5,10 +5,10 @@ import Cascader from '../src';
 import { addressOptions } from './demoOptions';
 
 // Mock `useActive` hook
-jest.mock('../src/OptionList/useActive', () => (multiple: boolean, open: boolean) => {
+jest.mock('../src/OptionList/useActive', () => (multiple: boolean, open: boolean, defaultActiveKey: React.Key[]) => {
   // Pass to origin hooks
   const originHook = jest.requireActual('../src/OptionList/useActive').default;
-  const [activeValueCells, setActiveValueCells] = originHook(multiple, open);
+  const [activeValueCells, setActiveValueCells] = originHook(multiple, open, defaultActiveKey);
 
   (global as any).activeValueCells = activeValueCells;