fix: shortcus with range (#400)

mengxiong10 · Dec 17, 2019 · bb0255f · bb0255f
1 parent b33c9b8
commit bb0255f
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/__test__/date-picker.test.js b/__test__/date-picker.test.js
@@ -212,6 +212,19 @@ describe('DatePicker', () => {
     btn.trigger('click');
     const emitted = wrapper.emitted();
     expect(emitted.input).toEqual([['2019/05/10']]);
+    wrapper.setProps({
+      range: true,
+      shortcuts: [
+        {
+          text: 'range',
+          onClick() {
+            return [date, date];
+          },
+        },
+      ],
+    });
+    btn.trigger('click');
+    expect(emitted.input[1]).toEqual([['2019/05/10', '2019/05/10']]);
   });
 
   it('prop: popupClass', () => {

diff --git a/src/date-picker.vue b/src/date-picker.vue
@@ -379,7 +379,7 @@ export default {
     handleSelectShortcut(item) {
       if (isObject(item) && typeof item.onClick === 'function') {
         const date = item.onClick(this);
-        if (date instanceof Date) {
+        if (date) {
           this.emitValue(date);
         }
       }