Fix: Muliple shares are merged correctly to provide the permissions f…

…or the share receiver, small refactoring Signed-off-by: Philipp Hempel <Philipp.Hempel1@web.de>
nextcloud · Jul 20, 2023 · 15ae5de · 15ae5de
1 parent 3b39982
commit 15ae5de
Show file tree

Hide file tree

Showing 5 changed files with 170 additions and 268 deletions.
diff --git a/lib/Db/ShareMapper.php b/lib/Db/ShareMapper.php
@@ -105,6 +105,17 @@ public function findAllSharesForNode(string $nodeType, int $nodeId, string $send
 		return $this->findEntities($qb);
 	}
 
+	public function findAllSharesForNodeFor(string $nodeType, int $nodeId, string $receiver, ?string $receiverType = 'user'): array {
+		$qb = $this->db->getQueryBuilder();
+		$qb->select('*')
+			->from($this->table)
+			->where($qb->expr()->eq('receiver', $qb->createNamedParameter($receiver, IQueryBuilder::PARAM_STR)))
+			->andWhere($qb->expr()->eq('receiver_type', $qb->createNamedParameter($receiverType, IQueryBuilder::PARAM_STR)))
+			->andWhere($qb->expr()->eq('node_type', $qb->createNamedParameter($nodeType, IQueryBuilder::PARAM_STR)))
+			->andWhere($qb->expr()->eq('node_id', $qb->createNamedParameter($nodeId, IQueryBuilder::PARAM_INT)));
+		return $this->findEntities($qb);
+	}
+
 	/**
 	 * @param int $nodeId
 	 * @param string $nodeType