diff options
| author | Carl Schwan <carl@carlschwan.eu> | 2022-03-10 15:10:08 +0100 |
|---|---|---|
| committer | backportbot[bot] <backportbot[bot]@users.noreply.github.com> | 2022-03-11 16:03:24 +0000 |
| commit | b7838c77900f1967da2d9b38d64399e330223a5e (patch) | |
| tree | 65e0a936a10dd9545348b14a37f0ae7655e83941 | |
| parent | f7557d84421563a426475bec4716f3d046424ab2 (diff) | |
| download | nextcloud-server-b7838c77900f1967da2d9b38d64399e330223a5e.tar.gz nextcloud-server-b7838c77900f1967da2d9b38d64399e330223a5e.zip | |
Don't recreate sql query each time
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
| -rw-r--r-- | apps/dav/lib/CardDAV/CardDavBackend.php | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/apps/dav/lib/CardDAV/CardDavBackend.php b/apps/dav/lib/CardDAV/CardDavBackend.php index 98e85096b3b..1c1754ff752 100644 --- a/apps/dav/lib/CardDAV/CardDavBackend.php +++ b/apps/dav/lib/CardDAV/CardDavBackend.php @@ -139,7 +139,7 @@ class CardDavBackend implements BackendInterface, SyncSupport { ->from('addressbooks') ->where($query->expr()->eq('principaluri', $query->createNamedParameter($principalUri))); - $result = $query->execute(); + $result = $query->executeQuery(); $column = (int) $result->fetchOne(); $result->closeCursor(); return $column; @@ -1131,18 +1131,18 @@ class CardDavBackend implements BackendInterface, SyncSupport { }, $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'))); - $result = $query->execute(); + foreach (array_chunk($matches, 1000) as $matchesChunk) { + $query->setParameter('matches', $matchesChunk, IQueryBuilder::PARAM_INT_ARRAY); + $result = $query->executeQuery(); $cards = array_merge($cards, $result->fetchAll()); $result->closeCursor(); } - return array_map(function ($array) { $array['addressbookid'] = (int) $array['addressbookid']; $modified = false; |