[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-pim
Subject: [Kde-pim] Re: Review Request: Fix ETM for monitoring multiple
From: "Torgny Nyblom" <kde () nyblom ! org>
Date: 2011-07-19 10:45:46
Message-ID: 20110719104546.19281.67954 () vidsolbach ! de
[Download RAW message or body]
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101209/#review4850
-----------------------------------------------------------
Any progress?
- Torgny
On April 23, 2011, 2:21 p.m., Christian Mollekopf wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101209/
> -----------------------------------------------------------
>
> (Updated April 23, 2011, 2:21 p.m.)
>
>
> Review request for KDEPIM and Stephen Kelly.
>
>
> Summary
> -------
>
> With this patch, the ETM properly supports monitoring multiple collections.
> While currently Collection::root() will be watched, as soon as multiple collections \
> are watched, with this patch only the needed ancestor collections will be included \
> (up to Collection::root(), without any items/collections in it).
> I added the insertAndFetchCollection function, which checks for necessary parents \
> and fetches them if needed.
>
> Diffs
> -----
>
> akonadi/entitytreemodel.h 881593102feef69a73fefc6c26822efdf2a97966
> akonadi/entitytreemodel_p.h b1f9c1e47496e08a269b031f40aab4d5defc8529
> akonadi/entitytreemodel_p.cpp e367cd79122054a3a195a76ec51de24060daa69b
>
> Diff: http://git.reviewboard.kde.org/r/101209/diff
>
>
> Testing
> -------
>
> works for me
>
>
> Thanks,
>
> Christian
>
>
_______________________________________________
KDE PIM mailing list kde-pim@kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic