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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

solotandem’s picture

solotandem CreditAttribution: solotandem commented
Status: Active » Needs review
FileSize
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
CreditAttribution: Jelle_S 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.