[Kde-pim] Review Request: add AgentInstance::setOption method, to set an option via DBus

[Volker Krause]

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


Patch looks mostly ok to me, still discussing the approach with Kevin though.


/trunk/KDE/kdepimlibs/akonadi/agentinstance.cpp
<http://reviewboard.kde.org/r/734/#comment741>

    I would suggest to keep the reconfigure() call out of setOption(). That way you can change multiple settings in one block before the resource reacts to them.



/trunk/KDE/kdepimlibs/akonadi/firstrun.cpp
<http://reviewboard.kde.org/r/734/#comment740>

    Looks like some of the error handling got lost here.


- Volker


On 2009-05-21 23:45:01, Constantin Berzan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/734/
> -----------------------------------------------------------
> 
> (Updated 2009-05-21 23:45:01)
> 
> 
> Review request for KDE PIM.
> 
> 
> Summary
> -------
> 
> This makes it possible to configure an agent / resource via DBus without having to go through the trouble of creating a QDBusInterface.  The code existed before in Firstrun, an internal class in kdepimlibs/akonadi.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdepimlibs/akonadi/agentinstance.h 970429 
>   /trunk/KDE/kdepimlibs/akonadi/agentinstance.cpp 970429 
>   /trunk/KDE/kdepimlibs/akonadi/agentinstance_p.h 970429 
>   /trunk/KDE/kdepimlibs/akonadi/firstrun.cpp 970429 
>   /trunk/KDE/kdepimlibs/akonadi/firstrun_p.h 970429 
> 
> Diff: http://reviewboard.kde.org/r/734/diff
> 
> 
> Testing
> -------
> 
> * maildir resource creation and configuration from the outboxinterface lib in playground/pim (not commited to svn yet)
> * maildir resource creation and configuration via Firstrun, by putting a default file in ~/kde/share/apps/akonadi/firstrun.  Settings of type QString and bool have been tested.
> 
> 
> Thanks,
> 
> Constantin
> 
>

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