1799368 - Adding duplicate email address in `Edit mailing list` dialog causes temporary ghost duplicate item in the list view, then deleting either duplicate removes both (i.e. the email address now gone from the list).

Status: RESOLVED FIXED

(Thunderbird :: Address Book, defect)

Description

[Thomas D. (:thomas8)]

+++ This bug was initially created as a clone of Bug #699357 +++

Seen on 102.4.2 (64-bit), Win10.

Steps to reproduce:

1. Create new mailing list
2. Right-click the mailing list in directory pane (sic!), Properties (aka edit list).
3. Add mail addresses in the New/Edit mailing list dialog, then confirm with OK (but don't click anything after that before next step):

• [e-mail]
• [e-mail]
• [e-mail]
• [e-mail] <- accidentally add a duplicate email address!

3. Verify the members in the mailing list view, and note duplicate contact items for [e-mail] in the contacts list.
4. Select and delete either one of the duplicate contact items in the mailing list view of contacts.

Actual results:

• both duplicate contact items gone, i.e. [e-mail] completely removed from the mailing list, against users intentions.
• As selecting something else in the directory pane before deletion of the alleged duplicates shows, the duplication is a temporary display artifact which does not represent the actual state of the list, where the duplicates have already been removed. So there's only a single entry in the real list, and deleting either one of the (ghost) duplicate display items will then remove that single entry.

Expected results:

• Don't show ghost duplicates which are no longer existing in the actual list.
• I.e. mailing list view in the contacts list should reflect the actual contacts contained in the mailing list.

Comment 1

[Hartmut Welpmann [:welpy-cw]]

Attachment: Bug 1799368 - Remove duplicate addresses before adding to mailing list. r=#thunderbird-reviewers

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/ddaa801c51f2
Remove duplicate addresses before adding to mailing list. r=mkmelin