Skip to content

Commit

Permalink
Refactor constructing date in onChangeExternal
Browse files Browse the repository at this point in the history
  • Loading branch information
wojtekmaj committed Mar 26, 2023
1 parent a532005 commit 2b86d17
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions src/DateInput.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -493,17 +493,20 @@ export default class DateInput extends PureComponent {

const values = {};
formElements.forEach((formElement) => {
values[formElement.name] = formElement.value;
values[formElement.name] =
'valueAsNumber' in formElement
? formElement.valueAsNumber
: parseInt(formElement.value, 10);
});

if (formElements.every((formElement) => !formElement.value)) {
onChange(null, false);
} else if (
formElements.every((formElement) => formElement.value && formElement.validity.valid)
) {
const year = parseInt(values.year, 10) || new Date().getFullYear();
const monthIndex = parseInt(values.month || 1, 10) - 1;
const day = parseInt(values.day || 1, 10);
const year = values.year || new Date().getFullYear();
const monthIndex = (values.month || 1) - 1;
const day = values.day || 1;

const proposedValue = new Date();
proposedValue.setFullYear(year, monthIndex, day);
Expand Down

0 comments on commit 2b86d17

Please sign in to comment.