Ecodev\Felix\Service\ImageResizer::resize()

Complexity

Conditions	4
Paths	5

Size

Total Lines	20
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	4.0092

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function resize(Image $image, int $maxHeight, bool $useWebp): string
    {
        if ($image->getMime() === 'image/svg+xml') {
            return $image->getPath();
        }

        $maxHeight = min($maxHeight, $image->getHeight());

        $basename = pathinfo($image->getFilename(), PATHINFO_FILENAME);
        $extension = $useWebp ? '.webp' : '.jpg';
        $path = realpath('.') . '/' . self::CACHE_IMAGE_PATH . $basename . '-' . $maxHeight . $extension;

Are you sure $basename of type array|string can be used in concatenation?

        if (file_exists($path)) {
            return $path;
        }

        $image = $this->imagine->open($image->getPath());
        $image->thumbnail(new Box(1000000, $maxHeight))->save($path);

        return $path;
    }