[Kde-pim] Review Request 118103: tag modifications (e.g. rename) are not propagated through TagModelPrivate::monitoredTagChanged()

[Dan Vrátil]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118103/#review57843
-----------------------------------------------------------

Ship it!


Ship It!

- Dan Vrátil


On May 13, 2014, 10:18 a.m., Jonathan Marten wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118103/
> -----------------------------------------------------------
> 
> (Updated May 13, 2014, 10:18 a.m.)
> 
> 
> Review request for KDEPIM-Libraries.
> 
> 
> Repository: kdepimlibs
> 
> 
> Description
> -------
> 
> I've been trying to implement a tag renaming operation, using an Akonadi::TagModifyJob.  This appears to work, and the change propagates to the Akonadi::TagModel where TagModelPrivate::monitoredTagChanged() gets called as intended.
> 
> There appears to be a problem here, though, in the usual case where the tag is not being reparented.  The mTags hash is updated, but the mChildTags list for the parent is not.  This means that TagModel::data(), which fetches the tag from the mChildTags hash using tagForIndex(), still returns the old data value and the change appears to have no effect.
> 
> 
> Diffs
> -----
> 
>   akonadi/tagmodel_p.cpp 0f6d5bb 
> 
> Diff: https://git.reviewboard.kde.org/r/118103/diff/
> 
> 
> Testing
> -------
> 
> Built kdepimlibs with this change, tested tag modification - the change is propagated correctly.
> 
> 
> Thanks,
> 
> Jonathan Marten
> 
>

_______________________________________________
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/