Change the way refs are saved in DateInput

This is to ensure unmounted components are removed from refs so that they couldn't have their values read from.

Author: wojtekmaj

src/DateInput.jsx

@@ -238,11 +238,9 @@ export default class DateInput extends PureComponent {
       onKeyDown: this.onKeyDown,
       // This is only for showing validity when editing
       required: required || isCalendarOpen,
-      itemRef: (ref) => {
-        if (!ref) return;
-
+      itemRef: (ref, name) => {
         // Save a reference to each input field
-        this[`${ref.name}Input`] = ref;
+        this[`${name}Input`] = ref;
       },
     };
   }

src/DateInput/DayInput.jsx

@@ -43,6 +43,7 @@ export default class DayInput extends PureComponent {
       className, disabled, itemRef, value, onChange, onKeyDown, required, showLeadingZeros,
     } = this.props;
 
+    const name = 'day';
     const hasLeadingZero = showLeadingZeros && value !== null && value < 10;
 
     return [
@@ -55,7 +56,7 @@ export default class DayInput extends PureComponent {
           hasLeadingZero && `${className}__input--hasLeadingZero`,
         )}
         disabled={disabled}
-        name="day"
+        name={name}
         max={maxDay}
         min={minDay}
         onChange={onChange}

src/DateInput/MonthInput.jsx

@@ -26,6 +26,7 @@ export default class MonthInput extends PureComponent {
       className, disabled, itemRef, value, onChange, onKeyDown, required, showLeadingZeros,
     } = this.props;
 
+    const name = 'month';
     const hasLeadingZero = showLeadingZeros && value !== null && value < 10;
 
     return [
@@ -38,7 +39,7 @@ export default class MonthInput extends PureComponent {
           hasLeadingZero && `${className}__input--hasLeadingZero`,
         )}
         disabled={disabled}
-        name="month"
+        name={name}
         max={maxMonth}
         min={minMonth}
         onChange={onChange}

src/DateInput/YearInput.jsx

@@ -34,14 +34,16 @@ export default class YearInput extends PureComponent {
       className, disabled, itemRef, value, onChange, onKeyDown, required,
     } = this.props;
 
+    const name = 'year';
+
     return (
       <input
         className={mergeClassNames(
           `${className}__input`,
           `${className}__year`,
         )}
         disabled={disabled}
-        name="year"
+        name={name}
         max={maxYear}
         min={minYear}
         onChange={onChange}