Page MenuHomekolab.org

Fix duplicated events in other users calendar if its subfolders are active (#5340)
ClosedPublic

Authored by machniak on Mar 2 2016, 4:47 PM.

Diff Detail

Repository
rRPK roundcubemail-plugins-kolab
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

machniak updated this revision to Diff 193.Mar 2 2016, 4:47 PM
machniak retitled this revision from to Fix duplicated events in other users calendar if its subfolders are active (#5340).
vanmeeuwen added a subscriber: vanmeeuwen.

Set the reviewers to the developer group please.

machniak updated this revision to Diff 194.Mar 4 2016, 10:57 AM
  • Fix matching recurring tasks with today/tomorrow/later filter (#4205)
machniak updated this revision to Diff 195.Mar 4 2016, 10:59 AM
  • Fix matching recurring tasks with today/tomorrow/later filter (#4205)
  • Revert "Fix matching recurring tasks with today/tomorrow/later filter (#4205)"
machniak updated this revision to Diff 196.Mar 4 2016, 11:04 AM

Cleanup

vanmeeuwen accepted this revision.Mar 6 2016, 9:09 AM
vanmeeuwen added a reviewer: vanmeeuwen.

This patch also applies to roundcubemail-plugins-kolab-3.2.

This revision is now accepted and ready to land.Mar 6 2016, 9:09 AM

From what I can see this could indeed work around the inconsistency described in #5340. It might still show some (temporary) duplicates in the calendar view until the user calendar is fully refreshed. Deciding when and what to reload should better be done on the client.

plugins/calendar/calendar_ui.js
2970

This is really an ugly hack. I wasn't aware that refetchEvents has issues. Should maybe be fixed independent to this.

This revision was automatically updated to reflect the committed changes.