-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
[stable30] fix: override iTip Broker to fix several issues #49268
Conversation
Signed-off-by: SebastianKrupinski <krupinskis05@gmail.com>
$icalMsg = new VCalendar(); | ||
|
||
foreach ($calendar->select('VTIMEZONE') as $timezone) { | ||
$icalMsg->add(clone $timezone); |
Check notice
Code scanning / Psalm
MixedClone Note
// the attendee was removed and we need to send them a CANCEL message. | ||
// Also If the meeting STATUS property was changed to CANCELLED | ||
// we need to send the attendee a CANCEL message. | ||
if (!$attendee['newInstances'] || $eventInfo['status'] === 'CANCELLED') { |
Check notice
Code scanning / Psalm
RiskyTruthyFalsyComparison Note
$message->method = $icalMsg->METHOD = 'CANCEL'; | ||
$message->significantChange = true; | ||
// clone base event | ||
$event = clone $eventInfo['instances']['master']; |
Check notice
Code scanning / Psalm
MixedClone Note
$oldEventInfo['significantChangeHash'] !== $eventInfo['significantChangeHash']; | ||
|
||
foreach ($attendee['newInstances'] as $instanceId => $instanceInfo) { | ||
$currentEvent = clone $eventInfo['instances'][$instanceId]; |
Check notice
Code scanning / Psalm
MixedClone Note
Backport of PR #48583
Requires: #49264