fix(DatePicker): call onChange on Done button (#2089)

Author: lmaillet

packages/components/src/DateTimePickers/DateTime/Manager/Manager.component.js

@@ -24,15 +24,6 @@ import {
 	INPUT_ERRORS,
 } from '../constants';
 
-function haveErrorsChanged(oldErrors, newErrors) {
-	if (oldErrors.length !== newErrors.length) {
-		return true;
-	}
-
-	const oldErrorMessages = oldErrors.map(error => error.message);
-	return newErrors.some(error => !oldErrorMessages.includes(error.message));
-}
-
 class ContextualManager extends React.Component {
 	static displayName = 'DateTime.Manager';
 	static propTypes = {
@@ -103,16 +94,13 @@ class ContextualManager extends React.Component {
 	}
 
 	onChange(event, origin) {
-		const { errorMessage, datetime, textInput, errors, previousErrors } = this.state;
-
-		if (
-			this.props.onChange &&
-			(this.dateHasChanged() || haveErrorsChanged(previousErrors, errors))
-		) {
-			// we need to update the initial state once it has been changed
-			this.initialState = { ...this.state };
-			this.props.onChange(event, { errors, errorMessage, datetime, textInput, origin });
+		if (!this.props.onChange) {
+			return;
 		}
+		const { errorMessage, datetime, textInput, errors } = this.state;
+		// we need to update the initial state once it has been changed
+		this.initialState = { ...this.state };
+		this.props.onChange(event, { errors, errorMessage, datetime, textInput, origin });
 	}
 
 	onInputChange(event) {
@@ -224,13 +212,6 @@ class ContextualManager extends React.Component {
 		return !!this.state.errors.find(stateError => errorCodesArray.indexOf(stateError.code) > -1);
 	}
 
-	dateHasChanged() {
-		const datetime = this.state.datetime;
-		return (
-			datetime !== this.initialState.datetime && !isSameSecond(datetime, this.initialState.datetime)
-		);
-	}
-
 	render() {
 		return (
 			<DateTimeContext.Provider