New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CalDAVBackend::createSubscription ignores a lot of properties #24469

Closed
georgehrke opened this Issue May 6, 2016 · 3 comments

Comments

Projects
None yet
3 participants
@georgehrke
Contributor

georgehrke commented May 6, 2016

https://github.com/owncloud/core/blob/master/apps/dav/lib/caldav/caldavbackend.php#L1063-1070

This code only inserts the hardcoded values. Properties like displayname, color, order, stripAlerts and stripAttachments are simply ignored without further notice.

createCalendar inserts all supported properties: https://github.com/owncloud/core/blob/master/apps/dav/lib/caldav/caldavbackend.php#L378-380

@georgehrke

This comment has been minimized.

Show comment
Hide comment
@georgehrke
Contributor

georgehrke commented May 6, 2016

@DeepDiver1975 DeepDiver1975 self-assigned this May 6, 2016

@DeepDiver1975 DeepDiver1975 added this to the 9.1-current milestone May 6, 2016

tcitworld added a commit to tcitworld/core that referenced this issue Jun 22, 2016

tcitworld added a commit to tcitworld/core that referenced this issue Jun 30, 2016

@PVince81

This comment has been minimized.

Show comment
Hide comment
@PVince81

PVince81 Jun 30, 2016

Member

@georgehrke I set the severity to medium, let me know if it is worse

Member

PVince81 commented Jun 30, 2016

@georgehrke I set the severity to medium, let me know if it is worse

@PVince81 PVince81 modified the milestones: 9.1.1-next-maintenance, 9.1-current Jun 30, 2016

@georgehrke

This comment has been minimized.

Show comment
Hide comment
@georgehrke

georgehrke Jun 30, 2016

Contributor

Medium is fine :)

Contributor

georgehrke commented Jun 30, 2016

Medium is fine :)

tcitworld added a commit to tcitworld/core that referenced this issue Jul 1, 2016

DeepDiver1975 added a commit that referenced this issue Jul 1, 2016

DeepDiver1975 added a commit that referenced this issue Jul 21, 2016

DeepDiver1975 added a commit that referenced this issue Jul 21, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment