[frameworks-kactivities-stats] [Bug 396102] ResultsModel emits wrong rowsMoved transactions

2018-07-03 Thread David Edmundson
https://bugs.kde.org/show_bug.cgi?id=396102

David Edmundson  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
  Latest Commit||https://commits.kde.org/kac
   ||tivities-stats/75d6103df944
   ||c02a7ff4f85f2d2e565d3fdc40b
   ||c
 Resolution|--- |FIXED

--- Comment #4 from David Edmundson  ---
Git commit 75d6103df944c02a7ff4f85f2d2e565d3fdc40bc by David Edmundson.
Committed on 03/07/2018 at 14:22.
Pushed by davidedmundson into branch 'master'.

Fix ResultModel item moving

Summary:
The intention of this code is to find the new insertion index in the
cache where a resource should now be by counting number of items in the
cache with a higher score/update time/whatever.

FixedItems comparison compares two resources by string IDs, checking if
they're user pinned. If so they go on top.

In FullOrdering if neither compared item is pinned it also compares literal
resourceId. Useful if you have no other sorting tests and want to
maintain order, not useful if we want to contintue to compare the score
as in 50% case of the cases we'll fail our lessThan early.

Test Plan:
Ran test app
Changed some scores by clicking in kickoff
They moved appropriately instead of being moved all over the place

Subscribers: kde-frameworks-devel

Tags: #frameworks

Differential Revision: https://phabricator.kde.org/D13866

M  +1-1src/resultmodel.cpp

https://commits.kde.org/kactivities-stats/75d6103df944c02a7ff4f85f2d2e565d3fdc40bc

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kactivities-stats] [Bug 396102] ResultsModel emits wrong rowsMoved transactions

2018-07-02 Thread Ivan Čukić
https://bugs.kde.org/show_bug.cgi?id=396102

--- Comment #3 from Ivan Čukić  ---
I'm currently without my development machine. I'll check this out when it
becomes operational.

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kactivities-stats] [Bug 396102] ResultsModel emits wrong rowsMoved transactions

2018-07-02 Thread Eike Hein
https://bugs.kde.org/show_bug.cgi?id=396102

--- Comment #2 from Eike Hein  ---
I'm currently too busy to attempt a fix in KActivitiesStats, so here's a
workaround for Kicker for now: https://phabricator.kde.org/D13856

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kactivities-stats] [Bug 396102] ResultsModel emits wrong rowsMoved transactions

2018-07-02 Thread Eike Hein
https://bugs.kde.org/show_bug.cgi?id=396102

--- Comment #1 from Eike Hein  ---
Note that KActivitiesStatsTestApp doesn't catch this because it doesn't seem to
use ResultsModel and doesn't do live updating. The query when re-run produces
the right results, so the db is fine; just the model doesn't update correctly.

-- 
You are receiving this mail because:
You are watching all bug changes.