Page MenuHomePhorge

Avoid relying on the sender for REPLY attendee matching.
ClosedPublic

Authored by mollekopf on Dec 6 2021, 10:30 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 2, 3:15 PM
Unknown Object (File)
Sun, Dec 1, 8:13 AM
Unknown Object (File)
Sun, Nov 17, 9:48 AM
Unknown Object (File)
Wed, Nov 13, 1:32 AM
Unknown Object (File)
Mon, Nov 11, 5:07 AM
Unknown Object (File)
Wed, Nov 6, 9:02 PM
Unknown Object (File)
Nov 1 2024, 3:53 AM
Unknown Object (File)
Oct 23 2024, 6:16 AM
Subscribers

Details

Summary

This fixes the case where we reply on behalf of someone (e.g. a
resource), and therefore can't use the sender for matching.
For both displaying the itip and processing the reply to update the calendar.

Diff Detail

Repository
rRPK roundcubemail-plugins-kolab
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 37388
Build 15002: arc lint + arc unit

Event Timeline

mollekopf created this revision.
mollekopf added a reviewer: Restricted Project.Dec 6 2021, 10:36 AM
machniak subscribed.
machniak added inline comments.
plugins/libcalendaring/lib/libcalendaring_itip.php
1001

A typo.

1008

This will log a warning when $attendee[$email_field] does not exist.

1020

I would filter out attendees without email attribute. Some code expects email to exist, e.g. line 639.

1026

You can remove the else after return.

1029

$itip_attendees should be used here.

This revision now requires changes to proceed.Dec 8 2021, 11:29 AM
mollekopf marked 5 inline comments as done.

Addressed comments

This revision was not accepted when it landed; it landed in state Needs Review.Dec 16 2021, 3:40 PM
This revision was automatically updated to reflect the committed changes.
This revision is now accepted and ready to land.Dec 17 2021, 1:10 PM
This revision was automatically updated to reflect the committed changes.