fix: no-disabled-range prop being ignored on auto-range (fixes #555)

Vuepic · Sep 8, 2023 · 5916ab3 · 5916ab3
1 parent e092dad
commit 5916ab3
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 2 deletions.
diff --git a/src/VueDatePicker/composables/validation.ts b/src/VueDatePicker/composables/validation.ts
@@ -131,8 +131,11 @@ export const useValidation = (props: PickerBasePropsType | AllPropsType) => {
 
     // Check if there are disabled dates for a given range
     const isDateRangeAllowed = (range: Date[]): boolean => {
-        const datesInBetween = eachDayOfInterval({ start: range[0], end: range[1] });
-        return !datesInBetween.some((date) => isDisabled(date));
+        if (props.noDisabledRange) {
+            const datesInBetween = eachDayOfInterval({ start: range[0], end: range[1] });
+            return !datesInBetween.some((date) => isDisabled(date));
+        }
+        return true;
     };
 
     // If min or max range is set, validate given range

diff --git a/tests/unit/behaviour.spec.ts b/tests/unit/behaviour.spec.ts
@@ -263,4 +263,25 @@ describe('It should validate various picker scenarios', () => {
         expect(cell.html()).toBeTruthy();
         dp.unmount();
     });
+
+    it('Should enable or disable auto range selecting with no-disabled-range prop (#555)', async () => {
+        const today = set(new Date(), { seconds: 0, milliseconds: 0 });
+        const disabledDates = [addDays(today, 1), addDays(today, 2)];
+        const dp = await openMenu({ disabledDates, noDisabledRange: true, range: true, autoRange: 5 });
+
+        const selectAutoRange = async () => {
+            await dp.find(`[data-test="${resetDateTime(today)}"]`).trigger('click');
+            await dp.find(`[data-test="select-button"]`).trigger('click');
+        };
+
+        await selectAutoRange();
+
+        expect(dp.emitted()).toHaveProperty('invalid-select');
+
+        await reOpenMenu(dp, { noDisabledRange: false });
+
+        await selectAutoRange();
+
+        expect(dp.emitted()).toHaveProperty('update:model-value', [[[today, addDays(today, 5)]]]);
+    });
 });