Application\Users\Row

Complexity

Total Complexity

9

Size/Duplication

Total Lines	107
Duplicated Lines	0 %

Coupling/Cohesion

Components	3
Dependencies	7

Test Coverage

Coverage

14.89%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
B	beforeSave()	0	43	3
A	beforeInsert()	0	4	1
A	beforeUpdate()	0	4	1
A	getRoles()	0	4	1
A	getPrivileges()	0	7	2
A	hasRole()	0	6	1

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

declare(strict_types=1);

namespace Application\Users;

use Application\Privileges;
use Application\Roles;
use Bluz\Auth\AbstractRowEntity;
use Bluz\Validator\Traits\Validator;

/**
 * User
 *
 * @package  Application\Users
 *
 * @property integer $id
 * @property string  $login
 * @property string  $email
 * @property string  $created
 * @property string  $updated
 * @property string  $status
 *
 * @SWG\Definition(definition="users", title="user", required={"id", "login", "status"})
 * @SWG\Property(property="id", type="integer", description="User UID", example=2)
 * @SWG\Property(property="login", type="string", description="Login", example="admin")
 * @SWG\Property(property="email", type="string", description="Email", example="admin@domain.com")
 * @SWG\Property(property="created", type="string", format="date-time", example="2017-01-01 20:17:01")
 * @SWG\Property(property="updated", type="string", format="date-time", example="2017-01-01 20:17:01")
 * @SWG\Property(property="status", type="string", enum={"pending", "active", "disabled", "deleted"})
 */
class Row extends AbstractRowEntity

There is one abstract method tryLogin in this class; you could implement it, or declare this class as abstract.

{
    use Validator;

    /**
     * Small cache of user privileges
     *
     * @var array
     */
    protected $privileges;

    /**
     * @return void
     */
    public function beforeSave()
    {
        $this->email = strtolower($this->email ?? '');

        $this->addValidator('login')
            ->required()
            ->latin()
            ->length(3, 255)
            ->callback(
                function ($login) {
                    $selector = static::getTable()
                        ::select()
                        ->where('login = ?', $login);

                    if ($this->id) {
                        $selector->andWhere('id != ?', $this->id);
                    }

                    $user = $selector->execute();
                    return !$user;
                },
                'User with this login is already exists'
            );

        $this->addValidator('email')
            ->required()
            ->email(true)
            ->callback(
                function ($email) {
                    $selector = static::getTable()
                        ::select()
                        ->where('email = ?', $email);

                    if ($this->id) {
                        $selector->andWhere('id != ?', $this->id);
                    }

                    $user = $selector->execute();
                    return !$user;
                },
                'User with this email is already exists'
            );
    }

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

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

    /**
     * Get user roles
     */
    public function getRoles()
    {
        return Roles\Table::getInstance()->getUserRoles($this->id);
    }

    /**
     * {@inheritdoc}
     */
    public function getPrivileges(): array
    {
        if (!$this->privileges) {

The expression $this->privileges of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            $this->privileges = Privileges\Table::getInstance()->getUserPrivileges($this->id);
        }
        return $this->privileges;
    }

    /**
     * Check user role
     *
     * @param integer $roleId
     *
     * @return boolean
     */
    public function hasRole($roleId)
    {
        $roles = Roles\Table::getInstance()->getUserRolesIdentity($this->id);

        return in_array($roleId, $roles);
    }
}