Skip to content

Shared calendar invitations #36756

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.

Sign up for GitHub

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

Closed
rotdrop wants to merge 3 commits into from
Closed

Shared calendar invitations #36756

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
d0ec954
Revert 'Revert "send invitations for shared calendars"'
rotdrop Feb 16, 2023
b303189
Try to fix some psalm things. No change to the code apart from that.
rotdrop Feb 17, 2023
bad66ad
getCalendarObjectByUID(): prefer objects in shared writable calendars…
rotdrop Feb 20, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
getCalendarObjectByUID(): prefer objects in shared writable calendars… 
… over private objects.

Signed-off-by: Claus-Justus Heine <[email protected]>
  • Loading branch information
@rotdrop
rotdrop committed May 5, 2023
commit bad66ad80797df1c4baa8269bb1645a2e3589a26
18 changes: 12 additions & 6 deletions apps/dav/lib/CalDAV/CalDavBackend.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2182,6 +2182,7 @@ public function getCalendarObjectByUID($principalUri, $uid) {
->selectAlias('c.uri', 'calendaruri')
->selectAlias('co.uri', 'objecturi')
->selectAlias('ds.access', 'access')
->selectAlias('ds.principaluri', 'shareprincipal')
->from('calendarobjects', 'co')
->leftJoin('co', 'calendars', 'c', $query->expr()->eq('co.calendarid', 'c.id'))
->leftJoin('co', 'dav_shares', 'ds', $query->expr()->eq('co.calendarid', 'ds.resourceid'))
Expand All @@ -2199,24 +2200,29 @@ public function getCalendarObjectByUID($principalUri, $uid) {
->setParameter('type', 'calendar')
->setParameter('shareprincipal', $principals, \Doctrine\DBAL\Connection::PARAM_STR_ARRAY);
$stmt = $query->executeQuery();
$row = $stmt->fetch();
$stmt->closeCursor();
if ($row) {
if ($row['principaluri'] != $principalUri) {
$calendarObjectUri = null;
while ($row = $stmt->fetch()) {
if ($row['principaluri'] != $principalUri && !empty($row['shareprincipal']) && $row['access'] == Backend::ACCESS_READ_WRITE) {
/**
* This seeems to be a false positive: we have "use Sabre\Uri" and Uri\split() IS defined.
*
* @psalm-suppress UndefinedFunction
*/
[, $name] = Uri\split($row['principaluri']);
$calendarUri = $row['calendaruri'] . '_shared_by_' . $name;
} elseif (!empty($calendarObjectUri)) {
// There could be multiple entries for the UID if the share
// permissions have been changed "in between". In this case we
// prefer the shared calendar object.
continue;
} else {
$calendarUri = $row['calendaruri'];
}
return $calendarUri . '/' . $row['objecturi'];
$calendarObjectUri = $calendarUri . '/' . $row['objecturi'];
}
$stmt->closeCursor();

return null;
return $calendarObjectUri;
}

public function getCalendarObjectById(string $principalUri, int $id): ?array {
Expand Down