fix: prevent missing target file extension

Signed-off-by: Elizabeth Danzberger <lizzy7128@tutanota.de>

1 files changed, 8 insertions, 12 deletions

diff --git a/lib/private/Files/Conversion/ConversionManager.php b/lib/private/Files/Conversion/ConversionManager.php
index db49f7afed9..0a6090cda01 100644
--- a/lib/private/Files/Conversion/ConversionManager.php
+++ b/lib/private/Files/Conversion/ConversionManager.php
@@ -92,16 +92,16 @@ class ConversionManager implements IConversionManager {
 		$fileMimeType = $file->getMimetype();
 		$validProvider = $this->getValidProvider($fileMimeType, $targetMimeType);
 
-		$targetExtension = '';
-		foreach ($this->getProvidersForMime($fileMimeType) as $mimeProvider) {
-			if ($mimeProvider->getTo() === $targetMimeType) {
-				$targetExtension = $mimeProvider->getExtension();
-				break;
-			}
-		}
-
 		if ($validProvider !== null) {
 			$convertedFile = $validProvider->convertFile($file, $targetMimeType);
+			
+			$targetExtension = '';
+			foreach ($validProvider->getSupportedMimeTypes() as $mimeProvider) {
+				if ($mimeProvider->getTo() === $targetMimeType) {
+					$targetExtension = $mimeProvider->getExtension();
+					break;
+				}
+			}
 
 			// If destination not provided, we use the same path
 			// as the original file, but with the new extension
@@ -122,10 +122,6 @@ class ConversionManager implements IConversionManager {
 	 * @return list<IConversionProvider>
 	 */
 	private function getRegisteredProviders(): array {
-		if (count($this->providers) > 0) {
-			return $this->providers;
-		}
-
 		$context = $this->coordinator->getRegistrationContext();
 		foreach ($context->getFileConversionProviders() as $providerRegistration) {
 			$class = $providerRegistration->getService();