From: Simon L Date: Wed, 8 Mar 2023 16:45:12 +0000 (+0100) Subject: imaginary - fix autorotate and improve the logic X-Git-Tag: v27.0.0beta1~356^2 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=b39329c21cfbafad348127b11b7dc202a38b7308;p=nextcloud-server.git imaginary - fix autorotate and improve the logic Signed-off-by: Simon L --- diff --git a/lib/private/Preview/Imaginary.php b/lib/private/Preview/Imaginary.php index 5d559b65f00..ca46383e58b 100644 --- a/lib/private/Preview/Imaginary.php +++ b/lib/private/Preview/Imaginary.php @@ -82,8 +82,14 @@ class Imaginary extends ProviderV2 { $httpClient = $this->service->newClient(); $convert = false; + $autorotate = true; switch ($file->getMimeType()) { + case 'image/heic': + // Autorotate seems to be broken for Heic so disable for that + $autorotate = false; + $mimeType = 'jpeg'; + break; case 'image/gif': case 'image/png': $mimeType = 'png'; @@ -92,50 +98,43 @@ class Imaginary extends ProviderV2 { case 'application/pdf': case 'application/illustrator': $convert = true; + // Converted files do not need to be autorotated + $autorotate = false; + $mimeType = 'png'; break; default: $mimeType = 'jpeg'; } + + $operations = []; if ($convert) { - $operations = [ - [ - 'operation' => 'convert', - 'params' => [ - 'type' => 'png', - ] - ], - [ - 'operation' => ($crop ? 'smartcrop' : 'fit'), - 'params' => [ - 'width' => $maxX, - 'height' => $maxY, - 'type' => 'png', - 'norotation' => 'true', - ] + $operations[] = [ + 'operation' => 'convert', + 'params' => [ + 'type' => $mimeType, ] ]; - } else { - $quality = $this->config->getAppValue('preview', 'jpeg_quality', '80'); - - $operations = [ - [ - 'operation' => 'autorotate', - ], - [ - 'operation' => ($crop ? 'smartcrop' : 'fit'), - 'params' => [ - 'width' => $maxX, - 'height' => $maxY, - 'stripmeta' => 'true', - 'type' => $mimeType, - 'norotation' => 'true', - 'quality' => $quality, - ] - ] + } elseif ($autorotate) { + $operations[] = [ + 'operation' => 'autorotate', ]; } + $quality = $this->config->getAppValue('preview', 'jpeg_quality', '80'); + + $operations[] = [ + 'operation' => ($crop ? 'smartcrop' : 'fit'), + 'params' => [ + 'width' => $maxX, + 'height' => $maxY, + 'stripmeta' => 'true', + 'type' => $mimeType, + 'norotation' => 'true', + 'quality' => $quality, + ] + ]; + try { $response = $httpClient->post( $imaginaryUrl . '/pipeline', [