WebThumbnailer\Utils\ImageUtils::generateThumbnail()

Complexity

Conditions	13
Paths	19

Size

Total Lines	67
Code Lines	41

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	0

<?php
    public static function generateThumbnail($imageStr, $target, $maxWidth, $maxHeight, $crop = false)
    {
        if (! touch($target)) {
            throw new ImageConvertException('Target file is not writable.');
        }

        if ($crop && ($maxWidth == 0  || $maxHeight == 0)) {
            throw new ImageConvertException('Both width and height must be provided for cropping');
        }

        $sourceImg = static::imageCreateFromString($imageStr);
        if ($sourceImg === false) {
            throw new NotAnImageException();
        }

        $originalWidth = imagesx($sourceImg);
        $originalHeight = imagesy($sourceImg);
        if ($maxWidth > $originalWidth) {
            $maxWidth = $originalWidth;
        }
        if ($maxHeight > $originalHeight) {
            $maxHeight = $originalHeight;
        }

        list($finalWidth, $finalHeight) = self::calcNewSize(
            $originalWidth,
            $originalHeight,
            $maxWidth,
            $maxHeight,
            $crop
        );

        $targetImg = imagecreatetruecolor($finalWidth, $finalHeight);
        if ($targetImg === false) {
            throw new ImageConvertException('Could not generate the thumbnail from source image.');
        }

        if (! imagecopyresized(
            $targetImg,
            $sourceImg,
            0,
            0,
            0,
            0,
            $finalWidth,
            $finalHeight,
            $originalWidth,
            $originalHeight
        )
        ) {
            static::imageDestroy($sourceImg);
            static::imageDestroy($targetImg);
            throw new ImageConvertException('Could not generate the thumbnail from source image.');
        }

        if ($crop) {
            $targetImg = imagecrop($targetImg, [
                'x' => $finalWidth >= $finalHeight ? ($finalWidth - $maxWidth) / 2 : 0,
                'y' => $finalHeight <= $finalWidth ? ($finalHeight - $maxHeight) / 2 : 0,
                'width' => $maxWidth,
                'height' => $maxHeight
            ]);
        }

        imagedestroy($sourceImg);
        imagejpeg($targetImg, $target);

It seems like $targetImg can also be of type boolean; however, parameter $image of imagejpeg() does only seem to accept resource, maybe add an additional type check?

        imagedestroy($targetImg);

It seems like $targetImg can also be of type boolean; however, parameter $image of imagedestroy() does only seem to accept resource, maybe add an additional type check?

    }