[Tickets #8734] Re: Synchronization of multiple calendars with SyncML

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/8734
------------------------------------------------------------------------------
  Ticket             | 8734
  Updated By         | mephisto at gmx.net
  Summary            | Synchronization of multiple calendars with SyncML
  Queue              | Kronolith
  Version            | 2.3
  Type               | Enhancement
  State              | Feedback
  Priority           | 2. Medium
  Milestone          |
  Patch              | 1
  Owners             | Horde Developers
+New Attachment     | horde3-kronolith-multiple-calendar-sync-version2.diff
------------------------------------------------------------------------------


mephisto at gmx.net (2009-12-03 12:08) wrote:

> It looks like one part is missing: finding the correct calendar out  
> of sync_cals when updating an existing event in _kronolith_replace().
> OTOH please remove the changes to _kronolith_synchronize(), this  
> method is for Kolab.
> Beside those two issues, it's a great patch.

Hello,

as far as I understood, the event returned by
$event = $kronolith_driver->getByUID($uid);
knows which calendar it belongs to, and hence no manual lookup is required.
For me, the patch works, and modified events appear in the correct  
calendar (and are not moved to the default calendar). As the  
synchronization framework only knows about the events returned by  
_kronolith_list, events from calendars not used for synchronization  
should not be modified.
I did not know what _kronolith_synchronize is used for, and in the  
turba/lib/api.php, the _turba_synchronize routine is modified in an  
analogous way - sorry, I  removed the modifications (they did not harm  
in my setup, however).

Best regards