GNOME Bugzilla – Bug 504062
Incorrect message list sorting: current view by sender
Last modified: 2008-03-11 10:21:14 UTC
Please describe the problem: Sorting the main message view by sender returns a wrong sorting. It makes no difference to click on the Subject header or to choose it from Main Menu->Current View->By Sender. Steps to reproduce: 1. Select message sorting by sender from Main Menu->Current View->By Sender 2. 3. Actual results: Message sorting is wrong. Expected results: Message sorting should be by sender field. Does this happen every time? Yes. Other information:
Created attachment 101128 [details] Screenshot Screenshot of message view wrongly orderer by sender.
please click on the "From" column header.
I checked both "From" and "Date" columns sorting and it returns the expected result. I checked it clicking on the header and also following the menu.
Confirmed here also
Is it possible that this is only Fedora issue? In other words, which distro are you using?
I don't know if it's a Fedora issue but I'm suspecting of it because I also opened another bug at the Fedora Bugzilla server and they just prompted me to check a new Evolution version, namely evolution-2.12.3-1.fc8. BTW, this is the link to the filled bug: https://bugzilla.redhat.com/show_bug.cgi?id=425980 I'll post any news from this new Evo version, if any.
BTW, that was me, whom pointed to that version. We had there some fix for other thing which touches related part of Evolution and I removed that patch in the version you mentioned above. I only wonder if Baptiste Mille-Mathias has also Fedora or not. (And of course if the patch I removed really caused this issue).
Hey there, No, I'm using Hardy, ubuntu development version, and from I'm seing now, the bug is still valid
Hmm, it works for me on Evo from trunk (I guess they use this). Can you check if they have any patch touching evolution/mail/message-list.c:get_normalised_string function?
I just tried evolution-2.12.3-1.fc8 and can confirm that the bug is still there :(
Created attachment 103630 [details] [review] proposed evo patch for evolution; I guess it worked for me because of some coincidence with used strings. The problem here was the "from/to" normalized strings has been compared as addresses, but the comparison was actually made on collate keys, which is obviously wrong. The thing with normalized subject was that there has been done comparisons based on collate, but again on collate keys, not the strings. Both fixed and works for me now.
Milan, seems fine to me. Commit to trunk.
Patch committed to SVN trunk as r34911 (http://svn.gnome.org/viewvc/evolution?view=revision&revision=34911) so.. if indeed the addresses are not strhash'ed, the patch would fix the issue.. but I can't seem to figure out the logic behind using 'COL_*_NORM' for 'from/to' if they aren't actually normalised.. misnomer.. maybe even misleading..
Bumping version to a stable release.
Closing as fixed. From/To "normalized" strings are not normalized, that's right, but they are stored in a e_poolv, so it consumes less memory. It's a bit confusing with terminology, but should be fine, I guess.