Closed Bug 7028 Opened 25 years ago Closed 25 years ago

Sort should be sorted by text, not by the From field

Categories

(MailNews Core :: Backend, defect, P3)

defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: fenella, Assigned: Bienvenu)

Details

RE: 5/24 builds on win_nt 4.0, win98(1999-05-24-08 m6), and Linux (1999-05-24-08 m6) Steps: 1. Open Messenger from the Task menu 2. Sort by View|Sort or by clicking on the Sender column, sort sorts by the From field Expected result: should be sorted by the text field so that users can visually see the sort effect. Have not tested Mac yet.
Assignee: phil → putterman
I'm taking this.
Target Milestone: M7
QA Contact: 4080 → 4104
Target Milestone: M7
This bug report is broken out from bug http://bugzilla.mozilla.org/show_bug.cgi?id=4731 per putterman's request. Scott - is this yours then?
<I hate updating bugs at around the same time. Ignore my question.>
Status: NEW → ASSIGNED
Target Milestone: M8
Setting to M8. David, this is one we talked about a while ago. Basically, I'm getting the author collation key, but I'm displaying the sender's name using the RFC822 parser. So it get's sorted by the from header which looks strange when the sender's name is what's being displayed. We talked about having the the parsing done in the DB code. Then we could have the collation key based on the sender's name. Scott
Target Milestone: M8 → M9
Moving to M9.
Assignee: putterman → bienvenu
Status: ASSIGNED → NEW
I think I have to do something here.
Status: NEW → ASSIGNED
Target Milestone: M9 → M10
I'm not sure I'm going to get to this before m10
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
I think this is fixed in nsMsgDatabase.cpp, by getting the sender from mime, and then generating the collation key. I think libmime also deals with mime decoding, so I don't do that anymore.
Status: RESOLVED → VERIFIED
Linux (1999-08-10-08 M9) Win32 (1999-08-10-09 M9) and Mac (1999-08-10-09 M9) The problem no longer exists.
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.