Closed
Bug 1344141
Opened 7 years ago
Closed 7 years ago
Nightly l10n build for ja-JP returns locale string that's not "ja-JP"
Categories
(Core :: Internationalization, defect)
Tracking
()
RESOLVED
FIXED
mozilla55
People
(Reporter: scottwu, Assigned: zbraniecki)
Details
Attachments
(1 file)
59 bytes,
text/x-review-board-request
|
jfkthame
:
review+
gchang
:
approval-mozilla-aurora+
|
Details |
When I tested the date picker UI on nightly builds for different locales, the locale string returned from the JP version for mac[0] is "ja-JP-mac" with chrome-registry[1], or "ja-JP-x-lvariant-mac" with localeservice[2]. This causes an invalid language tag error if used with getCalendarInfo (bug 1287503). [0] https://ftp.mozilla.org/pub/firefox/nightly/latest-mozilla-central-l10n/firefox-54.0a1.ja-JP-mac.mac.dmg [1] Components.classes["@mozilla.org/chrome/chrome-registry;1"].getService(Ci.nsIXULChromeRegistry).getSelectedLocale("global"); [2] Components.classes["@mozilla.org/intl/localeservice;1"].getService(Components.interfaces.mozILocaleService).getAppLocale();
Updated•7 years ago
|
Component: JavaScript: Internationalization API → Internationalization
Comment 1•7 years ago
|
||
Yeah, ja-JP-mac is a locale code we use in practice, and sadly it's violating specs. We need special casing in parts where we need to pass our locale code to libraries that validate the locale code.
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → gandalf
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•7 years ago
|
||
Not sure if it's related, but when I built ja-JP-mac langpack and installed it (and switched to ja-JP-mac), the LocaleService is still returning 'en-US' for me. Investigating.
Assignee | ||
Comment 3•7 years ago
|
||
OK, so the langpacks is a different bug, but this is fortunately trivial to fix. We just need not to canonicalized the language from chrome registry. Patch coming.
Comment hidden (mozreview-request) |
Assignee | ||
Comment 5•7 years ago
|
||
And the langpack bug moved to bug 1344355.
Comment 6•7 years ago
|
||
mozreview-review |
Comment on attachment 8843473 [details] Bug 1344141 - Do not BCP47 canonicalize languages from ChromeRegistry in LocaleService. https://reviewboard.mozilla.org/r/117206/#review119300 I guess this is fine for now. I'd be really happy if we eventually found ourselves using canonical locale tags everywhere, but that's a bigger issue...
Attachment #8843473 -
Flags: review?(jfkthame) → review+
Pushed by zbraniecki@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e3c420b74eb9 Do not BCP47 canonicalize languages from ChromeRegistry in LocaleService. r=jfkthame
Comment 8•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/e3c420b74eb9
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Assignee | ||
Comment 9•7 years ago
|
||
Comment on attachment 8843473 [details] Bug 1344141 - Do not BCP47 canonicalize languages from ChromeRegistry in LocaleService. Approval Request Comment [Feature/Bug causing the regression]: bug 1332207 [User impact if declined]: some date/times will be formatted in en-US when using ja-JP-mac locale [Is this code covered by automated tests?]: nope [Has the fix been verified in Nightly?]: yes [Needs manual test from QE? If yes, steps to reproduce]: no [List of other uplifts needed for the feature/fix]: none [Is the change risky?]: no [Why is the change risky/not risky?]: because it's a single boolean flag change that has been tested on nightly and at most affects one locale [String changes made/needed]: none
Attachment #8843473 -
Flags: approval-mozilla-aurora?
Comment 10•7 years ago
|
||
Comment on attachment 8843473 [details] Bug 1344141 - Do not BCP47 canonicalize languages from ChromeRegistry in LocaleService. Fix a date/times format with ja-JP-mac locale issue. Aurora54+.
Attachment #8843473 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Updated•7 years ago
|
Keywords: checkin-needed
Comment 11•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/200a60e3f77c
Keywords: checkin-needed
You need to log in
before you can comment on or make changes to this bug.
Description
•