ahtlon/deck
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

Merge pull request #3717 from nextcloud/backport/3625/stable23

Browse Source 
[stable23] Fix: Check all circle shares for permissions
This commit is contained in:
Julius Härtl
2022-04-11 18:27:51 +02:00
committed by GitHub
parent 1cdea9dba0 b6cce55325
commit 03a9915dd5
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
lib/Service/PermissionService.php
View File

@@ -213,7 +213,9 @@ class PermissionService {
if ($this->circlesService->isCirclesEnabled() && $acl->getType() === Acl::PERMISSION_TYPE_CIRCLE) {
try {
return $this->circlesService->isUserInCircle($acl->getParticipant(), $userId) && $acl->getPermission($permission);
if ($this->circlesService->isUserInCircle($acl->getParticipant(), $userId) && $acl->getPermission($permission)) {
return true;
}
} catch (\Exception $e) {
$this->logger->info('Member not found in circle that was accessed. This should not happen.');
}