Skip to content

Commit ca30aea

Browse files
committed
fix: normalize validation metadata required fields
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
1 parent 079e070 commit ca30aea

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

lib/Service/File/MetadataLoader.php

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,6 @@ public function loadMetadata(?File $file, stdClass $fileData): void {
3535
$fileNode = $this->getFileNode($file);
3636
$metadata = $file->getMetadata() ?? [];
3737

38-
$fileData->metadata = $metadata;
39-
4038
if (method_exists($fileNode, 'getSize')) {
4139
$fileData->size = $fileNode->getSize();
4240
}
@@ -49,9 +47,15 @@ public function loadMetadata(?File $file, stdClass $fileData): void {
4947
}
5048

5149
$fileData->pages = $this->getPages($file);
52-
5350
$fileData->totalPages = (int)($metadata['p'] ?? count($fileData->pages ?? []));
5451
$fileData->pdfVersion = (string)($metadata['pdfVersion'] ?? '');
52+
53+
$metadata['p'] = $fileData->totalPages;
54+
$extension = pathinfo($file->getName(), PATHINFO_EXTENSION);
55+
if (!isset($metadata['extension']) || !is_string($metadata['extension']) || trim($metadata['extension']) === '') {
56+
$metadata['extension'] = is_string($extension) && $extension !== '' ? strtolower($extension) : 'pdf';
57+
}
58+
$fileData->metadata = $metadata;
5559
} catch (\Throwable $e) {
5660
$this->logger->warning('Failed to load file metadata: ' . $e->getMessage());
5761
}

0 commit comments

Comments
 (0)