Apps\Model\Front\Profile\FormAvatarUpload

Complexity

Total Complexity

8

Size/Duplication

Total Lines	84
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	labels()	0	6	1
A	rules()	0	8	1
A	copyFile()	0	17	3
A	resizeAndSave()	0	21	2
A	sources()	0	6	1

<?php

namespace Apps\Model\Front\Profile;

use Ffcms\Core\App;
use Ffcms\Core\Arch\Model;
use Ffcms\Core\Interfaces\iUser;
use Gregwar\Image\Image;

class FormAvatarUpload extends Model
{
    /** @var \Symfony\Component\HttpFoundation\File\UploadedFile */
    public $file;

    const AVATAR_SIZE = 2097152; // 2mb
    const COMPRESS_QUALITY = 90;

    /**
    * Example of usage magic labels for future form helper usage
    */
    public function labels()
    {
        return [
            'file' => __('Select avatar')
        ];
    }

    /**
    * Example of usage magic rules for future usage in condition $model->validate()
    */
    public function rules()
    {
        return [
            ['file', 'required'],
            ['file', 'isFile', ['jpg', 'png', 'gif', 'jpeg']],
            ['file', 'sizeFile', [1, static::AVATAR_SIZE]]
        ];
    }

    public function sources()
    {
        return [
            'file' => 'file'
        ];
    }

    public function copyFile(iUser $user)
    {
        // move file to original folder
        $upload = $this->file->move(root . '/upload/user/avatar/original/', $user->id . '.' . $this->file->guessExtension());

Accessing id on the interface Ffcms\Core\Interfaces\iUser suggest that you code against a concrete implementation. How about adding an instanceof check?

        try {
            // big image
            $this->resizeAndSave($upload, $user->id, 'big');

Accessing id on the interface Ffcms\Core\Interfaces\iUser suggest that you code against a concrete implementation. How about adding an instanceof check?

$upload of type object<Symfony\Component\HttpFoundation\File\File> is not a sub-type of object<Symfony\Component\HttpFoundation\File\UploadedFile>. It seems like you assume a child class of the class Symfony\Component\HttpFoundation\File\File to be always present.

            $this->resizeAndSave($upload, $user->id, 'medium');

Accessing id on the interface Ffcms\Core\Interfaces\iUser suggest that you code against a concrete implementation. How about adding an instanceof check?

$upload of type object<Symfony\Component\HttpFoundation\File\File> is not a sub-type of object<Symfony\Component\HttpFoundation\File\UploadedFile>. It seems like you assume a child class of the class Symfony\Component\HttpFoundation\File\File to be always present.

            $this->resizeAndSave($upload, $user->id, 'small');

Accessing id on the interface Ffcms\Core\Interfaces\iUser suggest that you code against a concrete implementation. How about adding an instanceof check?

$upload of type object<Symfony\Component\HttpFoundation\File\File> is not a sub-type of object<Symfony\Component\HttpFoundation\File\UploadedFile>. It seems like you assume a child class of the class Symfony\Component\HttpFoundation\File\File to be always present.

        } catch (\Exception $e) {
            if (App::$Debug) {
                App::$Debug->addException($e);
            }
        }
    }

    /**
     * @param \Symfony\Component\HttpFoundation\File\UploadedFile $original
     * @param int $user_id
     * @param string $size
     * @throws \Exception
     * @return null
     */
    protected function resizeAndSave($original, $user_id, $size = 'small')
    {
        $sizeConvert = [
            'big' => [400, 400],
            'medium' => [200, 200],
            'small' => [100, 100]
        ];

        if (!array_key_exists($size, $sizeConvert)) {
            return null;
        }

        $image = new Image();
        $image->setCacheDir(root . '/Private/Cache/images');

        $image->open($original->getPathname())
            ->cropResize($sizeConvert[$size][0], $sizeConvert[$size][1])
            ->save(root . '/upload/user/avatar/' . $size . '/' . $user_id . '.jpg', 'jpg', static::COMPRESS_QUALITY);

        return null;
    }
}