From 858894ad572bfe734896c6dea9891752440380b3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tim=20B=C3=A4cker?= Date: Wed, 22 Jul 2026 12:46:34 +0200 Subject: [PATCH 1/2] [BUGFIX] Return file extension for processed files ProcessedFile::getProperty('extension') always returns null because processed-file records do not store an extension property. Use getExtension() instead so image metadata (e.g. ogImage) exposes the correct extension after processing. --- Classes/Utility/FileUtility.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Classes/Utility/FileUtility.php b/Classes/Utility/FileUtility.php index ad69759f..6467931f 100644 --- a/Classes/Utility/FileUtility.php +++ b/Classes/Utility/FileUtility.php @@ -153,7 +153,7 @@ public function process(FileInterface $fileReference, ProcessingConfiguration $p ], 'crop' => $crop, 'autoplay' => $fileReference->getProperty('autoplay'), - 'extension' => $fileReference->getProperty('extension'), + 'extension' => $fileReference->getExtension() ?: null, ]; $processedProperties = array_merge( From 318946df7fd994c0da73e08644cb4d0f5a3270ad Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tim=20B=C3=A4cker?= Date: Fri, 31 Jul 2026 14:02:52 +0200 Subject: [PATCH 2/2] [BUGFIX] Update FileUtilityTest for getExtension() usage Mock ProcessedFile::getExtension() and adjust the video baseline so unit tests match the FileUtility change that no longer reads extension via getProperty(). --- Tests/Unit/Utility/FileUtilityTest.php | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/Tests/Unit/Utility/FileUtilityTest.php b/Tests/Unit/Utility/FileUtilityTest.php index 67df0cfb..3be003f6 100644 --- a/Tests/Unit/Utility/FileUtilityTest.php +++ b/Tests/Unit/Utility/FileUtilityTest.php @@ -623,10 +623,11 @@ protected function getMockProcessedFileForData($data) { $processedFile = $this->createPartialMock( ProcessedFile::class, - ['getProperty', 'getMimeType', 'getSize', 'hasProperty', 'getPublicUrl'] + ['getProperty', 'getMimeType', 'getSize', 'hasProperty', 'getPublicUrl', 'getExtension'] ); $processedFile->method('getMimeType')->willReturn('image/jpeg'); $processedFile->method('getSize')->willReturn($data['size']); + $processedFile->method('getExtension')->willReturn((string)($data['extension'] ?? '')); $processedFile->method('getProperty')->willReturnCallback(static function ($key) use ($data) { return $data[$key] ?? null; }); @@ -762,7 +763,7 @@ protected function getBaselineResultArrayForVideoFileReference(): array ], 'crop' => '{"default":{"cropArea":{"x":0,"y":0,"width":1,"height":1},"selectedRatio":"NaN","focusArea":null}}', 'autoplay' => 0, - 'extension' => 'jpg', + 'extension' => null, ], ]; }