Closed Bug 1840086 Opened 1 year ago Closed 1 year ago

Add-ons manager broken in Thunderbird

Categories

(Toolkit :: Add-ons Manager, defect, P1)

Firefox 116
Desktop
All
defect

Tracking

()

RESOLVED FIXED
116 Branch
Tracking Status
thunderbird_esr102 --- unaffected
thunderbird_esr115 --- unaffected
firefox-esr102 --- unaffected
firefox-esr115 --- unaffected
firefox114 --- unaffected
firefox115 --- unaffected
firefox116 --- fixed

People

(Reporter: freaktechnik, Assigned: Paenglab)

References

(Regression)

Details

(Keywords: regression, Whiteboard: [stockwell needswork:owner])

Attachments

(2 files)

Attached image screenshot.png (deleted) —

The add-ons manager in Thunderbird (and any other potential consumer of toolkit that doesn't bundle browser/) is broken by the loading of a browser/ ftl from bug 1838041.

Commenting out the load of that ftl fixes the add-on manager (while leaving the one string provided by that file untranslated).

Example logs from opening about:addons in Thunderbird:

Missing resource in locale en-US: browser/components/mozFiveStar.ftl
[fluent] Couldn't find a message: text-action-search-text-box-clear 2 addons
[dom/l10n] Errors during l10n mutation frame. 2 addons
Missing resource in locale en-US: browser/components/mozFiveStar.ftl
[fluent] Couldn't find a message: extension-enabled-heading 2 addons
[fluent] Couldn't find a message: addon-badge-recommended2 2 addons
[fluent] Couldn't find a message: addon-badge-line3 2 addons
[fluent] Couldn't find a message: addon-badge-verified2 2 addons
[fluent] Couldn't find a message: addon-badge-private-browsing-allowed2 2 addons
[fluent] Couldn't find a message: extension-enable-addon-button-label 2 addons
[fluent] Couldn't find a message: add-on-options-button 2 addons
[fluent] Couldn't find a message: addon-options-button 2 addons
[fluent] Couldn't find a message: install-postponed-message 2 addons
[fluent] Couldn't find a message: install-postponed-button 2 addons
[fluent] Couldn't find a message: remove-addon-button 2 addons
[fluent] Couldn't find a message: install-update-button 2 addons
[fluent] Couldn't find a message: preferences-addon-button 2 addons
[fluent] Couldn't find a message: report-addon-button addons
[fluent] Couldn't find a message: manage-addon-button addons
[fluent] Couldn't find a message: report-addon-button addons
[fluent] Couldn't find a message: manage-addon-button addons
[fluent] Couldn't find a message: list-empty-get-extensions-message 2 addons
Uncaught (in promise) undefined 2
[fluent] Couldn't find a message: atn-addons-heading-search-input 2 addons
[fluent] Couldn't find a message: extension-heading 2 addons
[dom/l10n] Errors during l10n mutation frame. 2 addons

The easiest fix would probably be to move mozFiveStar.ftl to be part of toolkit.

Set release status flags based on info from the regressing bug 1838041

:niklas, since you are the author of the regressor, bug 1838041, could you take a look? Also, could you set the severity field?

For more information, please visit BugBot documentation.

Flags: needinfo?(nbaumgardner)
Assignee: nobody → elizabeth
Status: NEW → ASSIGNED

Assigning myself to create a temporary fix so that Thunderbird does not have so many failures.

Yes, if the author of bug 1838041 could take this on or give guidance about how we could fix this, that would help a lot. Thank you.

I was not able to figure this out on my own. Unassigning.

Assignee: elizabeth → nobody
Status: ASSIGNED → NEW

The ftl file should be migrated to toolkit, I suppose to https://searchfox.org/mozilla-central/source/toolkit/locales/en-US/toolkit

Add r=#fluent-reviewers to the review.

Assignee: nobody → richard.marti
Status: NEW → ASSIGNED

This should not be "fixed" by introducing a Thunderbird-specific file, but by moving the ftl from browser/ to toolkit/ as mentioned in comment 4.

Flags: needinfo?(richard.marti)

Yes, I know. Because I haven't seen any activity to move it I created this patch. When the move happens we can this patch back out again.

Flags: needinfo?(richard.marti)

(In reply to Richard Marti (:Paenglab) from comment #8)

Yes, I know. Because I haven't seen any activity to move it I created this patch. When the move happens we can this patch back out again.

Thank you, Richard.

Yes, this will get fixed inside m-c/mozilla-central.

It will help to have a good option until that happens. Thank you. I will look the patch over.

Keywords: leave-open
Attachment #9340832 - Attachment description: Bug 1840086 - Port bug 1838041 - Add moz-five-star reuseable component. r=#thunderbird-reviewers → Bug 1840086 - Port Bug 1838041 - Add moz-five-star reuseable component. r=#thunderbird-reviewers
Attachment #9340832 - Attachment description: Bug 1840086 - Port Bug 1838041 - Add moz-five-star reuseable component. r=#thunderbird-reviewers → Bug 1840086 - Port bug 1838041: Add moz-five-star reuseable component. r=#thunderbird-reviewers

Pushed by elizabeth@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/1669c2903caa
Port bug 1838041: Add moz-five-star reuseable component. r=elizabeth

Richard's patch is a temporary fix until m-c is updated. This patch does resolve the many failing tests in c-c caused by changes in bug 1838041.

(In reply to Elizabeth Mitchell from comment #11)

Richard's patch is a temporary fix until m-c is updated. This patch does resolve the many failing tests in c-c caused by changes in bug 1838041.

What bug covers fixing m-c if not this one (which is now assigned to Richard)?

Flags: needinfo?(richard.marti)
Flags: needinfo?(elizabeth)
Depends on: 1840410

I'm going to close this bug and file a follow-up bug so that it is obvious what issue is being tracked.

  • This bug: unblocking Thunderbird.
  • bug 1840410: fix the m-c issue (and revert the work-around from this patch).
Severity: -- → N/A
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Flags: needinfo?(richard.marti)
Flags: needinfo?(nbaumgardner)
Flags: needinfo?(elizabeth)
Keywords: leave-open
Priority: -- → P1
Resolution: --- → FIXED
Target Milestone: --- → 116 Branch

Set release status flags based on info from the regressing bug 1838041

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: