User::save() - Code Metrics - MuriloChianfa/Redstore-Microservice-Based - Measure and Improve Code Quality continuously with Scrutinizer

User::save() B
last analyzed 2023-05-24 06:00 UTC

↳ Parent: User

Complexity

Conditions	10
Paths	13

Size

Total Lines	55
Code Lines	34

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	34
dl	0
loc	55
rs	7.6666
c	1
b	0
f	0
cc	10
nc	13
nop	0

How to fix Long Method Complexity

<?php

namespace Source\Models;

use Source\Core\Model;

/**
 * @package Source\Models
 */
class User extends Model
{
    /**
     * User constructor.
     */
    public function __construct()
    {
        parent::__construct("user", ["id"], ["first_name", "last_name", "email", "password"]);
    }

    /**
     * @param string $name
     * @param string $email
     * @param string $password
     * @param string|null $document
     * @return User
     */
    public function bootstrap(
        string $first_name,
        string $last_name,
        string $email,
        string $password
    ): User {
        $this->first_name = $first_name;

        $this->last_name = $last_name;

        $this->email = $email;

        $this->password = $password;

        $this->cpf = 00000000000;

        return $this;
    }

    /**
     * @param string $email
     * @param string $columns
     * @return null|User
     */
    public function findByEmail(string $email, string $columns = "*"): ?User
    {
        $find = $this->find("email = :email", "email={$email}", $columns);

        return $find->fetch();
    }

    public function updateUser(): bool
    {
        $userId = $this->id;


        if ($this->find("email = :e AND id != :i", "e={$this->email}&i={$userId}", "id")->fetch()) {
            $this->error = "O e-mail informado já está cadastrado";
            return false;
        }

        $this->update($this->safe(), "id = :id", "id={$userId}");

        if ($this->fail()) {
            $this->error = "Erro ao atualizar, verifique os dados";
            return false;
        }

        return true;
    }

    /**
     * @return bool
     */
    public function save(): bool
    {
        if (!$this->required()) {
            $this->error = "Nome, email e senha são obrigatórios";
            return false;
        }

        if (!is_email($this->email)) {
            $this->error = "O e-mail informado não tem um formato válido";
            return false;
        }

        if (!is_passwd($this->password)) {
            $min = CONF_PASSWD_MIN_LEN;
            $max = CONF_PASSWD_MAX_LEN;
            $this->error = "A senha deve ter entre {$min} e {$max} caracteres";
            return false;
        } else {
            $this->password = passwd($this->password);

        }

        /** User Update */
        if (!empty($this->id)) {

            $userId = $this->id;

            if ($this->find("email = :e AND id != :i", "e={$this->email}&i={$userId}", "id")->fetch()) {
                $this->error = "O e-mail informado já está cadastrado";
                return false;
            }

            $this->update($this->safe(), "id = :id", "id={$userId}");
            if ($this->fail()) {
                error_log(($this->fail())->getMessage());
                $this->error = "Erro ao atualizar, verifique os dados";
                return false;
            }
        }

        /** User Create */
        if (empty($this->id)) {
            if ($this->findByEmail($this->email, 'id')) {
                $this->error = "O e-mail informado já está cadastrado";
                return false;
            }

            $userId = $this->create($this->safe());
            if ($this->fail()) {
                error_log(($this->fail())->getMessage());
                $this->error = "Erro ao cadastrar, verifique os dados";
                return false;
            }
        }

        $this->data = ($this->findById($userId))->data();

        return true;
    }
}


