Closed Bug 453301 Opened 15 years ago Closed 15 years ago

Provide a function to collect only a single address.

Categories

(MailNews Core :: Address Book, defect)

Product:
MailNews Core
Component:
Address Book
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
Thunderbird 3.0a3

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(1 file)

The fix
12.04 KB, patch
Bienvenu
: review+
Bienvenu
: superreview+
Details | Diff | Splinter Review
Attached patch The fix — — Splinter Review 
Currently we have nsIAbAddressCollector::collectAddress. This function is designed to collect multiple addresses with the format:

name2 <email1>, name2 <email2>...

For the inline add/edit code, I want to add just one address to the collected address book, especially as I already have the display name and the email separate.

I was going to rename collectAddress to collectAddresses and create a new function, but I decided on just creating a new collectSingleAddress function to make it easier for extensions.

The collectSingleAddress function is a subset of the collectAddress function, and is now called from the collectAddress function.
Attachment #336483 - Flags: superreview?(bienvenu)
Attachment #336483 - Flags: review?(bienvenu)
Flags: wanted-thunderbird3+
Attachment #336483 - Flags: superreview?(bienvenu)
Attachment #336483 - Flags: superreview+
Attachment #336483 - Flags: review?(bienvenu)
Attachment #336483 - Flags: review+
Checked in, changeset id: 248:bfa635b28c65
Status: NEW → RESOLVED
Closed: 15 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.