D17111: Do not sort twice when changing role and order at the same time

[Thomas Surrel]

thsurrel updated this revision to Diff 46276.
thsurrel added a comment.


  Fixes as per elvisangelaccio comments
  Thanks for the review!

REPOSITORY
  R318 Dolphin

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D17111?vs=46041&id=46276

BRANCH
  arc_sort_optim (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D17111

AFFECTED FILES
  src/kitemviews/kfileitemmodel.cpp
  src/kitemviews/kfileitemmodel.h
  src/kitemviews/kitemmodelbase.cpp
  src/kitemviews/kitemmodelbase.h
  src/kitemviews/private/kitemlistheaderwidget.cpp

To: thsurrel, #dolphin, elvisangelaccio
Cc: elvisangelaccio, kfm-devel, alexde, sourabhboss, feverfew, spoorun, navarromorales, firef, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20181126/71175254/attachment.htm>