--===============5126068872668864882== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112962/ ----------------------------------------------------------- Review request for Dolphin. Description ------- I just noticed that KFileItemModel::removeExpandedItems() fails to remove expanded items that are currently filtered, which can make expanded items re-appear incorrectly after switching from Details to Icons View. The fix is quite straightforward - just iterate over the filtered items and remove those which have a parent. This addresses bug 325344. http://bugs.kde.org/show_bug.cgi?id=325344 Diffs ----- dolphin/src/kitemviews/kfileitemmodel.cpp b6b6ee0 dolphin/src/tests/kfileitemmodeltest.cpp 391fe5b Diff: http://git.reviewboard.kde.org/r/112962/diff/ Testing ------- Fixes the bug for me. Old and new unit tests pass. Thanks, Frank Reininghaus --===============5126068872668864882== Content-Type: text/html; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit

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

Review request for Dolphin.

By Frank Reininghaus.

Description

I just noticed that KFileItemModel::removeExpandedItems() fails to remove expanded items that are currently filtered, which can make expanded items re-appear incorrectly after switching from Details to Icons View.

The fix is quite straightforward - just iterate over the filtered items and remove those which have a parent.

Testing

Fixes the bug for me. Old and new unit tests pass.

Bugs: 325344

Diffs

dolphin/src/kitemviews/kfileitemmodel.cpp (b6b6ee0)
dolphin/src/tests/kfileitemmodeltest.cpp (391fe5b)

--===============5126068872668864882==--