#16281 closed defect (fixed)
dojo.date.locale.parse not working on 1.8.1 CDN
| Reported by: | jajubear | Owned by: | Adam Peller |
|---|---|---|---|
| Priority: | undecided | Milestone: | 1.8.2 |
| Component: | Date | Version: | 1.8.1 |
| Keywords: | Cc: | Colin Snover | |
| Blocked By: | Blocking: |
Description
Attached is a simple code for showing dojo.date.locale.parse working on 1.8.0 CDN but not 1.8.1 CDN
1.8.0 returns a date value 1.8.1 returns null, not sure why it is so
Just change the CDN on the attached code to see the problem
Attachments (1)
Change History (6)
Changed 9 years ago by
| Attachment: | date_test.html added |
|---|
comment:1 Changed 9 years ago by
| Component: | General → Date |
|---|---|
| Owner: | set to Adam Peller |
comment:2 Changed 9 years ago by
| Milestone: | tbd → 1.8.2 |
|---|
comment:3 Changed 9 years ago by
| Cc: | Colin Snover added |
|---|
The CLDR data on the CDN does not match the main build. Specifically,
http://ajax.googleapis.com/ajax/libs/dojo/1.8.1/dojo/cldr/nls/en/gregorian.js
"dateTimeFormat-medium":"{1}, {0}"
"dateTimeFormat-full":"{1} 'at' {0}"
appears to be current, but the released code has:
"dateTimeFormat-medium":"{1} {0}"
"dateTimeFormat-full":"{1} {0}"
Note: See
TracTickets for help on using
tickets.
The regression appears to be that dates and times are separated by a comma in 1.8.1. Need to determine if this was a change to the CLDR locale data or some regression.
>dojo.date.locale.format(new Date(2012,9,30,0,11,22), {datePattern:'yyyy-MM-dd',timePattern:'HH:mm:ss'}) "2012-10-30, 00:11:22"Note that you'll probably get better performance if you do something like
dojo.date.stamp.fromISOString("2012-10-30 00:11:22".replace(" ","T"))which does not require any locale data