Closed Bug 1795751 Opened 2 years ago Closed 2 years ago

Contribute button for addons does not work in 102

Categories

(Thunderbird :: Add-Ons: General, defect)

Thunderbird 102
defect

Tracking

(thunderbird_esr102+ fixed, thunderbird107 fixed)

RESOLVED FIXED
108 Branch
Tracking Status
thunderbird_esr102 + fixed
thunderbird107 --- fixed

People

(Reporter: dskirk, Assigned: TbSync)

Details

Attachments

(3 files)

Steps to reproduce:

Click any addon in Thunderbird that has a 'contribute' button. Click it and nothing happens.

Actual results:

Nothing. It should have opened a web page to allow user to contribute funds to developer.

Expected results:

A web page should have opened that accepted financial contribution.

Component: Untriaged → Add-Ons: General
Attached image contribute.png

I cannot confirm this. Maybe the link at the specific add-on you tested is broken (wrongly configured by the developer)?

The contribute button for TbSync works:
https://addons.thunderbird.net/en-US/thunderbird/addon/tbsync

What add-on have you experienced this with?

I of course tested this in Thunderbird, after having searched for TbSync inside the Add-on Manager.

I tested with These plugins and the Contribute button did nothing:

  • EditEmailSubject MX
  • ImportExportTools
  • Provider for Google Calendar
  • ThunderHTMLedit
  • ToggleHTML
  • Xpunge

Each of those works for me.

Note: The contribute page should be opened in your default browser, not in Thunderbird.
Do you have a default browser configured?
What is your OS?

What happens if you click on the "Add-on home page" button of one of those add-ons?
What happens if you click on the sign-in button in the top right?

I use windows 11, latest Thunderbird, and Firefox is default. When I click a URL in email, the page opens right up. A user on SUMO reported the issue, and when I found it was the same for me, I created this bug report. When I click an addon's home page, the browser opens properly to the page. Sign-in worked properly. Puzzling.

Do you see any error on the console (Tools -> Developer Tools -> Error Console), which are triggered when clicking on the contribute button?

I've never used developer tools, but I did see this:
Uncaught (in promise) TypeError: windowRoot.ownerGlobal.openUILinkIn is not a function
handleEvent chrome://mozapps/content/extensions/aboutaddons.js:3126

Thanks. At least an error message to work with!

But I have no idea where this is called, we do not use it at all (only suite, which is old Seamonkey code):
https://searchfox.org/comm-esr102/search?q=openUILinkIn&path=&case=true&regexp=false

You are using a genuine Thunderbird 102, downloaded from thunderbird.net, right?

I tried 102, Daily and self built TB. All fail with the same error when I try the Contribute button of ThunderHTMLEdit.

Attached image contribute2.png

Got it. You did not click the contribute button on an ATN page viewed inside Thunderbird (which my first screenshot pointed to), but in the details page of an add-on. Correct?

At least I can reproduce the error there now. :-)

Status: UNCONFIRMED → NEW
Ever confirmed: true

Yes, pure Thunderbird. In fact, I just reinstalled and continue to receive the error:
Uncaught (in promise) TypeError: windowRoot.ownerGlobal.openUILinkIn is not a function
handleEvent chrome://mozapps/content/extensions/aboutaddons.js:3126

Assignee: nobody → john
Status: NEW → ASSIGNED
Target Milestone: --- → 108 Branch

Pushed by alessandro@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/d19295ace812
Implement missing openUILinkIn(). r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED

Last broken in bug 1500895.

Comment on attachment 9299239 [details]
Bug 1795751 - Implement missing openUILinkIn(). r=mkmelin

[Triage Comment]
Approved for beta, let's get this on 102 as it could affect addon contributions

[Approval Request Comment]
Regression caused by (bug #):
User impact if declined: Potentially affects addon authors revenue.
Testing completed (on c-c, etc.):
Risk to taking this patch (and alternatives if risky):Low, implements a missing function

TBSync - Any objections to this uplift request?

Attachment #9299239 - Flags: approval-comm-esr102?
Attachment #9299239 - Flags: approval-comm-beta+

Comment on attachment 9299239 [details]
Bug 1795751 - Implement missing openUILinkIn(). r=mkmelin

[Triage Comment]
Approved for esr102

Attachment #9299239 - Flags: approval-comm-esr102? → approval-comm-esr102+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: