Problem/Motivation

Date validation fails on month comparison as second digit of range value is ignored. For example, if end date = input date = 2012-08-31, then the validation says the date is not in the range (endpoint dates are included).

Note: You might add a component category of "JS validation" for these types of issues.

Proposed resolution

Add a second parameter to all parseInt() calls in validation code.

Remaining tasks

Review and test.

User interface changes

None.

API changes

None.

CommentFileSizeAuthor
#1 1765798-second-digit.patch1.97 KBsolotandem

Comments

solotandem’s picture

solotandem commented
Status: Active » Needs review
StatusFileSize
new 1765798-second-digit.patch1.97 KB

Patch adds missing parameter to parseInt() calls.

jelle_s’s picture

jelle_s
Primary language Dutch
Location Antwerp, Belgium
commented
Status: Needs review » Fixed

Fixed in latest dev. Thanks for the patch!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.