1712601 - Add a method to retrieve mailing lists to nsIAbManager

Status: RESOLVED FIXED

(Thunderbird :: Address Book, enhancement)

Description

[Lasana Murray]

For bug Bug 1705100 we want to be able to detect mailing lists in the public address fields and include their member addresses in the threshold count. nsIAbManager provides a method to check if a mailing lists exists by name but no explicit method for retrieving it. This bug adds such a method.

Comment 1

[Lasana Murray]

After spending some time on this I realized that the AddrBookMailingList class does not seem to support nested mailing lists. Looking at the database schema, that information seems to be lost as the nested mailing list is stored both as a list and a card separately. I'm not sure how much of this area is a priority.

https://searchfox.org/comm-central/rev/0ad407d04042e5801490dc054d65d4bb97a2c0e8/mailnews/addrbook/modules/AddrBookMailingList.jsm#62-79

Comment 2

[Lasana Murray]

Attachment: Bug 1712601 - Add getMailListFromName() to nsIAbManager. r?darktrojan

Comment 3

[Geoff Lankow (:darktrojan)]

(In reply to Lasana Murray from comment #1)

After spending some time on this I realized that the AddrBookMailingList class does not seem to support nested mailing lists. Looking at the database schema, that information seems to be lost as the nested mailing list is stored both as a list and a card separately. I'm not sure how much of this area is a priority.

https://searchfox.org/comm-central/rev/0ad407d04042e5801490dc054d65d4bb97a2c0e8/mailnews/addrbook/modules/AddrBookMailingList.jsm#62-79

Personally, I really don't care for nested lists, but they're technically possible to create and nsMsgCompose will expand them if they exist (there's even a unit test!). No UI or API supports them, and I'm not sure I've ever seen a complaint about that, so I wouldn't waste your time.

Comment 4

[Lasana Murray]

(In reply to Geoff Lankow (:darktrojan) from comment #3)

(In reply to Lasana Murray from comment #1)

After spending some time on this I realized that the AddrBookMailingList class does not seem to support nested mailing lists. Looking at the database schema, that information seems to be lost as the nested mailing list is stored both as a list and a card separately. I'm not sure how much of this area is a priority.

https://searchfox.org/comm-central/rev/0ad407d04042e5801490dc054d65d4bb97a2c0e8/mailnews/addrbook/modules/AddrBookMailingList.jsm#62-79

Personally, I really don't care for nested lists, but they're technically possible to create and nsMsgCompose will expand them if they exist (there's even a unit test!). No UI or API supports them, and I'm not sure I've ever seen a complaint about that, so I wouldn't waste your time.

Understood, however seems to me like you can create a nested list by adding the list name to an existing list.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/9a62ee7400de
Add getMailListFromName() to nsIAbManager. r=darktrojan