clones/friendica
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Revert "Fix unhandled image detection" 

This reverts commit 1069cfb570.
This commit is contained in:
Michael 2024-02-17 10:50:09 +00:00
parent 1069cfb570
commit 7d10518e94
4 changed files with 6 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/Module/Photo.php
View File

@ -207,13 +207,13 @@ class Photo extends BaseApi
$mimetype = $img->getType(); $mimetype = $img->getType();
} }
// if customsize is set and image resizing is supported for this image, resize it // if customsize is set and image is not a gif, resize it
if (Images::canResize($photo['type']) && $customsize > 0 && $customsize <= Proxy::PIXEL_THUMB && $square_resize) { if ($photo['type'] !== image_type_to_mime_type(IMAGETYPE_GIF) && $customsize > 0 && $customsize <= Proxy::PIXEL_THUMB && $square_resize) {
$img = new Image($imgdata, $photo['type'], $photo['filename']); $img = new Image($imgdata, $photo['type'], $photo['filename']);
$img->scaleToSquare($customsize); $img->scaleToSquare($customsize);
$imgdata = $img->asString(); $imgdata = $img->asString();
$mimetype = $img->getType(); $mimetype = $img->getType();
} elseif (Images::canResize($photo['type']) && $customsize > 0) { } elseif ($photo['type'] !== image_type_to_mime_type(IMAGETYPE_GIF) && $customsize > 0) {
$img = new Image($imgdata, $photo['type'], $photo['filename']); $img = new Image($imgdata, $photo['type'], $photo['filename']);
$img->scaleDown($customsize); $img->scaleDown($customsize);
$imgdata = $img->asString(); $imgdata = $img->asString();

4
src/Module/Proxy.php
View File

@ -106,8 +106,8 @@ class Proxy extends BaseModule
// stop. // stop.
} }
// reduce quality - if it is supported for this image type // reduce quality - if it isn't a GIF
if (Images::canResize($image->getType())) { if ($image->getImageType() != IMAGETYPE_GIF) {
$image->scaleDown($request['size']); $image->scaleDown($request['size']);
} }

2
src/Object/Image.php
View File

@ -66,7 +66,7 @@ class Image
if (Images::isSupportedMimeType($type)) { if (Images::isSupportedMimeType($type)) {
$this->imageType = Images::getImageTypeByMimeType($type); $this->imageType = Images::getImageTypeByMimeType($type);
} elseif (($type == '') || substr($type, 0, 6) == 'image/' || substr($type, 0, 12) == ' application/') { } elseif (($type == '') || substr($type, 0, 6) != 'image/' || substr($type, 0, 12) != ' application/') {
$this->imageType = IMAGETYPE_WEBP; $this->imageType = IMAGETYPE_WEBP;
DI::logger()->debug('Unhandled image mime type, use WebP instead', ['type' => $type, 'filename' => $filename, 'size' => strlen($data)]); DI::logger()->debug('Unhandled image mime type, use WebP instead', ['type' => $type, 'filename' => $filename, 'size' => strlen($data)]);
} else { } else {

15
src/Util/Images.php
View File

@ -183,21 +183,6 @@ class Images
return $types; return $types;
} }
/**
* Checks if the provided mime type can be handled for resizing.
* Only with Imagick installed, animated GIF and WebP keep their animation after resize.
*
* @param string $mimetype
* @return boolean
*/
public static function canResize(string $mimetype): bool
{
if (in_array(self::getImageTypeByMimeType($mimetype), [IMAGETYPE_GIF, IMAGETYPE_WEBP])) {
return class_exists('Imagick');
}
return true;
}
/** /**
* Fetch image mimetype from the image data or guessing from the file name * Fetch image mimetype from the image data or guessing from the file name
* *