Jackal\ImageMerge\Command\CropCommand::execute()

Complexity

Conditions	5
Paths	3

Size

Total Lines	20
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

<?php
    public function execute(Image $image)
    {
        if ($image->getWidth() == $this->options->getDimention()->getWidth() and $image->getHeight() == $this->options->getDimention()->getHeight()) {

The method getDimention() does not exist on Jackal\ImageMerge\Command\Options\CommandOptionInterface. It seems like you code against a sub-type of Jackal\ImageMerge\Command\Options\CommandOptionInterface such as Jackal\ImageMerge\Command\Options\DimensionCommandOption.

            return $image;
        }

        if ($this->options->getDimention()->getWidth() > $image->getWidth() || $this->options->getDimention()->getHeight() > $image->getHeight()) {
            throw new InvalidArgumentException(sprintf('Crop area exceed, max dimensions are: %s X %s', $image->getWidth(), $image->getHeight()));
        }

        /** @var CropCommandOption $options */
        $options = $this->options;
        $newImage = imagecrop($image->getResource(), [
            'x' => $options->getCoordinate1()->getX(),
            'y' => $options->getCoordinate1()->getY(),
            'width' => $options->getDimention()->getWidth(),
            'height' => $options->getDimention()->getHeight(),
        ]);

        return $image->assignResource($newImage);
    }