Application\Media\Row

Complexity

Total Complexity

12

Size/Duplication

Total Lines	96
Duplicated Lines	11.46 %

Coupling/Cohesion

Components	2
Dependencies	5

Test Coverage

Coverage

61.76%

Importance

Changes	3
Bugs	1	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	beforeSave()	0	7	1
A	createThumbnail()	0	13	1
B	deleteFiles()	6	9	5
A	beforeInsert()	5	18	3
A	beforeUpdate()	0	4	1
A	afterDelete()	0	4	1

<?php
/**
 * @copyright Bluz PHP Team
 * @link https://github.com/bluzphp/skeleton
 */

/**
 * @namespace
 */
namespace Application\Media;

use Application\Users;
use Bluz\Proxy\Auth;
use Bluz\Validator\Traits\Validator;
use Bluz\Validator\Validator as v;
use Image\Thumbnail;

/**
 * Media Row
 *
 * @category Application
 * @package  Media
 *
 * @property integer $id
 * @property integer $userId
 * @property string $title
 * @property string $module
 * @property string $type
 * @property string $file
 * @property string $thumb
 * @property integer $size
 * @property string $created
 * @property string $updated
 */
class Row extends \Bluz\Db\Row
{
    use Validator;

    const THUMB_HEIGHT = 196;
    const THUMB_WIDTH = 196;

    /**
     * {@inheritdoc}
     */
    protected function beforeSave()
    {
        $this->addValidator(
            'title',
            v::required()->latinNumeric(' -_.')
        );
    }

    /**
     * Create thumbnail
     *
     * @return string
     */
    public function createThumbnail()
    {
        // set full path
        $image = new Thumbnail(PATH_PUBLIC .'/'. $this->file);
        $image->setHeight(self::THUMB_HEIGHT);
        $image->setWidth(self::THUMB_WIDTH);
        $thumb = $image->generate();
        // crop full path
        $thumb = substr($thumb, strlen(PATH_PUBLIC) + 1);
        $this->thumb = $thumb;

        return $thumb;
    }

    /**
     * Delete Files
     *
     * @return void
     */
    public function deleteFiles()
    {
        if ($this->file && is_file(PATH_PUBLIC .'/'. $this->file)) {

This code seems to be duplicated across your project.

            @unlink(PATH_PUBLIC .'/'. $this->file);

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

        }
        if ($this->thumb && is_file(PATH_PUBLIC .'/'. $this->thumb)) {

This code seems to be duplicated across your project.

            @unlink(PATH_PUBLIC .'/'. $this->thumb);

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

        }
    }

    /**
     * __insert
     *
     * @return void
     */
    protected function beforeInsert()
    {
        $this->created = gmdate('Y-m-d H:i:s');

        // set default module
        if (!$this->module) {
            $this->module = 'users';
        }
        // set user ID
        if ($user = Auth::getIdentity()) {

This code seems to be duplicated across your project.

            $this->userId = $user->id;

Accessing id on the interface Bluz\Auth\EntityInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

        } else {
            $this->userId = Users\Table::SYSTEM_USER;
        }

        // create thumbnail
        $this->createThumbnail();
    }

    /**
     * __update
     *
     * @return void
     */
    protected function beforeUpdate()
    {
        $this->updated = gmdate('Y-m-d H:i:s');
    }

    /**
     * postDelete
     *
     * @return void
     */
    protected function afterDelete()
    {
        $this->deleteFiles();
    }
}