In the fix for https://www.drupal.org/node/1718292 when datepicker options are exploded into key/value pairs the split should happen only at the first colon. This is an issue because some optoins require a colon, such as yearRange: "1900:c".

CommentFileSizeAuthor
#1 bef-datepicker-settings-2493207-1.patch684 bytesawolfey
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

awolfey’s picture

awolfey CreditAttribution: awolfey commented
FileSize
bef-datepicker-settings-2493207-1.patch684 bytes

Here's the patch.

awolfey’s picture

awolfey CreditAttribution: awolfey commented
Status: Active » Needs review

  • mikeker committed 0b5dff6 on 7.x-3.x authored by awolfey 
    Issue #2493207 by awolfey: Datepicker options including a colon break...
mikeker’s picture

mikeker CreditAttribution: mikeker as a volunteer commented
Status: Needs review » Fixed

Good catch and thanks for the patch!

Fixed.

Status: Fixed » Closed (fixed)

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