Closed Bug 672022 Opened 8 years ago Closed 8 years ago

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

Categories

(Thunderbird :: Folder and Message Lists, defect)

All
Other
defect
Not set

Tracking

(thunderbird7+ fixed)

RESOLVED FIXED
Thunderbird 8.0
Tracking Status
thunderbird7 + fixed

People

(Reporter: groucho43, Assigned: Bienvenu)

References

(Depends on 1 open bug)

Details

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

Attachments

(1 file, 1 obsolete file)

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
Are you showing only the display name, per options/preferences, advanced, general? We sort by the full sender address, not the display name.
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.
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
Summary: Thunderbird 5 does not sort messages properly → Thunderbird 5 does not sort address columns correctly if showing only display name
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?
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.
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.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: wanted-thunderbird?
Keywords: regression
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.
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.
Yup, figured that!
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
Attached patch proposed fix (obsolete) — Splinter Review
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
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.
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)
When you have the next version kindly send me a download link. Thanks.
Flags: wanted-thunderbird?
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+
(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
Closed: 8 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 8.0
Congratulations. Is your new version compatible with the European language spelling dictionaries and with BiDi Mail UI? Where can I download it? Thanks.
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
Closed: 8 years ago8 years ago
Resolution: --- → FIXED
Depends on: 690707
Depends on: 695327
Duplicate of this bug: 650627
You need to log in before you can comment on or make changes to this bug.