remove onPickerChange

README.md

@@ -118,7 +118,6 @@ in [RNPlayground](https://github.com/react-component/RNPlayground)
 |date | The currently selected date. | moment |  |
 |visible | whether pop picker is visible | Boolean | false |
 |onChange | exec on ok | Function(date: moment) |  |
-|onPickerChange | Date inside picker change handler. | Function(date: moment) | '' |
 |onVisibleChange | called when pop picker visible change | Function | |
 |onDismiss | exec on dismiss | function |  |
 |okText | ok button text | String | 'Ok' |

examples/popup.tsx

@@ -54,6 +54,11 @@ class Demo extends React.Component<any, any> {
       date,
     });
   };
+
+  onPickerChange(date) {
+    console.log('onPickerChange', format(date));
+  }
+
   onDismiss = () => {
     console.log('onDismiss');
   };
@@ -72,6 +77,7 @@ class Demo extends React.Component<any, any> {
         maxDate={maxDate}
         defaultDate={now}
         mode={props.mode}
+        onDateChange={this.onPickerChange}
         locale={props.locale}
       />
     );

examples/react-native/popup.tsx

@@ -59,12 +59,11 @@ class Demo extends React.Component<any, any> {
       date,
     });
   };
+
   onDismiss = () => {
     console.log('onDismiss');
   };
-  onPickerChange = (date) => {
-    console.log('onPickerChange', format(date));
-  };
+
   show = () => {
     console.log('my click');
   };
@@ -91,7 +90,6 @@ class Demo extends React.Component<any, any> {
           styles={PopupStyles}
           title="Date picker"
           date={date}
-          onPickerChange={this.onPickerChange}
           onDismiss={this.onDismiss}
           onChange={this.onChange}
         >

package.json

@@ -1,6 +1,6 @@
 {
   "name": "rmc-date-picker",
-  "version": "5.0.0",
+  "version": "5.0.1",
   "description": "React Mobile DatePicker Component for web and react-native",
   "keywords": [
     "react",

src/Popup.tsx

@@ -44,7 +44,9 @@ export default class PopupDatePicker extends React.Component<PopupDatePickerProp
     this.setState({
       pickerDate,
     });
-    this.props.onPickerChange(pickerDate);
+    if (this.props.datePicker.props.onDateChange) {
+      this.props.datePicker.props.onDateChange(pickerDate);
+    }
   };
 
   onOk = () => {
@@ -66,14 +68,6 @@ export default class PopupDatePicker extends React.Component<PopupDatePickerProp
     this.datePicker = datePicker;
   };
 
-  getModal() {
-    return React.cloneElement(this.props.datePicker, {
-      date: this.state.pickerDate || this.props.date,
-      onDateChange: this.onPickerChange,
-      ref: this.saveRef,
-    } as DatePickerProps);
-  }
-
   fireVisibleChange = (visible) => {
     if (this.state.visible !== visible) {
       if (!('visible' in this.props)) {
@@ -84,11 +78,17 @@ export default class PopupDatePicker extends React.Component<PopupDatePickerProp
   };
 
   render() {
+    const dataPicker = React.cloneElement(this.props.datePicker, {
+      date: this.state.pickerDate || this.props.date,
+      onDateChange: this.onPickerChange,
+      ref: this.saveRef,
+    } as DatePickerProps);
+
     return (<PopupPicker
       {...this.props}
       onVisibleChange={this.fireVisibleChange}
       onOk={this.onOk}
-      content={this.getModal()}
+      content={dataPicker}
       visible={this.state.visible}
     />);
   }