Review Request 111158: Fix KLinkItemSelectionModel handling of QItemSelectionModel::Toggle flag

Stephen Kelly steveire at gmail.com
Mon Jun 24 06:32:32 BST 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111158/#review34939
-----------------------------------------------------------

Ship it!


Ship It!

- Stephen Kelly


On June 21, 2013, 10:45 a.m., Aurélien Gâteau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111158/
> -----------------------------------------------------------
> 
> (Updated June 21, 2013, 10:45 a.m.)
> 
> 
> Review request for kdelibs and Stephen Kelly.
> 
> 
> Description
> -------
> 
> Summarizing the comment block: KLISM::select(QModelIndex,...) was calling QISM::select(QModelIndex,...) but QISM::select(QModelIndex,...) calls QISM::select(QItemSelection,...) which is virtual, so it ends up calling KLISM::select(QItemSelection,...). When using the Toggle selection flag, this ends up toggling the selection twice.
> 
> This was breaking image multi-selection in Gwenview:
> - open an image Gwenview
> - show the thumbnail bar
> - click on the [+] button to select a second image => without the patch, image 2 does not get selected
> 
> With the patch it gets selected and... Gwenview crashes as soon as you deselect image 2, but that is a bug in Gwenview. Fix for it is going to be pushed.
> 
> 
> Diffs
> -----
> 
>   kdeui/itemviews/klinkitemselectionmodel.cpp 3c57c18239580497f659ef343c65be39998c2cb7 
>   kdeui/tests/CMakeLists.txt 235e31e826ab8331f30c6e15c874bdc647d9efac 
>   kdeui/tests/klinkitemselectionmodeltest.h PRE-CREATION 
>   kdeui/tests/klinkitemselectionmodeltest.cpp PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/111158/diff/
> 
> 
> Testing
> -------
> 
> Added a unit-test. Tested in Gwenview.
> 
> 
> Thanks,
> 
> Aurélien Gâteau
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130624/3244492d/attachment.htm>


More information about the kde-core-devel mailing list