Page MenuHomePhorge

Resource invitation enhancements
ClosedPublic

Authored by mollekopf on Dec 6 2021, 10:30 AM.
Tags
None
Referenced Files
F11590701: D3083.diff
Fri, Mar 29, 3:18 AM
F11588166: D3083.id.diff
Thu, Mar 28, 7:25 PM
Unknown Object (File)
Fri, Mar 22, 3:33 PM
Unknown Object (File)
Tue, Mar 5, 5:55 PM
Unknown Object (File)
Tue, Mar 5, 5:55 PM
Unknown Object (File)
Tue, Mar 5, 5:55 PM
Unknown Object (File)
Tue, Mar 5, 5:55 PM
Unknown Object (File)
Tue, Mar 5, 5:55 PM
Subscribers

Details

Summary
  • Display RSVP buttons for resource owners.
  • Adapted mail texts to make sense for new resource invitations.
  • When generating a REPLY, keep track if we are responding on behalf of a resource, and set SENT-BY accordingly

Diff Detail

Repository
rRPK roundcubemail-plugins-kolab
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 37265
Build 14915: 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/calendar/calendar.php
1253

You need to add !empty($attendee['cutype']) to prevent from a warning in newer PHP versions. Also, Roundcube has in_array_nocase() function which we should just use here.

1256

This will not gonna work when there's multiple resources (with the same owner) in the event.

1265

The only difference is the body label argument, so instead of adding a whole if block I would just change the argument depending on the $resourceConfirmation state.

2023

The foreach is redundant.

2899

This if condition is not good enough for this purpose. And it assumes that the owner DN contains email address, I'm not sure this is always true with old Kolab.

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

Addressed comments

plugins/calendar/calendar.php
1256

For the case of multiple resources we'll have to show individual RSVP buttons per matching attendee, that will be processed individually, so I think also in that case we can set _resource to the correct $attendee.

For the time being the entire codebase just does not deal with multiple matching attendees AFAIK, so nothing that I can reasonably fix in this patch I think.

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.

Fixed owned resource lookup

Fixed owned resource lookup

This revision is now accepted and ready to land.Dec 17 2021, 1:07 PM
This revision was automatically updated to reflect the committed changes.