App\Src\UseCases\Domain\Context\Queries\GetIcon

Complexity

Total Complexity

3

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	execute()	0	24	3

<?php


namespace App\Src\UseCases\Domain\Context\Queries;


use Intervention\Image\Facades\Image;

class GetIcon
{
    public function execute(string $uuid, ?int $dim)
    {
        $pathPicture = storage_path('app/public/characteristics/'.$uuid.'.png');

        $img = Image::make($pathPicture);
        $h = $img->height();
        $w = $img->width();

        if($dim == null){

It seems like you are loosely comparing $dim of type integer|null against null; this is ambiguous if the integer can be zero. Consider using a strict comparison === instead.

            return $img->response();
        }

        $img = Image::cache(function($image) use($pathPicture, $dim, $w, $h){
            if($w <= $h) {
                $image->make($pathPicture)->widen($dim, function ($constraint) {
                    $constraint->upsize();
                });
            }else{
                $image->make($pathPicture)->heighten($dim, function ($constraint) {
                    $constraint->upsize();
                });
            }
        }, 86400, true);
        return $img->response();
    }

}