969903 - Improve thread pane display name handling

Status: RESOLVED FIXED

(Thunderbird :: Folder and Message Lists, defect)

Description

[Jim Porter (:squib)]

Attachment: display-name.patch

This patch does three things to the From/Recipients columns in the thread pane:

1) Properly strips angle brackets when the from name is of the form <[e-mail]>
2) Add a space between multiple names in the recipients column
3) Adds much more thorough Mozmill tests for this code

I didn't add tests for the address book lookup for display names, but I could do that too.

Comment 1

[Joshua Cranmer [:jcranmer]]

Comment on attachment 8372869 [details] [diff] [review]
display-name.patch

Review of attachment 8372869 [details] [diff] [review]:
-----------------------------------------------------------------

::: mail/test/mozmill/folder-display/test-display-name.js
@@ +32,5 @@
> +  },
> +  { name: "from_email_in_angle_brackets",
> +    headers: { From: "<[e-mail]>" },
> +    expected: { column: "from", value: "[e-mail]" },
> +  },

I'd also include tests for empty From headers and ones with multiple addresses. Actually, come to think of it, you probably ought to have one with a From and a Sender and one with no From but a Sender as well.

@@ +58,5 @@
> +  { name: "to_display_name_multiple",
> +    headers: { To: "Carter Burke <[e-mail]>, " +
> +                   "Dwayne Hicks <[e-mail]>" },
> +    expected: { column: "recipients", value: "Carter Burke, Dwayne Hicks" },
> +  },

A part of me wants there to be tests with group addresses in here. But the other part reminds me that our handling of groups right now is complete and utter crap.

Comment 2

[Jim Porter (:squib)]

Attachment: Add more tests

I've added some more tests, testing address book contacts, mixing From/Sender, and mixing To/Cc. This should cover pretty much everything except groups.

Comment 3

[Jim Porter (:squib)]

Attachment: Add more tests, part 2

Even more tests! Now there are tests for missing From/To headers, empty From/To headers, and invalid From/To headers.

Comment 4

[Joshua Cranmer [:jcranmer]]

Comment on attachment 8373039 [details] [diff] [review]
Add more tests, part 2

Review of attachment 8373039 [details] [diff] [review]:
-----------------------------------------------------------------

This is the kind of tests I like seeing. Especially helpful in knowing that I'm not going to regress code in one of my major rewrites.

Comment 6

[Jim Porter (:squib)]

Landed: https://hg.mozilla.org/comm-central/rev/c7fc213e0bcf