Thunderbird 5 does not sort address columns correctly if showing only display name

RESOLVED FIXED in Thunderbird 8.0

Status

Thunderbird
Folder and Message Lists
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Avner Falk, Assigned: Bienvenu)

Tracking

(Depends on: 1 bug, {regression})

Thunderbird 8.0
All
Other
regression
Dependency tree / graph
Bug Flags:
in-testsuite +

Thunderbird Tracking Flags

(thunderbird7+ fixed)

Details

(Whiteboard: [has patch and unit test for review])

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

6 years ago
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:5.0) Gecko/20100101 Firefox/5.0
Build ID: 20110615151330

Steps to reproduce:

Click on the From column header


Actual results:

The messages were not sorted alphabetically by the sender's name


Expected results:

The messages should be sorted alphabetically by the sender's name
(Assignee)

Comment 1

6 years ago
Are you showing only the display name, per options/preferences, advanced, general? We sort by the full sender address, not the display name.
(Reporter)

Comment 2

6 years ago
The From column displays the sender's name, not his e-mail address. All previous versions of Thunderbird sorted the From column by the display name. Why would TB5 have to be any different in this respect? This is a real bug and needs to be fixed rather than trying to find excuses for it.
(Assignee)

Comment 3

6 years ago
It would be helpful if you could answer my question. It's unfortunate that you interpreted a question attempting to diagnose your problem as an excuse. But I'll keep trying. If you turn off "show only display name", do you see the correct sorting? TB 5 did indeed change what is displayed in the thread pane, but what we sort on did not change (that's this bug) - see https://bugzilla.mozilla.org/show_bug.cgi?id=243631
(Assignee)

Updated

6 years ago
Summary: Thunderbird 5 does not sort messages properly → Thunderbird 5 does not sort address columns correctly if showing only display name
(Reporter)

Comment 4

6 years ago
It's unfortunate that we are arguing instead of trying to solve the problem together. In your previous message you wrote me, "Are you showing only the display name, per options/preferences, advanced, general?" There is no such option under the Tools > Options > Advanced > General tab. How can I turn off the "show only display name" option? If you want me to use the Configuration Editor, you should say so and tell me which settings to change and how. How can I answer your question when it is phrased in a way which makes it impossible to understand?
(Assignee)

Comment 5

6 years ago
tools, options, advanced, reading & display.

Boying, I looked into what it's going to take to fix this - unforutnately, I think some of the collation key generation code is going to need to be lifted into the view code, in particular, nsMsgDBView is going to need an equivalent of nsMsgDatabase::RowCellColumnToCollationKey() that takes the display name string instead of a column directly from the database.
(Reporter)

Comment 6

6 years ago
Dear Boying and David, if you really want to save Thunderbird's position as the best e-mail client available, then you will do what it takes to fix this bug (as well as Bug 672023, the absence of the message-download-progress bar, which has erroneously been marked as a duplicate of an earlier bug). Once you have done this, I shall be delighted to reinstall TB5 on my computer and to recommend it to the thousands of people in my TB Address Book. Thank you very much.
(Assignee)

Updated

6 years ago
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: wanted-thunderbird?
Keywords: regression

Comment 7

6 years ago
David

I'm using TB 5 on a Mac 10.6.5.  Sorting on "from" works fine for me.  In the Tools menu dropdow however, I do not an "options" option.  I have:  saved files, add-ons, activity manager, message filters, run filters on folder, run filters on msg, run junk mail controls, delete marked mail as junk, import, error console, account settings.

but as I said, i'm not experiencing this bug.
(Assignee)

Comment 8

6 years ago
Mitchell, it's preferences on the mac (tools, options, on windows, and Avner is using windows, according to the bug). We use different menus to conform to standard app behavior on the OS, but it makes it awkward to tell users how to get to options/preferences.

Comment 9

6 years ago
Yup, figured that!
(Reporter)

Comment 10

6 years ago
Mitchell, this bug definitely exists in the Windows version of TB5 and many users have complained about it on various sites (e.g. http://forums.mozillazine.org/viewtopic.php?f=29&t=2210501&start=15). I'd love to see your people fix this bug ASAP. Thanks, Avner
(Assignee)

Comment 11

6 years ago
Created attachment 548053 [details] [diff] [review]
proposed fix

this will fix the sort issues, though it may cause a pretty big performance hit the first time you sort by sender, and thereafter if you change an address book card. Before this can get reviewed, I need a unit test for it.
Assignee: nobody → dbienvenu
Status: NEW → ASSIGNED
(Reporter)

Comment 12

6 years ago
Thanks for including me on this code-change proposition. I am not a programmer. However, if you send me a beta version of TB 5.1 I shall be happy to test it.
(Assignee)

Comment 13

6 years ago
Created attachment 548803 [details] [diff] [review]
fix with unit test

I noticed that we also weren't sorting by recipients correctly, and so I added both kinds of sort to a unit test.
Attachment #548053 - Attachment is obsolete: true
Attachment #548803 - Flags: review?(mbanner)
(Reporter)

Comment 14

6 years ago
When you have the next version kindly send me a download link. Thanks.
tracking-thunderbird7: --- → +
Flags: wanted-thunderbird?
(Assignee)

Updated

6 years ago
Whiteboard: [has patch and unit test for review]
Comment on attachment 548803 [details] [diff] [review]
fix with unit test

>-        rv = msgHdr->GetRecipientsCollationKey(len, result);
>         rv = msgHdr->GetAuthorCollationKey(len, result);

We should be able to remove these functions on trunk, unless you've got a specific reason for keeping them.

I'm a little concerned by the performance issues, presumably there's no easy way to batch these keys/changes up when we download messages (we could cover that in a follow-up bug).

I think this is a reasonable first step for getting in for 7 and seeing what feedback we get.
Attachment #548803 - Flags: review?(mbanner)
Attachment #548803 - Flags: review+
Attachment #548803 - Flags: approval-comm-aurora+
(Assignee)

Comment 16

6 years ago
(In reply to Mark Banner (:standard8) from comment #15)
> Comment on attachment 548803 [details] [diff] [review]
> fix with unit test
> 
> >-        rv = msgHdr->GetRecipientsCollationKey(len, result);
> >         rv = msgHdr->GetAuthorCollationKey(len, result);
> 
> We should be able to remove these functions on trunk, unless you've got a
> specific reason for keeping them.

I don't know if any extensions are using these. But I can file a bug to remove them.

> 
> I'm a little concerned by the performance issues, presumably there's no easy
> way to batch these keys/changes up when we download messages (we could cover
> that in a follow-up bug).

Since we cache the result of figuring out the display name in the .msf file, I'm not sure what you would batch...once the display name is cached, the new code performs very much like the old code.
> 
> I think this is a reasonable first step for getting in for 7 and seeing what
> feedback we get.
Checked into trunk: http://hg.mozilla.org/comm-central/rev/18242a1afda0
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 8.0
(Reporter)

Comment 18

6 years ago
Congratulations. Is your new version compatible with the European language spelling dictionaries and with BiDi Mail UI? Where can I download it? Thanks.
Checked into aurora: http://hg.mozilla.org/releases/comm-aurora/rev/47fc904950ff
status-thunderbird7: --- → fixed
(Reporter)

Comment 20

6 years ago
Even with the latest Beta of TB7, the sorting of messages in a given folder by the sender's name takes an inordinate amount of time (from 5 to 30 seconds, depending on the number of messages in the folder). In TB3 it takes less than a second. This is a serious problem which will prevent users from upgrading to TB7 or will make them revert to TB3. Therefore this bug cannot be marked as fixed before it is resolved.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Please file a new bug for other issues so they can be tracked properly.
Status: REOPENED → RESOLVED
Last Resolved: 6 years ago6 years ago
Resolution: --- → FIXED
Depends on: 690707
Depends on: 695327

Updated

6 years ago
Duplicate of this bug: 650627
You need to log in before you can comment on or make changes to this bug.