From 5d3b6d92a61fa64f4a719fd87eadc4d82df54293 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Julius=20H=C3=A4rtl?= Date: Mon, 21 Jun 2021 08:20:18 +0200 Subject: [PATCH] Always pass user id in share provider MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Julius Härtl --- lib/Sharing/DeckShareProvider.php | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/Sharing/DeckShareProvider.php b/lib/Sharing/DeckShareProvider.php index d9ca09bda..5e6655cd6 100644 --- a/lib/Sharing/DeckShareProvider.php +++ b/lib/Sharing/DeckShareProvider.php @@ -271,9 +271,9 @@ class DeckShareProvider implements \OCP\Share\IShareProvider { return $share; } - private function applyBoardPermission($share, $permissions) { + private function applyBoardPermission($share, $permissions, $userId) { try { - $this->permissionService->checkPermission($this->cardMapper, $share->getSharedWith(), Acl::PERMISSION_EDIT); + $this->permissionService->checkPermission($this->cardMapper, $share->getSharedWith(), Acl::PERMISSION_EDIT, $userId); } catch (NoPermissionException $e) { $permissions &= Constants::PERMISSION_ALL - Constants::PERMISSION_UPDATE; $permissions &= Constants::PERMISSION_ALL - Constants::PERMISSION_CREATE; @@ -281,7 +281,7 @@ class DeckShareProvider implements \OCP\Share\IShareProvider { } try { - $this->permissionService->checkPermission($this->cardMapper, $share->getSharedWith(), Acl::PERMISSION_SHARE); + $this->permissionService->checkPermission($this->cardMapper, $share->getSharedWith(), Acl::PERMISSION_SHARE, $userId); } catch (NoPermissionException $e) { $permissions &= Constants::PERMISSION_ALL - Constants::PERMISSION_SHARE; } @@ -646,7 +646,7 @@ class DeckShareProvider implements \OCP\Share\IShareProvider { $stmt = $query->execute(); while ($data = $stmt->fetch()) { - $this->applyBoardPermission($shareMap[$data['parent']], (int)$data['permissions']); + $this->applyBoardPermission($shareMap[$data['parent']], (int)$data['permissions'], $userId); $shareMap[$data['parent']]->setTarget($data['file_target']); }