Bug 207220 – app-portage/elogviewer doesn't sort the entries by year

• Product: Portage Development
• Status: RESOLVED FIXED

[Description ekz 2008-01-23 21:08:07 UTC]

elogviewer entries are sorted in a day rather than a year, because of the syntax of date (dd/mm/yyyy) (see attached image)

elogviewer version: 0.5.2

Reproducible: Always

Steps to Reproduce:
1. run elogviewer
2. click on 'Time' column, to sort by date


Actual Results:  
The dates are sorted per day

Example (dd/mm/yy):

06/12/07
07/01/08
07/08/07


Expected Results:  
The dates are sorted per year and month

Example (dd/mm/yy):

07/08/07
06/12/07
07/01/08

[Comment 1 ekz 2008-01-23 21:09:47 UTC]

Created attachment 141677 [details]
Bad-Sorted entries

[Comment 2 Christian Faulhammer (RETIRED) 2008-01-23 21:38:38 UTC]

adding author...Mathias, please have a look.

[Comment 3 Jouni Kosonen 2008-01-23 22:10:19 UTC]

Created attachment 141684 [details, diff]
Separation of localized tstamp display from the sort key

I think that the solution is something like this: show the LC_TIME formatted timestamp but sort by the YYYY-MM-DD hh:mm:ss formatted version.

I may have made a slight mistake somewhere in there, but with this modification the sorting works with fi_FI, ja_JP, en_US and POSIX locales as expected.

[Comment 4 ekz 2008-01-23 23:00:07 UTC]

(In reply to comment #3)
> Created an attachment (id=141684) [edit]

The patch works OK, thanks

[Comment 5 Christian Faulhammer (RETIRED) 2008-01-24 08:20:21 UTC]

(In reply to comment #4)
> (In reply to comment #3)
> > Created an attachment (id=141684) [edit]
> 
> The patch works OK, thanks

 From me, too. :)  My Python skills are somewhat crippled, so if you are bored, Jouni, you can have a look at bug 195429, which bothers me a bit.  Committed as -r1.  Thanks everyone.