Bug 100247

Summary: Sorting by date in search results dialog is broken
Product: [Applications] kmail Reporter: Hasso Tepper <hasso>
Component: generalAssignee: kdepim bugs <kdepim-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: andres.karner, kollix, sergio.cinos
Priority: NOR    
Version: 1.8   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In:
Attachments: Screenshot to illustrate 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. ***