Merge pull request #26079 from nextcloud/bugfix/noid/chunk-phone-search

Chunk the array of phone numbers

1 files changed, 9 insertions, 5 deletions

diff --git a/lib/private/Accounts/AccountManager.php b/lib/private/Accounts/AccountManager.php
index 05feaf87b8f..c5a0f21319e 100644
--- a/lib/private/Accounts/AccountManager.php
+++ b/lib/private/Accounts/AccountManager.php
@@ -229,19 +229,23 @@ class AccountManager implements IAccountManager {
 	}
 
 	public function searchUsers(string $property, array $values): array {
+		$chunks = array_chunk($values, 500);
 		$query = $this->connection->getQueryBuilder();
 		$query->select('*')
 			->from($this->dataTable)
 			->where($query->expr()->eq('name', $query->createNamedParameter($property)))
-			->andWhere($query->expr()->in('value', $query->createNamedParameter($values, IQueryBuilder::PARAM_STR_ARRAY)));
+			->andWhere($query->expr()->in('value', $query->createParameter('values')));
 
-		$result = $query->execute();
 		$matches = [];
+		foreach ($chunks as $chunk) {
+			$query->setParameter('values', $chunk, IQueryBuilder::PARAM_STR_ARRAY);
+			$result = $query->execute();
 
-		while ($row = $result->fetch()) {
-			$matches[$row['value']] = $row['uid'];
+			while ($row = $result->fetch()) {
+				$matches[$row['value']] = $row['uid'];
+			}
+			$result->closeCursor();
 		}
-		$result->closeCursor();
 
 		return $matches;
 	}