Wrong date creation based on destination.one repeatUntil #97

Merged

d-s-codappix merged 3 commits from 12653-v4 into support-v4.x 2026-04-14 06:29:07 +00:00

Conversation 1 Commits 3 Files changed 4 +130 -3

d-s-codappix commented 2026-04-01 13:49:04 +00:00 (Migrated from github.com)

Copy link

Respect the last day, if we import on that day.

A new test is added to reflect that situation.
The workaround for missing DatePeriod::INCLUDE_END_DATE is extended to cover that situation.

• Fix wrong determination of repeat minutes

  The used value was for month instead of minutes.
  We now use the correct identifier for minutes.

  This revealed that tests only passed because of the issue.
  We therefore fix the code to still pass correct tests by working around PHP < 8.2 limitation.

• Add test to ensure that weekly basis includes last day

Relates: #12653

Respect the last day, if we import on that day. A new test is added to reflect that situation. The workaround for missing `DatePeriod::INCLUDE_END_DATE` is extended to cover that situation. * Fix wrong determination of repeat minutes The used value was for month instead of minutes. We now use the correct identifier for minutes. This revealed that tests only passed because of the issue. We therefore fix the code to still pass correct tests by working around PHP < 8.2 limitation. * Add test to ensure that weekly basis includes last day Relates: #12653

maddy2101 (Migrated from github.com) approved these changes 2026-04-14 06:23:59 +00:00

Sign in to join this conversation.

Reviewers

No reviewers

maddy2101

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

typo3/events!97

No description provided.