My own address should be removed from Mail-Followup-To: on reply

RESOLVED FIXED in Thunderbird 3.0b1

Status

defect
RESOLVED FIXED
13 years ago
4 years ago

People

(Reporter: pierre-bugzilla, Assigned: mail)

Tracking

unspecified
Thunderbird 3.0b1

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [real request in comment 1] [has patch], URL)

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

13 years ago
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-GB; rv:1.8) Gecko/20060126 Fedora/1.5-5 Firefox/1.5
Build Identifier: version 1.6a1 (20060131)

When the header 'Sender:' is set thunderbirds puts my own email address in To:, not the one in From:.

I.e.

From: john@doe.com
Sender: john@doe.com
To: me@example.com

*reply*

From: me@example.com
To: me@example.com


Reproducible: Always

Steps to Reproduce:
(Reporter)

Comment 1

13 years ago
Sorry, false alarm. The mails in question also have a Mail-Followup-To: header, causing said behaviour.

So instead I have another bug report/feature request: Filter my own address from the Mail-Followup-To: list. (the same way it's filtered from the To:s and Cc:s).
Severity: major → normal
Summary: Replying when Sender: is set causes invalid To: → My own address should be removed from Mail-Followup-To:
QA Contact: message-compose
(Assignee)

Comment 2

11 years ago
Yes, the code that filters own address is applied to the new message's Cc header only, but the Mail-Followup-To addresses are put into the new To header.  I'm preparing a patch...
(Assignee)

Comment 3

11 years ago
Posted patch [patch] bugfix (obsolete) — Splinter Review
attached patch fixes this issue
(Assignee)

Updated

11 years ago
Attachment #328169 - Flags: review?(mscott)
Assignee: mscott → mail
Status: UNCONFIRMED → NEW
Component: Message Compose Window → MailNews: Backend
Ever confirmed: true
OS: Linux → All
Product: Thunderbird → Core
QA Contact: message-compose → backend
Hardware: PC → All
Summary: My own address should be removed from Mail-Followup-To: → My own address should be removed from Mail-Followup-To: on reply
Whiteboard: [real request in comment 1] [has patch]
(Assignee)

Updated

11 years ago
Attachment #328169 - Flags: review?(mscott) → review?(dmose)
Product: Core → MailNews Core
Comment on attachment 328169 [details] [diff] [review]
[patch] bugfix

+              if (type == nsIMsgCompType::ReplyAll && ! mailFollowupTo.IsEmpty())

nit: drop the space after the !

r=me with that fixed.

Would you consider doing unit tests for this bug as well?
Attachment #328169 - Flags: superreview?(bienvenu)
Attachment #328169 - Flags: review?(dmose)
Attachment #328169 - Flags: review+
Flags: in-testsuite?
(Assignee)

Comment 5

11 years ago
Posted patch bugfix v1.0.1Splinter Review
Should conform to Mozilla coding style now.
Attachment #328169 - Attachment is obsolete: true
Attachment #343892 - Flags: superreview?
Attachment #328169 - Flags: superreview?(bienvenu)
Comment on attachment 343892 [details] [diff] [review]
bugfix v1.0.1

Daniel, you either missed adding the email address for David when requesting review or got it wrong - I've fixed it for you.
Attachment #343892 - Flags: superreview? → superreview?(bienvenu)

Comment 7

11 years ago
Comment on attachment 343892 [details] [diff] [review]
bugfix v1.0.1

We're dropping the rv of RemoveDuplicateAddresses, but that's OK...thx for the patch.

Updated

11 years ago
Attachment #343892 - Flags: superreview?(bienvenu) → superreview+
Checked in, changeset id 648:8d3721ac95ed
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.0b1
This FIXED bug is flagged with in‑testsuite?   It would be great if assignee or someone else can clear the flag if a test is not appropriate.  And if appropriate, create a test and plus the flag to finish off the bug.
Flags: in-testsuite?
You need to log in before you can comment on or make changes to this bug.