From 8399b00a104328f64502e146c4591492e84ef2b3 Mon Sep 17 00:00:00 2001 From: Raul Ferreira Fuentes Date: Mon, 11 Apr 2022 19:46:05 +0200 Subject: [PATCH] Fix Entity class in `AclMapper` Signed-off-by: Raul Ferreira Fuentes --- lib/Db/AclMapper.php | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/lib/Db/AclMapper.php b/lib/Db/AclMapper.php index 95e1af55a..d95f88c46 100644 --- a/lib/Db/AclMapper.php +++ b/lib/Db/AclMapper.php @@ -34,7 +34,7 @@ class AclMapper extends DeckMapper implements IPermissionMapper { * @param IDBConnection $db */ public function __construct(IDBConnection $db) { - parent::__construct($db, 'deck_board_acl', Board::class); + parent::__construct($db, 'deck_board_acl', Acl::class); } /** @@ -46,7 +46,7 @@ class AclMapper extends DeckMapper implements IPermissionMapper { */ public function findAll($boardId, $limit = null, $offset = null) { $qb = $this->db->getQueryBuilder(); - $qb->select('*') + $qb->select('id', 'board_id', 'type', 'participant', 'permission_edit', 'permission_share', 'permission_manage') ->from('deck_board_acl') ->where($qb->expr()->eq('board_id', $qb->createNamedParameter($boardId, IQueryBuilder::PARAM_INT))) ->setMaxResults($limit) @@ -63,11 +63,10 @@ class AclMapper extends DeckMapper implements IPermissionMapper { */ public function isOwner($userId, $aclId): bool { $qb = $this->db->getQueryBuilder(); - - $qb->select('owner') - ->from($this->getTableName()) + $qb->select('*') + ->from($this->getTableName(), 'acl') ->innerJoin('acl', 'deck_boards','b', 'acl.board_id = b.id') - ->where($qb->expr()->eq('owner.id', $qb->createNamedParameter($userId, IQueryBuilder::PARAM_INT))) + ->where($qb->expr()->eq('owner', $qb->createNamedParameter($userId, IQueryBuilder::PARAM_STR))) ->andWhere($qb->expr()->eq('acl.id', $qb->createNamedParameter($aclId, IQueryBuilder::PARAM_INT))); return $qb->executeQuery()->rowCount() > 0;