User:Mikey/project2

From CDOT Wiki
< User:Mikey
Revision as of 21:29, 22 November 2009 by Mikey (talk | contribs) (Project Contributor(s))
Jump to: navigation, search

Project Name

Bug 286760 - Ending a mail address with a space in address book displays it wrong when composing

Project Description

This project involves fixing a problem with saved email addresses ending with trailing spaces in the address book causing sent messages to be lost. For example, the user will enter a new contact entry with an email address followed by a space, such as "test@example.com " (minus the quotes) and saves it. In the compose window, they will select the email address that they just saved and send their message, but this message will not be delivered to the recipient.

Project Leader(s)

Michael Dennis

Project Contributor(s)

Nobody yet : Test out my patch on Thunderbird & SeaMonkey. 

Project Details

After running the test with additional test cases, I found that the trailing spaces given in the email address are not discarded when the email address is concatenated with the full name of the contact. For example, if the name of the contact is John Doe and the email address of the contact is "jdoe@example.com " (minus the quotes)(take note of the trailing space), the result would give "John Doe <jdoe@example.com >" (minus the quotes)(including the trailing space).

My take on this is to remove the trailing spaces before the result is returned from the msg_quote_phrase_or_addr(...). I plan on using the macro, TRIM_WHITESPACE(...), to do this.

Project News

0.2 Release: In the end, I did not use the TRIM_WHITESPACE() macro, because of my small understanding of how it works. Just a little bit of code is added to the msg_quote_phrase_or_addr() function. This bit of code removes trailing spaces from the address given. However, I might be at fault, because this code might not be allowed due to some RFC regulations.