[Kde-pim] Review Request 111149: Add file monitoring to ICalDir and VCardDir resources and move common code to shared base class

Dan Vrátil dvratil at redhat.com
Tue Jul 9 13:31:51 BST 2013


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

(Updated July 9, 2013, 2:31 p.m.)


Status
------

This change has been discarded.


Review request for KDEPIM.


Description
-------

Similarily to iCal and vCard resources, the iCalDir and vCardDir resources are essentially identical. This patch moves all the code to a template-based DirResource class, making VCardDirResource and ICalDirResource specializations of the DirResource class, implementing only loading and saving of payload from/to file. QObject-related stuff is implemented in DirResourceBase class.

The shared code now also supports watching the directory (via KDirWatch) for changes and will automatically add, update or remove entries from Akonadi when a file is added, changed or removed in the resource's data directory.


This addresses bug 321343.
    http://bugs.kde.org/show_bug.cgi?id=321343


Diffs
-----

  resources/icaldir/CMakeLists.txt 3ca6c5e 
  resources/icaldir/icaldirresource.h 8fe7733 
  resources/icaldir/icaldirresource.cpp b294be0 
  resources/shared/dirresource.h PRE-CREATION 
  resources/shared/dirresourcebase.h PRE-CREATION 
  resources/shared/dirresourcebase.cpp PRE-CREATION 
  resources/vcarddir/CMakeLists.txt 7a58e6f 
  resources/vcarddir/vcarddirresource.h 1423bdf 
  resources/vcarddir/vcarddirresource.cpp 5d17d26 

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


Testing
-------

Set up vCardDir resource to watch for /tmp/vcarddir. Moved a new .vcard file into the folder and it was immediately added to Akonadi. Changing the file updated payload stored in Akonadi and removing the file removed respective item from Akonadi. Same for the iCalDir.


Thanks,

Dan Vrátil

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


More information about the kde-pim mailing list