From 0681ced2031869d260ba88fc42b8a1c6998b50e2 Mon Sep 17 00:00:00 2001 From: Anna Larch Date: Mon, 4 May 2026 18:59:09 +0200 Subject: [PATCH] fix(previews): use createParameter/setParameter to reuse query in chunk loop AI-Assisted-By: claude-sonnet-4-6 Signed-off-by: Anna Larch AI-Assisted-By: Claude Sonnet 4.6 --- .../Preview/Storage/LocalPreviewStorage.php | 26 ++++++++++--------- .../Storage/LocalPreviewStorageTest.php | 1 + 2 files changed, 15 insertions(+), 12 deletions(-) diff --git a/lib/private/Preview/Storage/LocalPreviewStorage.php b/lib/private/Preview/Storage/LocalPreviewStorage.php index a64fdfe295f..13e6a714516 100644 --- a/lib/private/Preview/Storage/LocalPreviewStorage.php +++ b/lib/private/Preview/Storage/LocalPreviewStorage.php @@ -310,13 +310,14 @@ class LocalPreviewStorage implements IPreviewStorage { } $result = []; + $qb = $this->connection->getTypedQueryBuilder(); + $qb->selectColumns('fileid', 'storage', 'etag', 'mimetype') + ->from('filecache') + ->where($qb->expr()->in('fileid', $qb->createParameter('fileIds'))) + ->runAcrossAllShards(); foreach (array_chunk($fileIds, 1000) as $chunk) { - $qb = $this->connection->getTypedQueryBuilder(); - $qb->selectColumns('fileid', 'storage', 'etag', 'mimetype') - ->from('filecache') - ->where($qb->expr()->in('fileid', $qb->createNamedParameter($chunk, IQueryBuilder::PARAM_INT_ARRAY))); - $rows = $qb->runAcrossAllShards() - ->executeQuery(); + $qb->setParameter('fileIds', $chunk, IQueryBuilder::PARAM_INT_ARRAY); + $rows = $qb->executeQuery(); while ($row = $rows->fetchAssociative()) { $result[(int)$row['fileid']] = $row; } @@ -336,13 +337,14 @@ class LocalPreviewStorage implements IPreviewStorage { } $result = []; + $qb = $this->connection->getTypedQueryBuilder(); + $qb->selectColumns('fileid', 'storage', 'etag', 'mimetype', 'parent', 'path_hash') + ->from('filecache') + ->where($qb->expr()->in('path_hash', $qb->createParameter('pathHashes'))) + ->runAcrossAllShards(); foreach (array_chunk($pathHashes, 1000) as $chunk) { - $qb = $this->connection->getTypedQueryBuilder(); - $qb->selectColumns('fileid', 'storage', 'etag', 'mimetype', 'parent', 'path_hash') - ->from('filecache') - ->where($qb->expr()->in('path_hash', $qb->createNamedParameter($chunk, IQueryBuilder::PARAM_STR_ARRAY))); - $rows = $qb->runAcrossAllShards() - ->executeQuery(); + $qb->setParameter('pathHashes', $chunk, IQueryBuilder::PARAM_STR_ARRAY); + $rows = $qb->executeQuery(); while ($row = $rows->fetchAssociative()) { $result[$row['path_hash']] = $row; } diff --git a/tests/lib/Preview/Storage/LocalPreviewStorageTest.php b/tests/lib/Preview/Storage/LocalPreviewStorageTest.php index dbc3cea5342..14e6419a9d5 100644 --- a/tests/lib/Preview/Storage/LocalPreviewStorageTest.php +++ b/tests/lib/Preview/Storage/LocalPreviewStorageTest.php @@ -129,6 +129,7 @@ class LocalPreviewStorageTest extends TestCase { $qbMock = $this->createMock(ITypedQueryBuilder::class); $qbMock->method('selectColumns')->willReturnSelf(); $qbMock->method('from')->willReturnSelf(); + $qbMock->method('where')->willReturnSelf(); $qbMock->method('andWhere')->willReturnSelf(); $qbMock->method('runAcrossAllShards')->willReturnSelf(); $qbMock->method('executeQuery')->willReturn($resultMock);