Review Request 109455: Follow-up to Emmanuel's restructurings in KFileItemModel

[Commit Hook]

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

(Updated March 14, 2013, 11:28 p.m.)


Status
------

This change has been marked as submitted.


Review request for Dolphin and Emmanuel Pescosta.


Description
-------

After https://git.reviewboard.kde.org/r/109180/, the main missing piece in the functionality around parent-child relationships that cannot be determined by parsing the URL is the removal or collapsing of expanded folders which have filtered children. This patch fixes it by removing the code that iterates over the filtered items from setExpanded() and slotItemsDeleted() to a new function removeFilteredChildren(), and making this function respect the parent-child relationships.


Diffs
-----

  dolphin/src/kitemviews/kfileitemmodel.h 1939671 
  dolphin/src/kitemviews/kfileitemmodel.cpp 61703e7 
  dolphin/src/tests/kfileitemmodeltest.cpp cd2e8d7 

Diff: http://git.reviewboard.kde.org/r/109455/diff/


Testing
-------

Tried by browsing smb:, where collapsing a folder with filtered children did not work correctly before this commit. That works now; old and new tests pass.


Thanks,

Frank Reininghaus

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20130314/ecc3c709/attachment.htm>