[Kde-pim] Re: Review Request: Add FreeBusyProvider base classes

[Grégory Oestreicher]

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

(Updated May 12, 2011, 6:33 p.m.)


Review request for KDEPIM-Libraries.


Changes
-------

Updated version that incorporates comments from Kevin and Sergio.


Summary
-------

Following Osnabrück meeting I implemented the next method of free-busy support into KDEPIM. This patch adds the base classes necessary for other changes that will go in kdepim.

In short those classes must be inherited by a resource that provides free-busy information, and add it a D-Bus interface. The resource can then be queried for f-b info.

This have been commented on in the list, and some improvments were made following feedback.


Diffs (updated)
-----

  akonadi/interfaces/org.freedesktop.Akonadi.Resource.FreeBusyProvider.xml PRE-CREATION 
  akonadi/kcal/CMakeLists.txt f53552f 
  akonadi/kcal/freebusyproviderbase.h PRE-CREATION 
  akonadi/kcal/freebusyproviderbase.cpp PRE-CREATION 
  akonadi/kcal/freebusyproviderbase_p.h PRE-CREATION 
  includes/Akonadi/KCal/FreeBusyProviderBase PRE-CREATION 

Diff: http://git.reviewboard.kde.org/r/101296/diff


Testing
-------

All tests were done successfully with the DAV resource (review requests to follow in the next days). The f-b info is correctly fetched.


Thanks,

Grégory

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