AdminUser::setCheckboxes() - Code Metrics - alexander-torosh/yona-cms - Measure and Improve Code Quality continuously with Scrutinizer

AdminUser::setCheckboxes() A
last analyzed 2019-11-25 19:20 UTC

↳ Parent: AdminUser

Complexity

Conditions	2
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	4
rs	10
c	0
b	0
f	0
cc	2
nc	1
nop	1

<?php
/**
 * @copyright Copyright (c) 2011 - 2014 Aleksandr Torosh (http://wezoom.com.ua)
 * @author Aleksandr Torosh <[email protected]>
 */

namespace Admin\Model;
use Phalcon\Validation;
use Phalcon\Validation\Validator\Email as EmailValidator;
use Phalcon\Validation\Validator\Uniqueness as UniquenessValidator;
use stdClass;

class AdminUser extends \Phalcon\Mvc\Model
{

    public function getSource()
    {
        return "admin_user";
    }

    public $id;
    public $role;
    public $login;
    public $email;
    public $name;
    public $password;
    public $active = 0;

    public static $roles = [
        'journalist' => 'Journalist',
        'editor'     => 'Editor',
        'admin'      => 'Admin',
    ];

    public function initialize()
    {
        
    }

    public function setCheckboxes($post)
    {
        $this->setActive(isset($post['active']) ? 1 : 0);
    }

    public function validation()
    {

       $validator = new Validation();
       $validator->add('login', new UniquenessValidator(
           [
               "model"   => $this,
               "message" => $this->getDi()->get('helper')->translate("The Login must be unique")

           ]
       ));
       $validator->add('email', new UniquenessValidator(
           [
               "model"   => $this,
               "message" => $this->getDi()->get('helper')->translate("The Email must be unique")

           ]
       ));
       return $this->validate($validator);
    }

    public function getId()
    {
        return $this->id;
    }

    public function getLogin()
    {
        return $this->login;
    }

    public function getEmail()
    {
        return $this->email;
    }

    public function getRole()
    {
        return $this->role;
    }

    public function getRoleTitle()
    {
        if (array_key_exists($this->role, self::$roles)) {
            return self::$roles[$this->role];
        }
    }

    public function setRole($role)
    {
        $this->role = $role;
    }

    public function getName()
    {
        return $this->name;
    }


    public function setName($name)
    {
        $this->name = $name;
    }

    public function getPassword()
    {
        return ''; // We don't need hash of password. Just return empty string.
    }

    public function checkPassword($password)
    {
        if (password_verify($password, $this->password)) {
            return true;
        }
    }

    public function getActive()
    {
        return $this->active;
    }

    public function isActive()
    {
        if ($this->active) {
            return true;
        }
    }

    public function setLogin($login)
    {
        $this->login = $login;
    }

    public function setEmail($email)
    {
        $this->email = $email;
    }

    public function setPassword($password)
    {
        if ($password) {
            $this->password = password_hash($password, PASSWORD_DEFAULT);
        }
    }

    public function setActive($active)
    {
        $this->active = $active;
    }

    public function getAuthData()
    {
        $authData = new stdClass();
        $authData->id = $this->getId();
        $authData->admin_session = true;
        $authData->login = $this->getLogin();
        $authData->email = $this->getEmail();
        $authData->name = $this->getName();
        return $authData;
    }

    public static function getRoleById($id)
    {
        $role = self::findFirst([
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
            'conditions' => 'id = :id:',
            'bind'       => ['id' => $id],
            'columns'    => ['role'],
            'cache'      => [
                'key'      => HOST_HASH . md5('Admin\Model\AdminUser::getRoleById::' . $id),
                'lifetime' => 60,
            ]
        ]);
        if ($role) {
            return $role->role;
        } else {
            return 'guest';
        }

    }

}


