nextcloud / nextcloud/calendar
Events from shared calendar with RW permission can no longer be modified if it has attendees
Open
Nobody has claimed this yet.
0. to triage
bug
- Dominant language
- JavaScript
- Stars
- 1.2k
- Forks
- 332
- Avg merge
- 16h 13m
- Merged PRs (30d)
- 137
Description
Steps to reproduce
- Update to a version where #6778 is merged
- Share calendar with modify permission to userB
- Create event and add any attendee
- userB will no longer be able to modify the event
Expected behavior
- userB should still be able to execute modify permissions (no matter whether userB is the organizer or not).
Actual behavior
- User gets warning
You are not allowed to edit this event as an attendee.message origin
Calendar app version
5.5.5
CalDAV-clients used
No response
Browser
No response
Client operating system
No response
Server operating system
No response
Web server
None
Database engine version
None
PHP engine version
None
Nextcloud version
No response
Updated from an older installed version or fresh install
Updated from an older version
List of activated apps
Nextcloud configuration
Web server error log
Log file
Browser log
Additional info
No response
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with src/fullcalendar/interaction/eventDrop.js around line 65 and review the permission check introduced by #6778. Reproduce the shared-calendar case with modify permission, an event attendee, and a non-organizer user. Done means the user can modify the event despite being an attendee, as expected for the shared calendar's modify permission.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100