D13866: Fix ResultModel item moving

2018-07-03 Thread David Edmundson
This revision was automatically updated to reflect the committed changes. Closed by commit R159:75d6103df944: Fix ResultModel item moving (authored by davidedmundson). REPOSITORY R159 KActivities Statistics CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D13866?vs=37100&id=37106 REVIS

D13866: Fix ResultModel item moving

2018-07-03 Thread Eike Hein
hein accepted this revision. hein added a comment. This revision is now accepted and ready to land. It works for me. REPOSITORY R159 KActivities Statistics BRANCH master REVISION DETAIL https://phabricator.kde.org/D13866 To: davidedmundson, ivan, hein Cc: hein, kde-frameworks-devel, m

D13866: Fix ResultModel item moving

2018-07-03 Thread David Edmundson
davidedmundson added a comment. In an email > Without being able to test, I can only rely on the fact that the reasoning sounds ok and that it is authored by David 5. > Thanks, you have my blessing to push. > Ivan So will do so, unless someone objects soon. REPOSITORY

D13866: Fix ResultModel item moving

2018-07-03 Thread Anthony Fieroni
anthonyfieroni added a reviewer: ivan. REPOSITORY R159 KActivities Statistics REVISION DETAIL https://phabricator.kde.org/D13866 To: davidedmundson, ivan Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D13866: Fix ResultModel item moving

2018-07-03 Thread David Edmundson
davidedmundson created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: kde-frameworks-devel. davidedmundson requested review of this revision. REVISION SUMMARY The intention of this code is to find the new insertion index in the cac