From 18077894f271e102c80af14e62ad2aeac07152fe Mon Sep 17 00:00:00 2001
From: Graeme Yeates <yeatesgraeme@gmail.com>
Date: Tue, 7 Mar 2017 05:00:51 -0500
Subject: [PATCH] Remove DatePicker temporary values [fix #5189] (#5191)

---
 components/date-picker/createPicker.tsx | 33 +++++++++----------------
 1 file changed, 11 insertions(+), 22 deletions(-)

diff --git a/components/date-picker/createPicker.tsx b/components/date-picker/createPicker.tsx
index 4d0b503293..2bcfa6ff23 100644
--- a/components/date-picker/createPicker.tsx
+++ b/components/date-picker/createPicker.tsx
@@ -39,7 +39,6 @@ export default function createPicker(TheCalendar) {
       }
       return {
         value,
-        tempValue: undefined,
       };
     },
 
@@ -47,7 +46,6 @@ export default function createPicker(TheCalendar) {
       if ('value' in nextProps) {
         this.setState({
           value: nextProps.value,
-          tempValue: nextProps.value,
         });
       }
     },
@@ -66,34 +64,25 @@ export default function createPicker(TheCalendar) {
       props.onChange(value, (value && value.format(props.format)) || '');
     },
 
-    handleTempChange(tempValue) {
-      this.setState({ tempValue });
+    handleTempChange(value) {
+      const props = this.props;
+      if (!('value' in props)) {
+        this.setState({ value });
+      }
+      props.onChange(value, (value && value.format(props.format)) || '');
     },
 
-    // Clear temp value and trigger onChange when hide DatePicker[showTime] panel
+    // Trigger onChange when hide DatePicker[showTime] panel
     handleOpenChange(open) {
-      const { showTime, onOpenChange, onChange, format } = this.props;
-      if (!open) {
-        // tricky code to avoid triggering onChange multiple times
-        // when click `Now` button
-        let tempValue;
-        this.setState(prevState => {
-          tempValue = prevState.tempValue;
-          const nextState = { tempValue: undefined } as any;
-          if (showTime && tempValue) {
-            nextState.value = tempValue;
-            onChange(tempValue, (tempValue && tempValue.format(format)) || '');
-          }
-          return nextState;
-        });
-      }
+      const { onOpenChange } = this.props;
+
       if (onOpenChange) {
         onOpenChange(open);
       }
     },
 
     render() {
-      const { value, tempValue } = this.state;
+      const { value } = this.state;
       const props = omit(this.props, ['onChange']);
       const { prefixCls, locale } = props;
 
@@ -169,7 +158,7 @@ export default function createPicker(TheCalendar) {
         </span>
       );
 
-      const pickerValue = tempValue || value;
+      const pickerValue = value;
       const localeCode = getLocaleCode(this.context);
       if (pickerValue && localeCode) {
         pickerValue.locale(localeCode);