Merge remote-tracking branch 'upstream/5.2-dev' into 5.3-dev-upmerge-2025-03-11

richard67 · richard67 · commit 124711d72881 · 2025-03-11T17:24:46.000+01:00
diff --git a/plugins/filesystem/local/src/Adapter/LocalAdapter.php b/plugins/filesystem/local/src/Adapter/LocalAdapter.php
@@ -531,6 +531,10 @@ private function copyFile(string $sourcePath, string $destinationPath, bool $for
             $destinationPath .= '/' . $this->getFileName($sourcePath);
         }
 
+        if (!MediaHelper::checkFileExtension(pathinfo($destinationPath, PATHINFO_EXTENSION))) {
+            throw new \Exception(Text::_('COM_MEDIA_MOVE_FILE_EXTENSION_INVALID'));
+        }
+
         if (file_exists($destinationPath) && !$force) {
             throw new \Exception(Text::_('COM_MEDIA_COPY_FILE_NOT_POSSIBLE_FILE_ALREADY_EXISTS'));
         }

Original file line number	Diff line number	Diff line change
`@@ -531,6 +531,10 @@ private function copyFile(string $sourcePath, string $destinationPath, bool $for`
`531`	`531`	`$destinationPath .= '/' . $this->getFileName($sourcePath);`
`532`	`532`	`}`
`533`	`533`
	`534`	`+ if (!MediaHelper::checkFileExtension(pathinfo($destinationPath, PATHINFO_EXTENSION))) {`
	`535`	`+ throw new \Exception(Text::_('COM_MEDIA_MOVE_FILE_EXTENSION_INVALID'));`
	`536`	`+ }`
	`537`	`+`
`534`	`538`	`if (file_exists($destinationPath) && !$force) {`
`535`	`539`	`throw new \Exception(Text::_('COM_MEDIA_COPY_FILE_NOT_POSSIBLE_FILE_ALREADY_EXISTS'));`
`536`	`540`	`}`