Spiral\Validation\Checkers\ImageChecker

Complexity

Total Complexity

17

Size/Duplication

Total Lines	162
Duplicated Lines	19.75 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	type()	0	12	3
A	valid()	0	4	1
B	smaller()	16	16	5
B	bigger()	16	16	5
A	imageData()	0	10	2

<?php
/**
 * Spiral Framework.
 *
 * @license   MIT
 * @author    Anton Titov (Wolfy-J)
 */

namespace Spiral\Validation\Checkers;

use Interop\Container\ContainerInterface;
use Spiral\Core\Container\SingletonInterface;
use Spiral\Files\FilesInterface;
use Spiral\Validation\Checkers\Traits\FileTrait;
use Spiral\Validation\Prototypes\AbstractChecker;

/**
 * Image based validations.
 */
class ImageChecker extends AbstractChecker implements SingletonInterface
{
    use FileTrait;

    /**
     * Getimagesize constants.
     */
    const WIDTH      = 0;
    const HEIGHT     = 1;
    const IMAGE_TYPE = 2;

    /**
     * {@inheritdoc}
     */
    protected $messages = [
        'type'    => '[[Image does not supported.]]',
        'valid'   => '[[Image does not supported (allowed JPEG, PNG or GIF).]]',
        'smaller' => '[[Image size should not exceed {0}x{1}px.]]',
        'bigger'  => '[[The image dimensions should be at least {0}x{1}px.]]',
    ];

    /**
     * Known image types.
     *
     * @var array
     */
    protected $imageTypes = [
        'null',
        'gif',
        'jpeg',
        'png',
        'swf',
        'psd',
        'bmp',
        'tiff',
        'tiff',
        'jpc',
        'jp2',
        'jpx',
        'jb2',
        'swc',
        'iff',
        'wbmp',
        'xbm',
    ];

    /**
     * @param FilesInterface     $files
     * @param ContainerInterface $container
     */
    public function __construct(FilesInterface $files, ContainerInterface $container = null)
    {
        $this->files = $files;

        parent::__construct($container);
    }

    /**
     * Check if image in a list of allowed image types.
     *
     * @param mixed       $filename
     * @param array|mixed $types
     *
     * @return bool
     */
    public function type($filename, $types): bool
    {
        if (empty($image = $this->imageData($filename))) {
            return false;
        }

        if (!is_array($types)) {
            $types = array_slice(func_get_args(), 1);
        }

        return in_array($this->imageTypes[$image[self::IMAGE_TYPE]], $types);
    }

    /**
     * Shortcut to check if image has valid type (JPEG, PNG and GIF are allowed).
     *
     * @param mixed $filename
     *
     * @return bool
     */
    public function valid($filename): bool
    {
        return $this->type($filename, ['jpeg', 'png', 'gif']);
    }

    /**
     * Check if image smaller that specified rectangle (height check if optional).
     *
     * @param mixed $filename
     * @param int   $width
     * @param int   $height Optional.
     *
     * @return bool
     */
    public function smaller($filename, int $width, int $height = null): bool

This method seems to be duplicated in your project.

    {
        if (empty($image = $this->imageData($filename))) {
            return false;
        }

        if ($image[self::WIDTH] >= $width) {
            return false;
        }

        if (!empty($height) && $image[self::HEIGHT] >= $height) {
            return false;
        }

        return true;
    }

    /**
     * Check if image is bigger that specified rectangle (height check is optional).
     *
     * @param mixed $filename
     * @param int   $width
     * @param int   $height Optional.
     *
     * @return bool
     */
    public function bigger($filename, int $width, int $height = null): bool

This method seems to be duplicated in your project.

    {
        if (empty($image = $this->imageData($filename))) {
            return false;
        }

        if ($image[self::WIDTH] < $width) {
            return false;
        }

        if (!empty($height) && $image[self::HEIGHT] < $height) {
            return false;
        }

        return true;
    }

    /**
     * Internal method, return image details fetched by getimagesize() or false.
     *
     * @see getimagesize()
     *
     * @param string|mixed $filename
     *
     * @return array|bool
     */
    protected function imageData($filename)
    {
        try {
            return getimagesize($this->filename($filename));
        } catch (\Exception $e) {
            //We can simply invalidate image if system can't read it
        }

        return false;
    }
}