fix: When value is null, the dateString in onchange is null (#942)

* fix: When value is null dateString=null

* feat: add test to multiple picker

---------

Co-authored-by: 刘欢 <[email protected]>

Author: EmilyyyLiu

src/PickerInput/hooks/useRangeValue.ts

@@ -283,10 +283,11 @@ export default function useRangeValue<ValueType extends DateType[], DateType ext
 
       // Trigger `onChange` if needed
       if (onChange && !isSameMergedDates) {
+        const everyEmpty = clone.every((val) => !val);
         onChange(
           // Return null directly if all date are empty
-          isNullValue && clone.every((val) => !val) ? null : clone,
-          getDateTexts(clone),
+          isNullValue && everyEmpty ? null : clone,
+          everyEmpty ? null : getDateTexts(clone),
         );
       }
     }

tests/multiple.spec.tsx

@@ -2,7 +2,7 @@
 import { fireEvent, render } from '@testing-library/react';
 import { resetWarned } from '@rc-component/util/lib/warning';
 import React from 'react';
-import { DayPicker, getDay, isOpen, openPicker, selectCell } from './util/commonUtil';
+import { clearValue, DayPicker, getDay, isOpen, openPicker, selectCell } from './util/commonUtil';
 
 const fakeTime = getDay('1990-09-03 00:00:00').valueOf();
 
@@ -57,6 +57,8 @@ describe('Picker.Multiple', () => {
     ]);
 
     expect(onChange.mock.calls[0][0]).toHaveLength(3);
+    clearValue();
+    expect(onChange).toHaveBeenCalledWith(null, null);
   });
 
   it('panel click to remove', () => {

tests/picker.spec.tsx

@@ -283,7 +283,7 @@ describe('Picker.Basic', () => {
         onChange.mockReset();
 
         clearValue();
-        expect(onChange).toHaveBeenCalledWith(null, '');
+        expect(onChange).toHaveBeenCalledWith(null, null);
         expect(isOpen()).toBeFalsy();
 
         openPicker(container);

tests/range.spec.tsx

@@ -202,7 +202,7 @@ describe('Picker.Range', () => {
     expect(onChange).toHaveBeenCalledWith([expect.anything(), null], ['1990-09-11', '']);
 
     clearValue();
-    expect(onChange).toHaveBeenCalledWith(null, ['', '']);
+    expect(onChange).toHaveBeenCalledWith(null, null);
     onChange.mockReset();
 
     // Not allow empty with startDate