Application\Roles\Row

Complexity

Total Complexity

3

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	3

Test Coverage

Coverage

6.9%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	beforeInsert()	0	12	1
B	beforeUpdate()	0	24	1
A	isBasic()	0	4	1

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

declare(strict_types=1);

namespace Application\Roles;

use Bluz\Validator\Traits\Validator;

/**
 * User Role
 *
 * @package Application\Roles
 *
 * @property integer $id
 * @property string  $name
 */
class Row extends \Bluz\Db\Row
{
    use Validator;

    /**
     * Before Insert/Update
     *
     * @return void
     */
    protected function beforeInsert()
    {
        $this->addValidator('name')

It seems like required() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            ->required()
            ->latin()
            ->callback(
                function ($name) {
                    return !Table::findRowWhere(['name' => $name]);
                }
            )
            ->setDescription('Role already exists');
    }

    /**
     * Before Update
     *
     * @return void
     */
    protected function beforeUpdate()
    {
        $this->addValidator('name')

It seems like required() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            ->required()
            ->latin()
            ->callback(
                function ($name) {
                    return !in_array(strtolower($name), Table::getInstance()->getBasicRoles(), false);
                },
                'Role is basic and can\'t be editable'
            )
            ->callback(
                function ($name) {
                    return $this->clean['name'] != $name;
                },
                'Role name is the same as original'
            )
            ->callback(
                function ($name) {
                    return !Table::findRowWhere(['name' => $name]);
                },
                'Role already exists'
            );
    }

    /**
     * isBasic
     *
     * @return boolean
     */
    public function isBasic() : bool
    {
        return in_array(strtolower($this->name), Table::getInstance()->getBasicRoles(), false);
    }
}