Difference between revisions of "User:Mikey/project3"

From CDOT Wiki
Jump to: navigation, search
(Created page with '== Project Name == [https://bugzilla.mozilla.org/show_bug.cgi?id=286760 Bug 286760 - Ending a mail address with a space in address book displays it wrong when composing] == Proj...')
 
(Project Name)
Line 1: Line 1:
 
== Project Name ==
 
== Project Name ==
[https://bugzilla.mozilla.org/show_bug.cgi?id=286760 Bug 286760 - Ending a mail address with a space in address book displays it wrong when composing]
+
[https://bugzilla.mozilla.org/show_bug.cgi?id=528508 Bug 528508 - In folder tabs, indicate unread count in tab title]
  
 
== Project Description ==
 
== Project Description ==

Revision as of 13:13, 7 December 2009

Project Name

Bug 528508 - In folder tabs, indicate unread count in tab title

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.