floor12\files\logic\FileCropRotate

Complexity

Total Complexity

14

Size/Duplication

Total Lines	106
Duplicated Lines	0 %

Importance

Changes	3
Bugs	2	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	21	6
A	execute()	0	32	2
A	imageCreateFromAny()	0	28	6

<?php
/**
 * Created by PhpStorm.
 * User: floor12
 * Date: 03.01.2018
 * Time: 19:37
 */

namespace floor12\files\logic;


use floor12\files\models\File;
use floor12\files\models\FileType;
use Yii;
use yii\base\ErrorException;
use yii\web\BadRequestHttpException;
use yii\web\NotFoundHttpException;

class FileCropRotate
{
    private $_file;
    private $_width;
    private $_height;
    private $_top;
    private $_left;
    private $_rotated;


    public function __construct(array $data)
    {
        $this->_file = File::findOne($data['id']);

        if (!$this->_file)
            throw new NotFoundHttpException('File not found');

        if ($this->_file->type != FileType::IMAGE)
            throw new BadRequestHttpException('Requested file is not an image.');

        if (!file_exists($this->_file->rootPath))

It seems like $this->_file->rootPath can also be of type null; however, parameter $filename of file_exists() does only seem to accept string, maybe add an additional type check?

            throw new BadRequestHttpException('File not found in file storage.');

        $this->_height = (int)$data['height'];
        $this->_width = (int)$data['width'];
        $this->_top = (int)$data['top'];
        $this->_left = (int)$data['left'];
        $this->_rotated = (int)($data['rotated'] ?? 0);

        if (!$this->_height && !$this->_width) {
            list($this->_width, $this->_height) = getimagesize($this->_file->rootPath);

It seems like $this->_file->rootPath can also be of type null; however, parameter $filename of getimagesize() does only seem to accept string, maybe add an additional type check?

        }

    }

    public function execute()
    {

        $src = $this->imageCreateFromAny();

        $src = imagerotate($src, -$this->_rotated, 0);

        $dest = imagecreatetruecolor($this->_width, $this->_height);

        imagecopy($dest, $src, 0, 0, $this->_left, $this->_top, $this->_width, $this->_height);

        $newName = new PathGenerator(Yii::$app->getModule('files')->storageFullPath) . '.jpeg';

        $newPath = Yii::$app->getModule('files')->storageFullPath . '/' . $newName;

Are you sure Yii::app->getModule('files')->storageFullPath of type mixed|null|object can be used in concatenation?

        $oldPath = $this->_file->rootPath;

        imagejpeg($dest, $newPath, 80);

        imagedestroy($dest);

        imagedestroy($src);

        $this->_file->filename = $newName;
        $this->_file->content_type = $this->_file->mime_content_type($newPath);
        $this->_file->size = filesize($newPath);
        $this->_file->changeHash();
        if ($this->_file->save()) {
            @unlink($oldPath);

It seems like $oldPath can also be of type null; however, parameter $filename of unlink() does only seem to accept string, maybe add an additional type check?

It seems like you do not handle an error condition for unlink(). This can introduce security issues, and is generally not recommended.

            return $this->_file->href;
        } else
            throw new ErrorException("Error while saving file model.");


    }


    /**
     * Method to read files from any mime types
     * @return resource
     * @throws BadRequestHttpException
     */

    private function imageCreateFromAny()
    {
        $type = exif_imagetype($this->_file->rootPath);

It seems like $this->_file->rootPath can also be of type null; however, parameter $filename of exif_imagetype() does only seem to accept string, maybe add an additional type check?

        $allowedTypes = array(
            1, // [] gif
            2, // [] jpg
            3, // [] png
            6   // [] bmp
        );
        if (!in_array($type, $allowedTypes)) {
            throw new BadRequestHttpException('File must have GIF, JPG, PNG or BMP mime-type.');

        }
        switch ($type) {
            case 1 :
                $im = imageCreateFromGif($this->_file->rootPath);
                break;
            case 2 :
                $im = imageCreateFromJpeg($this->_file->rootPath);
                break;
            case 3 :
                $im = imageCreateFromPng($this->_file->rootPath);
                break;
            case 6 :
                $im = imageCreateFromBmp($this->_file->rootPath);
                break;
        }
        return $im;

The variable $im does not seem to be defined for all execution paths leading up to this point.

    }
}