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

Complexity

Conditions	1
Paths	1

Size

Total Lines	45
Code Lines	24

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

<?php
    public function execute(Image $image)
    {
        /** @var MultiCoordinateCommandOption $options */
        $options = $this->options;

        $builder = new ImageBuilder($image);

        // Setup the merge image from the source image with scaling
        $mergeImage = ImageCreateTrueColor($image->getWidth(), $image->getHeight());
        imagecopyresampled($mergeImage, $image->getResource(), 0, 0, 0, 0, $image->getWidth(), $image->getHeight(), imagesx($image->getResource()), imagesy($image->getResource()));

It seems like $mergeImage can also be of type false; however, parameter $dst_image of imagecopyresampled() does only seem to accept resource, maybe add an additional type check?

        $maskPolygon = imagecreatetruecolor($image->getWidth(), $image->getHeight());
        $borderColor = ColorUtils::colorIdentifier($maskPolygon, new Color('01feff'));
        imagefill($maskPolygon, 0, 0, $borderColor);

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

        // Add the transparent polygon mask
        $transparency = imagecolortransparent($maskPolygon, ColorUtils::colorIdentifier($maskPolygon, new Color('ff01fe')));

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

        imagesavealpha($maskPolygon, true);

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

        imagefilledpolygon($maskPolygon, $options->toArray(), $options->countPoints(), $transparency);

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

        // Apply the mask
        imagesavealpha($mergeImage, true);
        imagecopymerge($mergeImage, $maskPolygon, 0, 0, 0, 0, $image->getWidth(), $image->getHeight(), 100);

It seems like $mergeImage can also be of type false; however, parameter $dst_im of imagecopymerge() does only seem to accept resource, maybe add an additional type check?

It seems like $maskPolygon can also be of type false; however, parameter $src_im of imagecopymerge() does only seem to accept resource, maybe add an additional type check?

        // Create the final image
        $destImage = ImageCreateTrueColor($image->getWidth(), $image->getHeight());
        imagesavealpha($destImage, true);
        imagealphablending($destImage, true);

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

        imagecopy($destImage, $mergeImage,

It seems like $mergeImage can also be of type false; however, parameter $src_im of imagecopy() does only seem to accept resource, maybe add an additional type check?

It seems like $destImage can also be of type false; however, parameter $dst_im of imagecopy() does only seem to accept resource, maybe add an additional type check?

            0, 0,
            0, 0,
            $image->getWidth(), $image->getHeight());

        // Make the the border transparent (we're assuming there's a 2px buffer on all sides)

        $borderTransparency = ColorUtils::colorIdentifier($destImage, new Color('ff01fe'), true);
        imagesavealpha($destImage, true);
        imagealphablending($destImage, true);
        imagefill($destImage, 0, 0, $borderTransparency);

        $image->assignResource($destImage);

        $builder->crop($options->getMinX(), $options->getMinY(), $options->getCropDimention()->getWidth(), $options->getCropDimention()->getHeight());

        return $image;
    }