nextcloud / calendar

📆 Calendar app for Nextcloud
https://apps.nextcloud.com/apps/calendar
GNU Affero General Public License v3.0
981 stars 241 forks source link

Appointment expansion does not take into account recurrence exceptions #5537

Open bed7 opened 1 year ago

bed7 commented 1 year ago

Steps to reproduce

  1. Create periodic events.
  2. Create appointments in these slots.
  3. Change the time or duration of a recurring event over a week
  4. Preview appointments

Expected behavior

Appointment slots become available when a weekly event is modified (time or duration)

Actual behaviour

Free slots are not updated when a weekly event is modified for a shorter duration or moved. You must delete the event and create a new one to obtain the expected result. Looks like the weekly repeat of the event was still there in the calendar.

Calendar app version

4.5.2

CalDAV-clients used

Thunderbird Lightning / IOs calendar

Browser

No response

Client operating system

windows/ios

Server operating system

Arch Linux

Web server

Apache

Database engine version

MySQL

PHP engine version

PHP 8.1

Nextcloud version

27.1.2

Updated from an older installed version or fresh install

None

List of activated apps

No response

Nextcloud configuration

No response

Web server error log

No response

Log file

No response

Browser log

No response

Additional info

No response

miaulalala commented 1 year ago

To make sure I understand the issue:

Create a config Config creates slots Slots are booked The booked event is modified manually -> The free slots on the booking page should take into account having more time to fit in slots but don't

Is that correct?

bed7 commented 1 year ago

yes but I'm talking about a weekly event present in the calendar. And I add that an event over the day or over several days does not block the slots

miaulalala commented 1 year ago

So it's a recurring event and you modify one in the series?

bed7 commented 1 year ago

that's it

miaulalala commented 9 months ago

Related PR: https://github.com/nextcloud/server/pull/40541

miaulalala commented 2 months ago

So, I tested this with a DAILY recurrence rule where I modified the second event of the recurrence. I could not reproduce the issue.

Can you please share the VEVENT that you created, with the recurrence @bed7 ? You can export your event in the three- dot menu. Please use the first event in the series to do so.

Thanks!

miaulalala commented 2 months ago

Recurrence:

image

Slots for the 26th:

image

Slots for the 27th:

image