1			<?php
2
3			namespace Source\Models;
4
5			use Source\Core\Model;
6
7			/**
8			* @package Source\Models
9			*/
10			class User extends Model
11			{
12			/**
13			* User constructor.
14			*/
15			public function __construct()
16			{
17			parent::__construct("user", ["id"], ["first_name", "last_name", "email", "password"]);
18			}
19
20			/**
21			* @param string $name
22			* @param string $email
23			* @param string $password
24			* @param string\|null $document
25			* @return User
26			*/
27			public function bootstrap(
28			string $first_name,
29			string $last_name,
30			string $email,
31			string $password
32			): User {
33			$this->first_name = $first_name;
			0 ignored issues – show Bug Best Practice introduced 2021-09-13 19:00 UTC by Report Bug Copy Issue Report The property `first_name` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
34			$this->last_name = $last_name;
			0 ignored issues – show Bug Best Practice introduced 2021-09-13 19:00 UTC by Report Bug Copy Issue Report The property `last_name` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
35			$this->email = $email;
			0 ignored issues – show Bug Best Practice introduced 2021-09-13 19:00 UTC by Report Bug Copy Issue Report The property `email` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
36			$this->password = $password;
			0 ignored issues – show Bug Best Practice introduced 2021-09-13 19:00 UTC by Report Bug Copy Issue Report The property `password` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
37			$this->cpf = 00000000000;
			0 ignored issues – show Bug Best Practice introduced 2021-09-13 19:00 UTC by Report Bug Copy Issue Report The property `cpf` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
38			return $this;
39			}
40
41			/**
42			* @param string $email
43			* @param string $columns
44			* @return null\|User
45			*/
46			public function findByEmail(string $email, string $columns = "*"): ?User
47			{
48			$find = $this->find("email = :email", "email={$email}", $columns);
49
50			return $find->fetch();
51			}
52
53			public function updateUser(): bool
54			{
55			$userId = $this->id;
			0 ignored issues – show Bug Best Practice introduced 2021-09-13 19:00 UTC by Report Bug Copy Issue Report The property `id` does not exist on `Source\Models\User`. Since you implemented `__get`, consider adding a @property annotation. Loading history...
56
57			if ($this->find("email = :e AND id != :i", "e={$this->email}&i={$userId}", "id")->fetch()) {
58			$this->error = "O e-mail informado já está cadastrado";
59			return false;
60			}
61
62			$this->update($this->safe(), "id = :id", "id={$userId}");
63
64			if ($this->fail()) {
65			$this->error = "Erro ao atualizar, verifique os dados";
66			return false;
67			}
68
69			return true;
70			}
71
72			/**
73			* @return bool
74			*/
75			public function save(): bool
76			{
77			if (!$this->required()) {
78			$this->error = "Nome, email e senha são obrigatórios";
79			return false;
80			}
81
82			if (!is_email($this->email)) {
83			$this->error = "O e-mail informado não tem um formato válido";
84			return false;
85			}
86
87			if (!is_passwd($this->password)) {
88			$min = CONF_PASSWD_MIN_LEN;
89			$max = CONF_PASSWD_MAX_LEN;
90			$this->error = "A senha deve ter entre {$min} e {$max} caracteres";
91			return false;
92			} else {
93			$this->password = passwd($this->password);
			0 ignored issues – show Bug Best Practice introduced 2021-09-13 19:00 UTC by Report Bug Copy Issue Report The property `password` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
94			}
95
96			/** User Update */
97			if (!empty($this->id)) {
			0 ignored issues – show Bug Best Practice introduced 2021-09-13 19:00 UTC by Report Bug Copy Issue Report The property `id` does not exist on `Source\Models\User`. Since you implemented `__get`, consider adding a @property annotation. Loading history...
98			$userId = $this->id;
99
100			if ($this->find("email = :e AND id != :i", "e={$this->email}&i={$userId}", "id")->fetch()) {
101			$this->error = "O e-mail informado já está cadastrado";
102			return false;
103			}
104
105			$this->update($this->safe(), "id = :id", "id={$userId}");
106			if ($this->fail()) {
107			error_log(($this->fail())->getMessage());
108			$this->error = "Erro ao atualizar, verifique os dados";
109			return false;
110			}
111			}
112
113			/** User Create */
114			if (empty($this->id)) {
115			if ($this->findByEmail($this->email, 'id')) {
116			$this->error = "O e-mail informado já está cadastrado";
117			return false;
118			}
119
120			$userId = $this->create($this->safe());
121			if ($this->fail()) {
122			error_log(($this->fail())->getMessage());
123			$this->error = "Erro ao cadastrar, verifique os dados";
124			return false;
125			}
126			}
127
128			$this->data = ($this->findById($userId))->data();
			0 ignored issues – show Comprehensibility Best Practice introduced 2021-09-13 19:00 UTC by Report Bug Copy Issue Report The variable `$userId` does not seem to be defined for all execution paths leading up to this point. Loading history...
129			return true;
130			}
131			}
132

MuriloChianfa / Redstore-Microservice-Based

User::save() B last analyzed 2023-05-24 06:00 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like

User::save() B
last analyzed 2023-05-24 06:00 UTC