#7826 closed defect (fixed)
[regression] NumberTextBox invalid values being submitted as NaN
Reported by: | Douglas Hays | Owned by: | Douglas Hays |
---|---|---|---|
Priority: | high | Milestone: | 1.2.1 |
Component: | Dijit - Form | Version: | 1.2beta |
Keywords: | Cc: | Nathan Toone | |
Blocked By: | Blocking: |
Description
Starting with 1.2, blank NumberTextBox? form values submit as "NaN". It should be "" (blank) as in 1.1.1.
Attachments (1)
Change History (6)
Changed 12 years ago by
Attachment: | 7826_12x.patch added |
---|
comment:1 Changed 12 years ago by
Run test_validate.html, press "View data" button at the bottom and you can see that the "fav" box is submitting "NaN". It should be blank. This is a regression from 1.1.1.
comment:2 Changed 12 years ago by
Cc: | Nathan Toone added |
---|
toonetown, can you review this patch since bill is away?
comment:3 Changed 12 years ago by
Looks OK to me - the only question I would have is the need to change the postCreate for TextBox? (though it looks like an OK change to make...)
comment:4 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
[15507] References #7826 for 1.2.1. Add serialize method to NumberTextBox? to always send a blank for invalid values.
[15508] Fixes #7826 in trunk. Add serialize method to NumberTextBox? to always send a blank on form submit for invalid values.
comment:5 Changed 10 years ago by
Component: | Dijit → Dijit - Form |
---|
overwrite serialize method and don't filter form reset value