fix: should use moment.isMoment, close: #4147

ant-design · Dec 6, 2016 · 273faaf · 273faaf · afc163 · Dec 6, 2016
1 parent 8830668
commit 273faaf
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/components/calendar/index.tsx b/components/calendar/index.tsx
@@ -65,7 +65,7 @@ export default class Calendar extends React.Component<CalendarProps, any> {
   constructor(props) {
     super(props);
     const value = props.value || props.defaultValue || moment();
-    if (!(value instanceof moment)) {
+    if (!moment.isMoment(value)) {
       throw new Error(
         'The value/defaultValue of Calendar must be a moment object after `antd@2.0`, ' +
         'see: http://u.ant.design/calendar-value'

diff --git a/components/date-picker/RangePicker.tsx b/components/date-picker/RangePicker.tsx
@@ -16,8 +16,8 @@ export default class RangePicker extends React.Component<any, any> {
     super(props);
     const value = props.value || props.defaultValue || [];
     if (
-      value[0] && !(value[0] instanceof moment) ||
-      value[1] && !(value[1] instanceof moment)
+      value[0] && !moment.isMoment(value[0]) ||
+      value[1] && !moment.isMoment(value[1])
     ) {
       throw new Error(
         'The value/defaultValue of RangePicker must be a moment object array after `antd@2.0`, ' +

diff --git a/components/time-picker/index.tsx b/components/time-picker/index.tsx
@@ -64,7 +64,7 @@ export default class TimePicker extends React.Component<TimePickerProps, any> {
   constructor(props) {
     super(props);
     const value = props.value || props.defaultValue;
-    if (value && !(value instanceof moment)) {
+    if (value && !moment.isMoment(value)) {
       throw new Error(
         'The value/defaultValue of TimePicker must be a moment object after `antd@2.0`, ' +
         'see: http://u.ant.design/time-picker-value'