57021 - Mailing list msg displays multiple copies of addressees.

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Address Book & Contacts, defect, P3)

Description

[laurel]

Using oct16 mn6 branch commercial build mac OS 9.0

After sending a message in seamonkey addressed to a mailing list, the message
shows multiple copies of the address list in the displayed header envelope.

1.  Open address book and add a new mailing list with 3 or 4 names.
2.  Compose a new message, address to the mailing list, type subject and body, send.
3.  Get the message and check the addressee header envelope display.

Result:  the display shows the addressees listed several times (only one copy of
the message is actually received by each addressee).

Expected:  the display should show 3 or four addressees as is the true mailing list.

Comment 1

[laurel]

Also seen on linux rh6.0, assume All platforms.

Comment 2

[laurel]

Esther said this didn't happen for her, but she used a small 3 person mail list
with herself as an addressee.  My instances were at least 4 persons, so we're
thinking this might be a number game, but then most mailing lists are more than
a few people.

Comment 3

[laurel]

Oh, and a reply-all to such a message will only populate the compose window
address block and when sent/received/displayed will only show one copy of the
addressees.

Comment 4

[Suresh]

I have seen this bug sometimes in a different way. If you click on List and 
click edit, the entries will be repeated. 

Comment 5

[laurel]

I tried several times and never saw the duplication in Edit List.

Comment 6

[chuang]

I saw it happened in mailing list when I try to fix bug 54530, but it is with 
the change I made for the bug.  I think I fix what I've seen, hopefully they are 
realted.

Comment 7

[scottputterman]

reassigning to chuang.

Comment 8

[laurel]

*** Bug 57264 has been marked as a duplicate of this bug. ***

Comment 9

[laurel]

I'm seeing MANY MANY copies of the addressees on the mail list in header display
when using nov2 branch build.  I only have about 6 people on the mail list, and
the message display shows about 10-12 copies of the list in the header envelope,
making the expanded header list look really large. EEK! What's going to happen
when someone uses a large mail list ?!

Comment 10

[chuang]

I have a fix for this bug.  I think this is a dup for 57442.  

Comment 11

[chuang]

Attachment: A safe patch for the fix

Comment 12

[chuang]

*** Bug 57442 has been marked as a duplicate of this bug. ***

Comment 13

[laurel]

*** Bug 59756 has been marked as a duplicate of this bug. ***

Comment 14

[chuang]

Attachment: Patch file for fix

Comment 15

[chuang]

Got r=ducarroz.

Comment 16

[David :Bienvenu]

is this code just duplicated, or am I misreading the diff? couldn't we have a
method to do this code, instead of cloning it?

Comment 17

[chuang]

Attachment: Patch using diff -u

Comment 18

[Scott MacGregor]

ditto what David said. How about a little method RemoveDupsFromMailingList?

Comment 19

[chuang]

It's not exactly the same.  One is for nsIAbDirectory and the other is for 
nsIAbCard.

Comment 20

[scottputterman]

marking nsbeta1+ and moving to mozilla0.8

Comment 21

[scottputterman]

Does more work still need to be done on this patch?

Comment 22

[scottputterman]

moving to mozilla0.9

Comment 23

[scottputterman]

marking nsbeta1- and moving to future milestone.  However, if this patch is
valid, let's get it in.

Comment 24

[chuang]

The patch is good and we should really fix this problem.

Comment 25

[David :Bienvenu]

sr=bienvenu

Comment 26

[scottputterman]

Let's get this one in then.

Comment 27

[chuang]

fix checked in

Comment 28

[fenella]

Cannot verify this bug until bug 68874 is fixed.

Comment 29

[fenella]

Cannot verify this bug until bug 68874 is fixed.

Comment 30

[laurel]

Update: I was able to send to a mailing list with mar21 commercial trunk, win98.
I did not see the previous multiple copies problem. 

Comment 31

[fenella]

I was able to send it too on Linux (2001-03-21-08 mtrunk)

Comment 32

[fenella]

Verifying it.