Merge pull request #4602 from yuki0410-dev/fix/4594

fix: #4594 fix setSelected
Hacker0x01 · Mar 16, 2024 · a044048 · a044048
2 parents 72ded5b + 77f3516
commit a044048
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 1 deletion.
diff --git a/src/index.jsx b/src/index.jsx
@@ -697,7 +697,9 @@ export default class DatePicker extends React.Component {
         if (noRanges) {
           onChange([changedDate, null], event);
         } else if (hasStartRange) {
-          if (isDateBefore(changedDate, startDate)) {
+          if (changedDate === null) {
+            onChange([null, null], event);
+          } else if (isDateBefore(changedDate, startDate)) {
             onChange([changedDate, null], event);
           } else {
             onChange([startDate, changedDate], event);

diff --git a/test/datepicker_test.test.js b/test/datepicker_test.test.js
@@ -1444,6 +1444,30 @@ describe("DatePicker", () => {
     });
     expect(cleared).toBe(true);
   });
+  it("should correctly clear date with empty input string (selectsRange)", () => {
+    let instance;
+    const onChangeSpy = jest.fn();
+
+    render(
+      <DatePicker
+        ref={(node) => {
+          instance = node;
+        }}
+        selectsRange
+        startDate={utils.newDate("2016-11-22")}
+        endDate={null}
+        onChange={onChangeSpy}
+        isClearable
+      />,
+    );
+    fireEvent.change(instance.input, {
+      target: {
+        value: "",
+      },
+    });
+
+    expect(onChangeSpy.mock.calls.at(-1)[0]).toStrictEqual([null, null]);
+  });
   it("should correctly update the input when the value prop changes", () => {
     let instance;
     const { rerender } = render(