#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 10 years ago by
| Attachment: | date_test.html added |
|---|
comment:1 Changed 10 years ago by
| Component: | General → Date |
|---|---|
| Owner: | set to Adam Peller |
comment:2 Changed 10 years ago by
| Milestone: | tbd → 1.8.2 |
|---|
comment:3 Changed 10 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