From df22ef2644fc398a95aef369d82fe36e37610725 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E4=BA=8C=E8=B4=A7=E6=9C=BA=E5=99=A8=E4=BA=BA?=
 <smith3816@gmail.com>
Date: Mon, 25 Nov 2024 14:53:01 +0800
Subject: [PATCH] fix: multiple select none current panel should not trigger
 change

---
 src/PickerInput/SinglePicker.tsx |  5 +++++
 tests/multiple.spec.tsx          | 36 ++++++++++++++++++++++++++++++++
 2 files changed, 41 insertions(+)

diff --git a/src/PickerInput/SinglePicker.tsx b/src/PickerInput/SinglePicker.tsx
index 9fe862d84..552c29e7d 100644
--- a/src/PickerInput/SinglePicker.tsx
+++ b/src/PickerInput/SinglePicker.tsx
@@ -441,6 +441,11 @@ function Picker<DateType extends object = any>(
   const onPanelSelect = (date: DateType) => {
     lastOperation('panel');
 
+    // Not change values if multiple and current panel is to match with picker
+    if (multiple && internalMode !== picker) {
+      return;
+    }
+
     const nextValues = multiple ? toggleDates(getCalendarValue(), date) : [date];
 
     // Only trigger calendar event but not update internal `calendarValue` state
diff --git a/tests/multiple.spec.tsx b/tests/multiple.spec.tsx
index fa96e4676..f12facf80 100644
--- a/tests/multiple.spec.tsx
+++ b/tests/multiple.spec.tsx
@@ -155,4 +155,40 @@ describe('Picker.Multiple', () => {
       ).toBeFalsy();
     });
   });
+
+  it('click year panel should not select', () => {
+    const onChange = jest.fn();
+    const onCalendarChange = jest.fn();
+    const { container } = render(
+      <DayPicker multiple onChange={onChange} onCalendarChange={onCalendarChange} needConfirm />,
+    );
+
+    expect(container.querySelector('.rc-picker-multiple')).toBeTruthy();
+
+    openPicker(container);
+
+    // Select year
+    fireEvent.click(document.querySelector('.rc-picker-year-btn'));
+    selectCell(1998);
+    expect(onChange).not.toHaveBeenCalled();
+    expect(onCalendarChange).not.toHaveBeenCalled();
+
+    // Select Month
+    selectCell('Oct');
+    expect(onChange).not.toHaveBeenCalled();
+    expect(onCalendarChange).not.toHaveBeenCalled();
+
+    // Select Date
+    selectCell(23);
+    expect(onChange).not.toHaveBeenCalled();
+    expect(onCalendarChange).toHaveBeenCalledWith(
+      expect.anything(),
+      ['1998-10-23'],
+      expect.anything(),
+    );
+
+    // Confirm
+    fireEvent.click(document.querySelector('.rc-picker-ok button'));
+    expect(onChange).toHaveBeenCalledWith(expect.anything(), ['1998-10-23']);
+  });
 });