diff --git a/lib/Db/BoardMapper.php b/lib/Db/BoardMapper.php index 31a8cf43a..363916172 100644 --- a/lib/Db/BoardMapper.php +++ b/lib/Db/BoardMapper.php @@ -234,20 +234,16 @@ class BoardMapper extends DeckMapper implements IPermissionMapper { \OC::$server->getLogger()->debug('Group ' . $acl->getId() . ' not found when mapping acl ' . $acl->getParticipant()); return null; } - // TODO: get circles list if ($acl->getType() === Acl::PERMISSION_TYPE_CIRCLE) { try { - $circle = \OCA\Circles\Api\v1\Circles::detailsCircle($acl->getParticipant()); + $circle = \OCA\Circles\Api\v1\Circles::detailsCircle($acl->getParticipant(), true); if ($circle) { return new Circle($circle); } - } catch (QueryException $e) { } catch (\Exception $e) { - // TODO catch not found } return null; } - // TODO: get circles list throw new \Exception('Unknown permission type for mapping Acl'); }); } diff --git a/lib/Service/PermissionService.php b/lib/Service/PermissionService.php index 3ea6530bf..edc26bd0d 100644 --- a/lib/Service/PermissionService.php +++ b/lib/Service/PermissionService.php @@ -190,10 +190,9 @@ class PermissionService { if ($this->circlesEnabled && $acl->getType() === Acl::PERMISSION_TYPE_CIRCLE) { try { - \OCA\Circles\Api\v1\Circles::getMember($acl->getParticipant(), $this->userId, 1); + \OCA\Circles\Api\v1\Circles::getMember($acl->getParticipant(), $this->userId, 1, true); return $acl->getPermission($permission); } catch (\Exception $e) { - // TODO: getMember doesn't work for personal circles $this->logger->info('Member not found in circle that was accessed. This should not happen.'); } }