From e3179530b73283b581d95fa4a56694e70c76a2c9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Julius=20H=C3=A4rtl?= Date: Tue, 15 Sep 2020 22:10:42 +0200 Subject: [PATCH] Only return if result was found 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 | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/Db/CardMapper.php b/lib/Db/CardMapper.php index bbbcdb560..8d055c035 100644 --- a/lib/Db/CardMapper.php +++ b/lib/Db/CardMapper.php @@ -202,14 +202,14 @@ class CardMapper extends DeckMapper implements IPermissionMapper { $sql = 'SELECT owner FROM `*PREFIX*deck_boards` WHERE `id` IN (SELECT board_id FROM `*PREFIX*deck_stacks` WHERE id IN (SELECT stack_id FROM `*PREFIX*deck_cards` WHERE id = ?))'; $stmt = $this->execute($sql, [$cardId]); $row = $stmt->fetch(); - return ($row['owner'] === $userId); + return $row ? ($row['owner'] === $userId) : false; } public function findBoardId($cardId) { $sql = 'SELECT id FROM `*PREFIX*deck_boards` WHERE `id` IN (SELECT board_id FROM `*PREFIX*deck_stacks` WHERE id IN (SELECT stack_id FROM `*PREFIX*deck_cards` WHERE id = ?))'; $stmt = $this->execute($sql, [$cardId]); $row = $stmt->fetch(); - return $row['id']; + return $row ? $row['id'] : null; } public function mapOwner(Card &$card) {