[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-pim
Subject: Re: [Kde-pim] Review Request 116877: Make ItemFetchJob work for in Item whose remote ID is specified
From: "Kevin Krammer" <krammer () kde ! org>
Date: 2014-03-19 10:19:47
Message-ID: 20140319101947.20796.23254 () probe ! kde ! org
[Download RAW message or body]
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/116877/#review53408
-----------------------------------------------------------
Looks good to me but I would feel safer if someone else would also have a look and \
confirm the assessment :)
- Kevin Krammer
On March 19, 2014, 10:09 a.m., David Jarvie wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/116877/
> -----------------------------------------------------------
>
> (Updated March 19, 2014, 10:09 a.m.)
>
>
> Review request for KDEPIM-Libraries.
>
>
> Repository: kdepimlibs
>
>
> Description
> -------
>
> Currently when an Item with only remote ID specified is supplied to the \
> ItemFetchJob constructor, calling setCollection() to set its context does not \
> (contrary to the apidox) work. In fact, setCollection() only works when fetching \
> whole Collections, for which a separate constructor is provided. This patch makes \
> setCollection(), and fetching by remote ID, work the way they are supposed to.
>
> Diffs
> -----
>
> akonadi/itemfetchjob.cpp 93168c3
>
> Diff: https://git.reviewboard.kde.org/r/116877/diff/
>
>
> Testing
> -------
>
> Tested with new KAlarm code.
>
>
> Thanks,
>
> David Jarvie
>
>
_______________________________________________
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