[Kde-pim] Review Request: Put new items from resource sync into proper childEntities collection when using InvisibleCollectionFetch

[Kevin Krammer]

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


Looks in line with similar checks earlier on.
Do the ETM unit tests still pass with this change? Do you think you could come up with an additional test that fails without this and succeeds with it? (so we avoid introducing a regression later on)

- Kevin Krammer


On Oct. 31, 2012, 2:59 p.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107153/
> -----------------------------------------------------------
> 
> (Updated Oct. 31, 2012, 2:59 p.m.)
> 
> 
> Review request for KDEPIM-Libraries and Stephen Kelly.
> 
> 
> Description
> -------
> 
> When using ETM with InvisibleCollectionFetch collection fetch strategy, new added items from resource syncing ends up in wrong collection and just empty data are displayed in the view. Debugging this I discovered that the newly created items must be stored with "m_rootCollection.id()" as key instead of "collection.id()" (m_childEntities[collection.id()] is actually empty/non-existant even though the model has valid items from this collection) as the model is actually just a flat list.
> 
> 
> Diffs
> -----
> 
>   akonadi/entitytreemodel_p.cpp b1c8378 
> 
> Diff: http://git.reviewboard.kde.org/r/107153/diff/
> 
> 
> Testing
> -------
> 
> Tested my SocialFeed model, which uses InvisibleCollectionFetch, this fixes showing new items in the plasmoid.
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/