Build query once instead of in-loop

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
nextcloud · Sep 7, 2023 · d18bb7e · d18bb7e
1 parent 35069ad
commit d18bb7e
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/lib/private/Group/Database.php b/lib/private/Group/Database.php
@@ -352,12 +352,13 @@ public function groupsExists(array $gids): array {
 			}
 		}
 
-		foreach (array_chunk($notFoundGids, 1000) as $chunk) {
-			$qb = $this->dbConn->getQueryBuilder();
-			$result = $qb->select('gid', 'displayname')
+		$qb = $this->dbConn->getQueryBuilder();
+		$qb->select('gid', 'displayname')
 				->from('groups')
-				->where($qb->expr()->in('gid', $qb->createNamedParameter($chunk, IQueryBuilder::PARAM_STR_ARRAY)))
-				->executeQuery();
+				->where($qb->expr()->in('gid', $qb->createParameter('ids')));
+		foreach (array_chunk($notFoundGids, 1000) as $chunk) {
+			$qb->setParameter('ids', $chunk, IQueryBuilder::PARAM_STR_ARRAY);
+			$result = $qb->executeQuery();
 			while ($row = $result->fetch()) {
 				$this->groupCache[(string)$row['gid']] = [
 					'displayname' => (string)$row['displayname'],