Remove unnecessary use of Services.locale.appLocalesAsBCP47 from address book files
Categories
(Thunderbird :: Address Book, defect)
Tracking
(thunderbird_esr102 wontfix)
Tracking | Status | |
---|---|---|
thunderbird_esr102 | --- | wontfix |
People
(Reporter: peter.schlemihl, Assigned: peter.schlemihl)
Details
Attachments
(2 files)
5.21 KB,
patch
|
darktrojan
:
review+
|
Details | Diff | Splinter Review |
1.13 KB,
patch
|
darktrojan
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/109.0.0.0 Safari/537.36
Steps to reproduce:
The use is not necessary or even undesired, see bug 1758739.
Most other code uses undefined
as the first argument:
https://searchfox.org/comm-central/search?q=Services.intl.DateTimeFormat&path=&case=false®exp=false
Looks like
https://searchfox.org/comm-central/rev/05c024a0e3e23602b52eac638565a2545a9d2ee8/mail/components/compose/content/MsgComposeCommands.js#7112
should also be changed and perhaps also
https://searchfox.org/comm-central/rev/05c024a0e3e23602b52eac638565a2545a9d2ee8/mail/components/customizableui/content/panelUI.js#883
Updated•2 years ago
|
Comment 3•2 years ago
|
||
Comment on attachment 9316301 [details] [diff] [review]
1815419.patch
Yeah, I think you're right.
Updated•2 years ago
|
Updated•2 years ago
|
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/0fbc2a2ae92d
Remove unnecessary use of Services.locale.appLocalesAsBCP47 from address book files. r=darktrojan
https://hg.mozilla.org/comm-central/rev/e501bc90a85e
Remove unnecessary use of Services.locale.appLocaleAsBCP47 from compose. r=darktrojan
Description
•