Skip to content

Commit

Permalink
fix: handle NaN when parsing number inputs closes #4328
Browse files Browse the repository at this point in the history
  • Loading branch information
logaretm committed Jun 23, 2023
1 parent 435e785 commit fc41691
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 1 deletion.
5 changes: 5 additions & 0 deletions .changeset/neat-phones-crash.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'vee-validate': patch
---

fix: handle NaN when parsing number inputs closes #4328
2 changes: 1 addition & 1 deletion packages/vee-validate/src/utils/events.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import { getBoundValue, hasValueBinding } from './vnode';

function parseInputValue(el: HTMLInputElement) {
if (el.type === 'number') {
return el.valueAsNumber;
return Number.isNaN(el.valueAsNumber) ? el.value : el.valueAsNumber;
}

return el.value;
Expand Down
4 changes: 4 additions & 0 deletions packages/vee-validate/tests/useField.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -892,6 +892,10 @@ describe('useField()', () => {
setValue(input, '123');
await flushPromises();
expect(field.value.value).toBe(123);

setValue(input, '');
await flushPromises();
expect(field.value.value).toBe('');
});

test('a validator can return multiple messages', async () => {
Expand Down

0 comments on commit fc41691

Please sign in to comment.