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

* Your Email Address
* Spam protection	Enter the letters below: \ /.__ . . .. . >< [__)\| \|__\|\|\/\| / \\| \|___\| \|\| \|
Comment	>> 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 >
Attachment
Watch this ticket