Closed Bug 1612480 Opened 4 years ago Closed 4 years ago

browser.compose.begin* functions should not expand mailing lists

Categories

(Thunderbird :: Add-Ons: Extensions API, defect)

defect
Not set
normal

Tracking

(thunderbird_esr68 fixed)

RESOLVED FIXED
Thunderbird 74.0
Tracking Status
thunderbird_esr68 --- fixed

People

(Reporter: darktrojan, Assigned: darktrojan)

Details

Attachments

(1 file)

You can pass a mailing list to the begin functions of browser.compose. Currently it expands the mailing list and adds all of the members to the message recipients. (Or it would if it were not broken.) This shouldn't happen, just the mailing list should be added to the message recipients.

Attachment #9123786 - Flags: review?(mkmelin+mozilla)
Comment on attachment 9123786 [details] [diff] [review]
1612480-compose-mailing-list-1.diff

Review of attachment 9123786 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM, r=mkmelin
Attachment #9123786 - Flags: review?(mkmelin+mozilla) → review+

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/f66abf60532e
Stop expanding mailing lists when passed to the compose.begin* API functions. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 74.0
Comment on attachment 9123786 [details] [diff] [review]
1612480-compose-mailing-list-1.diff

This should be backported.
Attachment #9123786 - Flags: approval-comm-esr68?
Attachment #9123786 - Flags: approval-comm-beta?
Comment on attachment 9123786 [details] [diff] [review]
1612480-compose-mailing-list-1.diff

Won't do more betas this cycle.
Attachment #9123786 - Flags: approval-comm-beta?
Comment on attachment 9123786 [details] [diff] [review]
1612480-compose-mailing-list-1.diff

Approved for ESR
Attachment #9123786 - Flags: approval-comm-esr68? → approval-comm-esr68+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: