Don't recreate sql query each time

Signed-off-by: Carl Schwan <carl@carlschwan.eu>
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>

Author: CarlSchwan

apps/dav/lib/CardDAV/CardDavBackend.php

@@ -1111,18 +1111,18 @@ private function searchByAddressBookIds(array $addressBookIds,
 		}, $matches);
 
 		$cards = [];
-		foreach (array_chunk($matches, 1000) as $matche) {
-			$query = $this->db->getQueryBuilder();
-			$query->select('c.addressbookid', 'c.carddata', 'c.uri')
-				->from($this->dbCardsTable, 'c')
-				->where($query->expr()->in('c.id', $query->createNamedParameter($matche, IQueryBuilder::PARAM_INT_ARRAY)));
+		$query = $this->db->getQueryBuilder();
+		$query->select('c.addressbookid', 'c.carddata', 'c.uri')
+			->from($this->dbCardsTable, 'c')
+			->where($query->expr()->in('c.id', $query->createParameter('matches')));
 
+		foreach (array_chunk($matches, 1000) as $matchesChunk) {
+			$query->setParameter('matches', $matchesChunk, IQueryBuilder::PARAM_INT_ARRAY);
 			$result = $query->execute();
 			$cards = array_merge($cards, $result->fetchAll());
 			$result->closeCursor();
 		}
 
-
 		return array_map(function ($array) {
 			$array['addressbookid'] = (int) $array['addressbookid'];
 			$modified = false;