From ac1b1e826f59a8357384dc236f946c4df223e374 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Julius=20H=C3=A4rtl?= Date: Tue, 3 Oct 2017 18:48:06 +0200 Subject: [PATCH] Fix scritinizer issues MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Julius Härtl --- lib/Db/CardMapper.php | 2 +- lib/Service/PermissionService.php | 8 ++++++-- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/lib/Db/CardMapper.php b/lib/Db/CardMapper.php index f42d7a339..64de8b55e 100644 --- a/lib/Db/CardMapper.php +++ b/lib/Db/CardMapper.php @@ -35,7 +35,7 @@ class CardMapper extends DeckMapper implements IPermissionMapper { private $labelMapper; /** @var IUserManager */ private $userManager; - /** @var Manager */ + /** @var IManager */ private $notificationManager; public function __construct( diff --git a/lib/Service/PermissionService.php b/lib/Service/PermissionService.php index c605ae0cd..cf5631a51 100644 --- a/lib/Service/PermissionService.php +++ b/lib/Service/PermissionService.php @@ -184,14 +184,18 @@ class PermissionService { * Required to allow assigning them to cards * * @param $boardId - * @return array|null + * @return array */ public function findUsers($boardId) { // cache users of a board so we don't query them for every cards if (array_key_exists((string)$boardId, $this->users)) { return $this->users[(string)$boardId]; } - $board = $this->boardMapper->find($boardId); + try { + $board = $this->boardMapper->find($boardId); + } catch (DoesNotExistException $e) { + return []; + } $users = [ new User($this->userManager->get($board->getOwner())) ];