Page MenuHomePhorge

Lookup existing events even when creating a new event
ClosedPublic

Authored by mollekopf on Jul 26 2023, 12:00 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 2, 9:31 AM
Unknown Object (File)
Tue, Feb 13, 3:57 PM
Unknown Object (File)
Tue, Feb 13, 7:48 AM
Unknown Object (File)
Tue, Feb 13, 7:44 AM
Unknown Object (File)
Mon, Feb 12, 4:55 PM
Unknown Object (File)
Mon, Feb 12, 11:00 AM
Unknown Object (File)
Jan 28 2024, 12:28 PM
Unknown Object (File)
Jan 23 2024, 11:51 AM
Subscribers

Details

Summary

If the client doesn't know about an already existing event, we will
otherwise overwrite whatever was there (because the uid and therefor
serverid will be the same). In case of outlooks dummy event
on acceptance that means resetting the attendee status to 0 (after it
was set to accepted via the MeetingResponse).

Diff Detail

Repository
rS syncroton
Branch
dev/mollekopf
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 43682
Build 17471: arc lint + arc unit

Event Timeline

mollekopf created this revision.

I suppose we'll need the same thing for all non mail types.

mollekopf retitled this revision from Lookup existing events even when creating a new event to Lookup existing events/todos/... even when creating a new entry.
mollekopf edited the summary of this revision. (Show Details)

Generalized the lookup to apply to all types

That last change was not useful, it's actually event specific. No other type has a uid in activesync (not even tasks), so I don't suppose there is a method to deduplicate anything.

mollekopf retitled this revision from Lookup existing events/todos/... even when creating a new entry to Lookup existing events even when creating a new event.
mollekopf edited the summary of this revision. (Show Details)

Revert to be event specific

machniak subscribed.
machniak added inline comments.
lib/kolab_sync_data_calendar.php
359

After you removed $event this line needs to change.

This revision now requires changes to proceed.Jul 26 2023, 3:04 PM

Check the $entry instead of the $event

This revision is now accepted and ready to land.Jul 26 2023, 7:21 PM
This revision was automatically updated to reflect the committed changes.