Bug 100247 - Sorting by date in search results dialog is broken
Summary: Sorting by date in search results dialog is broken
Status: RESOLVED FIXED
Alias: None
Product: kmail
Classification: Applications
Component: general (show other bugs)
Version: 1.8
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
: 96994 102087 102291 (view as bug list)
Depends on:
Blocks:
 
Reported: 2005-02-25 13:48 UTC by Hasso Tepper
Modified: 2007-09-14 12:17 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Screenshot to illustrate bug. (135.05 KB, image/png)
2005-02-25 13:49 UTC, Hasso Tepper
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Hasso Tepper 2005-02-25 13:48:55 UTC
Version:           1.8 (using KDE 3.4.0, compiled sources)
Compiler:          gcc version 3.3.5 (Debian 1:3.3.5-8)
OS:                Linux (i686) release 2.6.10

It isn't sorting by date actually, but alphabetically which breaks sorting completely for me. See attached screenshot for illustration.
Comment 1 Hasso Tepper 2005-02-25 13:49:40 UTC
Created attachment 9829 [details]
Screenshot to illustrate bug.
Comment 2 Hasso Tepper 2005-02-25 14:01:15 UTC
Commit which broke it - http://lists.kde.org/?l=kde-cvs&m=110511548007484&w=2
Comment 3 Don Sanders 2005-03-16 09:53:15 UTC
CVS commit by sanders: 

Revert commit "Show localized instead of ISO date in search result list"
http://lists.kde.org/?l=kde-cvs&m=110511548007484&w=2
which introduced "Sorting by date in search results dialog is broken"
http://bugs.kde.org/show_bug.cgi?id=100247

--Volker_Krause_Cookies;
++Hasso_Tepper_Cookies;

CCMAIL:100247-done@bugs.kde.org

Volker if you read this then firstly let me say hi, thanks for making an
effort to contribute to KMail, secondly please be more careful and check 
that you don't introduce any (obvious) regressions like this one when 
committing code to KMail.


  M +1 -1      kmfldsearch.cpp   1.142


--- kdepim/kmail/kmfldsearch.cpp  #1.141:1.142
@@ -496,5 +496,5 @@ void KMFldSearch::slotAddMsg(int idx)
 
     (void)new KListViewItem(mLbxMatches, mLbxMatches->lastItem(),
-                            msg->subject(), from, msg->dateStr(),
+                            msg->subject(), from, msg->dateIsoStr(),
                             fName,
                             QString::number(mFolder->serNum(idx)));
Comment 4 Tom Albers 2005-03-16 10:01:03 UTC
*** Bug 96994 has been marked as a duplicate of this bug. ***
Comment 5 Tom Albers 2005-03-23 19:12:09 UTC
*** Bug 102087 has been marked as a duplicate of this bug. ***
Comment 6 Tom Albers 2005-03-23 19:12:51 UTC
*** Bug 102291 has been marked as a duplicate of this bug. ***