Review Request 115828: Add a workaround for broken sorting

David Edmundson david at davidedmundson.co.uk
Mon Feb 17 12:22:27 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115828/
-----------------------------------------------------------

Review request for Telepathy.


Repository: ktp-common-internals


Description
-------

Add a workaround for broken sorting

There is a bug in QSortFilterProxyModel in which if sort() is called
on a model where all items are filtered it will not start sorting
as it cannot map the proxy column to the source column.

This gets updated whenever there's an insertRows but not from
a dataChanged. With the kpeople cache this means that it's not sorting
as we never do an insertRows.

This bug is fixed in Qt5.2, but not in 4.x


Diffs
-----

  KTp/Models/contacts-filter-model.h a891d6b 
  KTp/Models/contacts-filter-model.cpp df12112 

Diff: https://git.reviewboard.kde.org/r/115828/diff/


Testing
-------


Thanks,

David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20140217/847450dc/attachment.html>


More information about the KDE-Telepathy mailing list