Opened 13 years ago

Closed 13 years ago

Last modified 10 years ago

#1152 closed defect (fixed)

dojo.validation.isInRange("a") validates as true

Reported by: dylan Owned by: dylan
Priority: high Milestone: 0.9
Component: Core Version: 0.2
Keywords: Cc: ktiedt@…
Blocked By: Blocking:

Description

dojo.validation.isInRange("a") validates as true

Change History (3)

comment:1 Changed 13 years ago by dylan

Resolution: → fixed
Status: new → closed

fixed in [4901], and testcase added to unit tests

comment:2 Changed 13 years ago by dylan

Milestone: 0.4 → 0.5
Resolution: fixed
Status: closed → reopened

This won't work because it breaks things like +1,000 ... reverting for now, need a better solution later.

comment:3 Changed 13 years ago by tk

Cc: ktiedt@… added
Resolution: → fixed
Status: reopened → closed

r5829 improves upon this method, if a seperator is supplied, it strips that charactor from the value, if not, it assumes ',' as the separator. This allows isNaN() to be used which returns false to "A"

fixed by r5829

Note: See TracTickets for help on using tickets.