Review Request: Drop InvisibleCollectionFetch Fetch Strategy in Favor of KDescendantsProxyModel

[David Narváez]

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

Review request for Plasma and Christian Mollekopf.


Description
-------

As the InvisibleCollectionFetch fetch strategy is currently broken, I implemented a suggestion by Christian Mollekopf of working around the issues by flatting the model through a KDescendantsProxyModel to fix crashes in Plasma related to calendars.


This addresses bug 279322.
    http://bugs.kde.org/show_bug.cgi?id=279322


Diffs
-----

  plasma/generic/dataengines/calendar/calendarengine.cpp 53f98c0 

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


Testing
-------

1. Add an iCal resource from the internet
2. Refresh the iCal resource
3. Refresh the iCal resource again if Plasma didn't crash on step 2

Withouth this patch, Plasma crashes when trying to remove an item that has no collection in the Calendar Model embedded in the Dataengine.

You should also be able to replicate bug 279322 without this patch and this patch should fix the situation described there.


Thanks,

David Narváez

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20111110/4aaf02b8/attachment.html>