Closed Bug 452614 Opened 11 years ago Closed 11 years ago

nsIAbCard needs a method of searching all email addresses (well both of them)

Categories

(MailNews Core :: Address Book, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3.0a3

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(1 file)

Attached patch The fixSplinter Review
I'm doing the inline edit card dialog, and as part of that work for processing updates to the address book, I need to be checking if an email address matches the primary or secondary email address in the card.

I can do this via two lots of getProperty and various to lower case and equals in JS, but that isn't ideal, especially when we could just do it on the card itself.

Therefore I'm proposing that we have nsIAbCard::hasEmailAddress, this is potentially a temporary function until we sort out proper multi-valued attributes, but it makes this work easier, and we already have at least one other place we can use it.

Attaching patch with test case.
Attachment #335880 - Flags: superreview?(bienvenu)
Attachment #335880 - Flags: review?(bienvenu)
Flags: wanted-thunderbird3+
Attachment #335880 - Flags: superreview?(bienvenu)
Attachment #335880 - Flags: superreview+
Attachment #335880 - Flags: review?(bienvenu)
Attachment #335880 - Flags: review+
Checked in, changeset id: 211:8e89694ee82f
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.