App\Src\UseCases\Domain\Agricultural\Queries\GetIcon::execute()

Complexity

Conditions	3
Paths	2

Size

Total Lines	24
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    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();
                });
            }
        }, 3600, true);
        return $img->response();
    }