Bug 312927

Summary:	kontact summary sorts events by some unknown order; should sort by time
Product:	[Applications] kontact	Reporter:	Mathias Homann <Mathias.Homann>
Component:	summary	Assignee:	kdepim bugs <kdepim-bugs>
Status:	RESOLVED FIXED
Severity:	normal	CC:	amaury, h6zb8-kdebugs20120801, kdenis, montel, rubenvanos, stakanov.s, tom-kde.bugs
Priority:	NOR
Version:	5.4.3
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://commits.kde.org/korganizer/fc3c1d073c9c2881fb8dbe0423da695b39dc5d3c	Version Fixed In:	master
Sentry Crash Report:
Attachments:	Digital Clock Plasmoid Kontakt screenshot of the bug (with inversion of the task order on the most recent day as an example of the problem) Still present in 5.0.2

Description Mathias Homann 2013-01-09 09:40:10 UTC

I have the contact summary show upcoming events, and it seems to sort them by calendar...
I think the upcoming events piece should sort all events just by time.

Reproducible: Always

Steps to Reproduce:
1. use more than one calendar
2. make sure they all have more than one event for the same day
3. observe the weird sorting order in kontact summary
Actual Results:  
events should simply be sorted by time

Expected Results:  
events seem to be sorted by calendar first, and *then* by time

Comment 1 Laurent Montel 2013-10-07 11:27:22 UTC

I confirm it.

Comment 2 rubenvanos 2014-09-04 20:45:41 UTC

Created attachment 88568 [details]
Digital Clock Plasmoid

Comment 3 rubenvanos 2014-09-04 20:46:02 UTC

Created attachment 88569 [details]
Kontakt

Comment 4 rubenvanos 2014-09-04 20:46:28 UTC

This bug happens to me as well, both on KDE 4.8.5 and KDE 4.12.5. It doesn't only involve Kontact, though, the Digital Clock Plasmoid features the same incorrect behaviour in its calendar, rendering it practically useless. Only KOrganizer seems to do it right.

See attached (censored) screenshots of a single calendar being sorted incorrectly in the Digital Clock Plasmoid and Kontact.

Comment 5 rubenvanos 2014-09-04 20:48:35 UTC

(In reply to rubenvanos from comment #4)
> This bug happens to me as well, both on KDE 4.8.5 and KDE 4.12.5. It doesn't
> only involve Kontact, though, the Digital Clock Plasmoid features the same
> incorrect behaviour in its calendar, rendering it practically useless. Only
> KOrganizer seems to do it right.
> 
> See attached (censored) screenshots of a single calendar being sorted
> incorrectly in the Digital Clock Plasmoid and Kontact.

Edit: I only just realized Kontakt seems to sort correctly. My apologies. It seemed incorrect when I looked over it several times to make sure. The Digital Clock Plasmoid does sort incorrectly, but that is probably another bug.

Comment 6 stakanov.s 2014-11-06 09:53:15 UTC

Created attachment 89473 [details]
screenshot of the bug (with inversion of the task order on the most recent day as an example of the problem)

The problem is the reversed order of items PER day. 
The order of subsequent day tasks is fine.

Comment 7 stakanov.s 2014-11-06 09:53:35 UTC

I confirm this bug for Kontakt 4.14.2 from standard repos openSUSE 13.2
In the attachment you can see what was also the case for the others:
the events of the respective day are ordered from the most last to the first while the order should be the first to come then followed in the timeline by the others. The events of the days coming up after are correctly lined up from the first coming to the last. Logic wants that the event coming right next is the one to have the attention and not the last event of the day. 
So the problem seems to the order of the events of the current day in the summary. This order is actually inverted. 
The order of subsequent days is fine. 
See screen-shot.

Comment 8 Tom Mittelstädt 2015-10-29 08:28:23 UTC

Created attachment 95200 [details]
Still present in 5.0.2

the bug is still present in 5.0.2
should be confirmed

Comment 9 Carioca 2016-01-02 11:20:42 UTC

Here is the same problem.

Comment 10 Amaury Carrade 2016-01-03 15:45:16 UTC

*** Bug 325020 has been marked as a duplicate of this bug. ***

Comment 11 Denis Kurz 2017-06-23 20:41:17 UTC

This bug has never been confirmed for a KDE PIM version that is based on KDE Frameworks, except possibly a Technology Preview version 5.0.x. Those Framework-based versions differ significantly from the old 4.x series. Therefore, I plan to close it in around two or three months. In the meantime, it is set to WAITINGFORINFO to give reporters the opportunity to check if it is still valid. As soon as someone confirms it for a recent version (at least 5.1, ideally even more recent), I'll gladly reopen it.

Please understand that we lack the manpower to triage bugs reported for versions almost two years beyond their end of life.

Comment 12 Tom Mittelstädt 2017-06-24 13:20:29 UTC

Still present in kontact 5.4.3.

Sort order seems to be something like:

Today       16:00
Today       09:00
Tomorrow    15:00
Tomorrow    10:00

Comment 13 Laurent Montel 2017-06-24 14:00:36 UTC

IT's sorted from A to Z
Ok I need to look at it.

Comment 14 Carioca 2017-06-24 19:28:51 UTC

As Tom Mittelstädt wrote.

Comment 15 Allen Winter 2017-09-22 21:52:30 UTC

Git commit fc3c1d073c9c2881fb8dbe0423da695b39dc5d3c by Allen Winter.
Committed on 22/09/2017 at 21:52.
Pushed by winterz into branch 'master'.

src/kontactplugin/korganizer/summaryeventinfo.cpp - fix sorting

Differential Revision: https://phabricator.kde.org/D7931
FIXED-IN: master

M  +4    -4    src/kontactplugin/korganizer/summaryeventinfo.cpp

https://commits.kde.org/korganizer/fc3c1d073c9c2881fb8dbe0423da695b39dc5d3c