14447 - [FEATURE] [HAB] we should add any email we send a message TO into our HAB.

Status: VERIFIED FIXED

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

Description

[(not reading, please use [e-mail] instead)]

right now we add any email we receive

we should also any email address we send to.

Comment 1

[hangas]

This is correct, we should add email addresses for all recipients we send
messages to.  Sending bug to David because I think he wrote this code.

Comment 2

[David :Bienvenu]

I only wrote the address capture when reading messages code. I think it's not
nearly as useful to capture address you send to, since you don't have the user
name, but I can try to figure out where the sending code happens (I've never
touched that code before). cc'ing people who might have a clue about message
sending.

Comment 3

[David :Bienvenu]

fixed in compose\src\nsMsgSend.cpp

Comment 4

[(not reading, please use [e-mail] instead)]

excellent!

Comment 5

[David :Bienvenu]

the curmudgeon in me knows that the second people discover that misaddressed
addresses still go in the history AB, I'm going to get lots of bugs filed on
that, but there's nothing I can do about it.

Comment 6

[(not reading, please use [e-mail] instead)]

help wanted bug:

when you get a bounced email, be fancy and pop an alert that says:  "this looks
like a bounce.  do you want to remove address <foobar@cheese> from your address
books?"

Comment 7

[lchiang]

Another help wanted bug should be:  if the recipient you send to is already in
your personal address book, then perhaps that person shouldn't get added to the
hab?

Comment 8

[esther]

Using 19991028 builds on win98, mac and linux this is fixed, any name that is
addressed using To: is entered in the HB.