Comment 104 for bug 369150

Revision history for this message
In , Martin-wilck (martin-wilck) wrote :

Created an attachment (id=319890)
an attempt to fix this problem

This patch fixes handling of addresses of the form 'Spät, Karl <email address hidden>' for me. Please give it a try. The Patch is against HEAD (May 07, 2008).

Rationale:

In many parts of the MailNews sources, MIME headers containing RFC 2822 address lists are treated by decoding MIME first, then expanding address lists. This is wrong if the MIME-encoded data contains a comma (MIME encoded data should be treated as an RFC 2822 atom).

This patch changes the behavior such that MIME decoding is done last, immediately before displaying the headers, and in particular, after splitting up the address list.

Individual changes:

 - nsMsgCompFields::SplitRecipients(): run postponed late MIME conversions
 - nsMsgDBView::FetchAuthor(): fixes message list display
 - nsMsgCompose::CreateMessage(): postpone MIME conversion for some fields (for "Reply")
 - QuotingOutputStreamListener():: postpone more MIME conversions (for "Reply")
 - QuotingOutputStreamListener::OnStopRequest(): dito
 - CreateCompositionFields(): dito, for "Edit as New"
 - MimeHeaders_write_all_headers(): dito, for message display window