Closed Bug 1870014 Opened 5 months ago Closed 5 months ago

Port bug 1869051 - Remove unused strings from passwordmgr.properties

Categories

(Thunderbird :: Upstream Synchronization, task)

Tracking

(thunderbird_esr115 unaffected)

RESOLVED FIXED
122 Branch
Tracking Status
thunderbird_esr115 --- unaffected

People

(Reporter: mkmelin, Assigned: mkmelin)

References

Details

Attachments

(1 file)

Bug 1869051 removed some strings that Thunderbird had borrowed (well, kept using after toolkit dropped some of the functionality).

This is causing

[task 2023-12-14T11:00:24.064Z] 11:00:24     INFO - TEST-PASS | comm/mail/components/addrbook/test/browser/browser_cardDAV_init.js | Correct status message - "carddav-loading" == "carddav-loading" - 
[task 2023-12-14T11:00:24.065Z] 11:00:24     INFO - Console message: [JavaScript Error: "NS_ERROR_FAILURE: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIStringBundle.GetStringFromName]" {file: "resource:///modules/CardDAVUtils.jsm" line: 648}]
[task 2023-12-14T11:00:24.065Z] 11:00:24     INFO - promptAuth@resource:///modules/CardDAVUtils.jsm:648:10

But I think some other places may need change as well. https://searchfox.org/comm-central/search?q=passwordmgr.properties&path=mail&case=false&regexp=false

Summary: Remove unused strings from passwordmgr.properties → Port bug 1869051 - Remove unused strings from passwordmgr.properties
Assignee: nobody → mkmelin+mozilla

@mkmelin I can ask for this patch to be backed out again if the strings and translations are still in use in Thunderbird. Would that help?

Flags: needinfo?(mkmelin+mozilla)

Thanks. We shouldn't really re-use string that are not "ours" so I think it's best to fix that properly. Seems like it's just one string.

Flags: needinfo?(mkmelin+mozilla)
Status: NEW → ASSIGNED
Target Milestone: --- → 122 Branch
Duplicate of this bug: 1870071

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/aa1bf7240c08
Port bug 1869051 - use our own string for "Remember password". r=aleca

Status: ASSIGNED → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Regressions: 1870724
See Also: → 1872559
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: