Opened 8 years ago
Closed 8 years ago
#17071 closed defect (fixed)
gregorian.js cldr nls no longer has fields
Reported by: | Pete Smith | Owned by: | bill |
---|---|---|---|
Priority: | undecided | Milestone: | 1.9 |
Component: | Internationalization | Version: | 1.9.0b2 |
Keywords: | Cc: | Douglas Hays, [email protected]…, srl | |
Blocked By: | Blocking: |
Description
I was happily using the field- properties in the gregorian resource for things like:
"field-year": "Year"
And they are gone in the 1.9.0RC1.
Change History (13)
comment:1 follow-up: 3 Changed 8 years ago by
Cc: | Douglas Hays added |
---|
comment:2 Changed 8 years ago by
Cc: | [email protected]… added |
---|
comment:3 Changed 8 years ago by
Replying to peller:
It looks like the LDML schema changed, and the fields tag is now a peer of calendar, which makes sense. Unfortunately, it means fixing our xslt...
CLDR bug is CldrBug:5512 http://unicode.org/cldr/trac/ticket/5512 (you can add me to cc)
comment:4 Changed 8 years ago by
Cc: | srl added |
---|
Thanks, srl. Also copying emmo to see if this data is included in the new JSON structure.
comment:5 Changed 8 years ago by
So previously in en.xml, <fields> was under the <calendars> / <calendar type=gregorian>. Now it's a sibling of <calendars>. Which I guess means that the fields apply to every calendar, not just the gregorian one.
The weird thing though is that we only have <fields> data for English, so it's useless for your app to pull this info out of the gregorian.js file. I.E. no matter what the user's language setting is, you are still going to get "Year" as the value of that field, so you might as well just hardcode it. Maybe there will be translations in a future version of CLDR.
comment:6 Changed 8 years ago by
Milestone: | tbd → 1.9 |
---|---|
Owner: | changed from Adam Peller to bill |
Status: | new → assigned |
Actually, that last paragraph was wrong, I'm seeing field data in many languages.
comment:8 Changed 8 years ago by
Interesting. The $locale is an artifact of the loader (something generated at runtime, probably added since doug generated the output. see #15144) and probably shouldn't be in the script output. Shouldn't have any impact either way, so not that important. I guess we'd have to modify the script to exclude this particular property.
comment:12 Changed 8 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
Builds are failing with missing locale errors.
error(354) Root bundle indicates localized bundles that don't exist. Root: dojo/cldr/nls/roc; missing bundles: 'ca'.
error(354) Root bundle indicates localized bundles that don't exist. Root: dojo/cldr/nls/indian; missing bundles: 'fr'.
error(354) Root bundle indicates localized bundles that don't exist. Root: dojo/cldr/nls/coptic; missing bundles: 'fr','ro'.
error(354) Root bundle indicates localized bundles that don't exist. Root: dojo/cldr/nls/ethiopic; missing bundles: 'fr','ro'.
It looks like the LDML schema changed, and the fields tag is now a peer of calendar, which makes sense. Unfortunately, it means fixing our xslt...