1			<?php
2			/**
3			* @copyright Copyright (c) 2011 - 2014 Aleksandr Torosh (http://wezoom.com.ua)
4			* @author Aleksandr Torosh <[email protected]>
5			*/
6
7			namespace Admin\Model;
8			use Phalcon\Validation;
9			use Phalcon\Validation\Validator\Email as EmailValidator;
10			use Phalcon\Validation\Validator\Uniqueness as UniquenessValidator;
11			use stdClass;
12
13			class AdminUser extends \Phalcon\Mvc\Model
14			{
15
16			public function getSource()
17			{
18			return "admin_user";
19			}
20
21			public $id;
22			public $role;
23			public $login;
24			public $email;
25			public $name;
26			public $password;
27			public $active = 0;
28
29			public static $roles = [
30			'journalist' => 'Journalist',
31			'editor' => 'Editor',
32			'admin' => 'Admin',
33			];
34
35			public function initialize()
36			{
37
38			}
39
40			public function setCheckboxes($post)
41			{
42			$this->setActive(isset($post['active']) ? 1 : 0);
43			}
44
45			public function validation()
46			{
47
48			$validator = new Validation();
49			$validator->add('login', new UniquenessValidator(
50			[
51			"model" => $this,
52			"message" => $this->getDi()->get('helper')->translate("The Login must be unique")
			0 ignored issues – show Bug introduced 2018-03-22 08:36 UTC by Report Bug Copy Issue Report The method `get` cannot be called on `$this->getDi()` (of type `null`). Methods can only be called on objects. This check looks for methods being called on variables that have been inferred to never be objects. Loading history...
53			]
54			));
55			$validator->add('email', new UniquenessValidator(
56			[
57			"model" => $this,
58			"message" => $this->getDi()->get('helper')->translate("The Email must be unique")
			0 ignored issues – show Bug introduced 2018-03-22 08:36 UTC by Report Bug Copy Issue Report The method `get` cannot be called on `$this->getDi()` (of type `null`). Methods can only be called on objects. This check looks for methods being called on variables that have been inferred to never be objects. Loading history...
59			]
60			));
61			return $this->validate($validator);
62			}
63
64			public function getId()
65			{
66			return $this->id;
67			}
68
69			public function getLogin()
70			{
71			return $this->login;
72			}
73
74			public function getEmail()
75			{
76			return $this->email;
77			}
78
79			public function getRole()
80			{
81			return $this->role;
82			}
83
84			public function getRoleTitle()
85			{
86			if (array_key_exists($this->role, self::$roles)) {
87			return self::$roles[$this->role];
88			}
89			}
90
91			public function setRole($role)
92			{
93			$this->role = $role;
94			}
95
96			public function getName()
97			{
98			return $this->name;
99			}
100
101
102			public function setName($name)
103			{
104			$this->name = $name;
105			}
106
107			public function getPassword()
108			{
109			return ''; // We don't need hash of password. Just return empty string.
110			}
111
112			public function checkPassword($password)
113			{
114			if (password_verify($password, $this->password)) {
115			return true;
116			}
117			}
118
119			public function getActive()
120			{
121			return $this->active;
122			}
123
124			public function isActive()
125			{
126			if ($this->active) {
127			return true;
128			}
129			}
130
131			public function setLogin($login)
132			{
133			$this->login = $login;
134			}
135
136			public function setEmail($email)
137			{
138			$this->email = $email;
139			}
140
141			public function setPassword($password)
142			{
143			if ($password) {
144			$this->password = password_hash($password, PASSWORD_DEFAULT);
145			}
146			}
147
148			public function setActive($active)
149			{
150			$this->active = $active;
151			}
152
153			public function getAuthData()
154			{
155			$authData = new stdClass();
156			$authData->id = $this->getId();
157			$authData->admin_session = true;
158			$authData->login = $this->getLogin();
159			$authData->email = $this->getEmail();
160			$authData->name = $this->getName();
161			return $authData;
162			}
163
164			public static function getRoleById($id)
165			{
166			$role = self::findFirst([
			0 ignored issues – show Bug introduced 2018-03-22 08:36 UTC by Report Bug Copy Issue Report Are you sure the assignment to `$role` is correct as `self::findFirst(array('c...d), 'lifetime' => 60)))` (which targets `Phalcon\Mvc\Model::findFirst()`) seems to always return null. This check looks for function or method calls that always return null and whose return value is assigned to a variable. class A { function getObject() { return null; } } $a = new A(); $object = $a->getObject(); The method `getObject()` can return nothing but null, so it makes no sense to assign that value to a variable. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
167			'conditions' => 'id = :id:',
168			'bind' => ['id' => $id],
169			'columns' => ['role'],
170			'cache' => [
171			'key' => HOST_HASH . md5('Admin\Model\AdminUser::getRoleById::' . $id),
172			'lifetime' => 60,
173			]
174			]);
175			if ($role) {
176			return $role->role;
177			} else {
178			return 'guest';
179			}
180
181			}
182
183			}
184

alexander-torosh / yona-cms

AdminUser::setCheckboxes() A last analyzed 2019-11-25 19:20 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

AdminUser::setCheckboxes() A
last analyzed 2019-11-25 19:20 UTC