UserDAO::updatePassword() - Code Metrics - Inspection of "add melhorias em DAO e UserDAO" - winframework/winphp - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 774d12...7a0ba2 )

by Wanderson

created 2017-05-13 17:26 UTC

UserDAO::updatePassword() B

↳ Parent: UserDAO

Complexity

Conditions	5
Paths	3

Size

Total Lines	9
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	5
eloc	7
nc	3
nop	3
dl	0
loc	9
rs	8.8571
c	0
b	0
f	0

<?php

namespace Win\Authentication;

use Win\DAO\DAO;
use Win\Authentication\User;
use Win\Calendar\Date;

/**
 * User DAO
 */
class UserDAO extends DAO implements UserDAOInterface {

	const TABLE = 'person';
	const ALIAS = 'Usuário';

	/** @var User */
	protected $obj;

	/**
	 * @return string|null
	 */
	protected function validate() {
		if (strlen($this->obj->getName()) < 2) {
			return 'O campo Nome deve possuir pelo menos 2 caracteres.';
		} elseif (!$this->obj->accessIsDenied() && strlen($this->obj->getEmail()) == 0) {
			return 'O campo E-mail deve ser preenchido.';
		} elseif (!$this->obj->accessIsDenied() && !filter_var($this->obj->getEmail(), FILTER_VALIDATE_EMAIL)) {
			return 'O campo E-mail deve ser um e-mail válido.';
		} elseif (!$this->obj->accessIsDenied() && $this->obj->getConfirmEmail() !== null && $this->obj->getConfirmEmail() != $this->obj->getEmail()) {
			return 'O campo E-mail deve ser informado duas vezes iguais.';
		} elseif (strlen($this->obj->getEmail()) > 0 and $this->emailIsUsed()) {
			return 'Já existe um usuário com este e-mail.';
		} elseif (!$this->obj->accessIsDenied() && $this->obj->getPassword() !== null && strlen($this->obj->getPassword()) < 4) {
			return 'A senha deve possuir pelo menos 4 caracteres.';
		} elseif ($this->obj->getConfirmPassword() != $this->obj->getPassword()) {
			return 'O campo Senha deve ser informado duas vezes iguais.';
		}
		return null;
	}

	/**
	 * @param array $row
	 * @return User
	 */
	public static function mapObject($row) {
		$obj = new User();
		$obj->setId($row['person_id']);
		$obj->setEnabled($row['is_enabled']);
		$obj->setAccessLevel($row['access_level']);
		//$obj->setGroupId($row['group_id']);

		$obj->setName($row['name']);
		$obj->setEmail($row['email']);
		$obj->setConfirmEmail($row['email']);
		$obj->setPasswordHash($row['password_hash']);
		$obj->setRecoreryHash($row['recovery_hash']);
		$obj->getImage()->setName($row['image']);
		$obj->setLoginDate(new Date($row['login_date']));
		return $obj;
	}

	/**
	 * @param User $obj
	 * @return mixed[]
	 */
	public static function mapRow($obj) {
		$row['person_id'] = $obj->getId();
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
		$row['is_enabled'] = (int) $obj->isEnabled();
		$row['access_level'] = $obj->getAccessLevel();
		$row['name'] = strClear($obj->getName());
		$row['email'] = strClear($obj->getEmail());
		$row['image'] = $obj->getImage()->getName();
		$row['login_date'] = $obj->getLoginDate()->toSql();
		if (!is_null($obj->getPasswordHash())) {
			$row['password_hash'] = $obj->getPasswordHash();
		}
		if (!is_null($obj->getRecoreryHash())) {
			$row['recovery_hash'] = $obj->getRecoreryHash();
		}
		return $row;
	}

	/**
	 * Atualiza data ultimo login
	 * @param User $user
	 * @return string|null
	 */
	public function updateLoginDate(User $user) {
		$now = new Date();
		$userClone = clone $user;
		$userClone->setLoginDate($now);
		return $this->save($userClone);
	}

	/**
	 * Gera/Atualiza um novo recoveryHash
	 * @param User $user
	 * @return string|null
	 */
	public function updateRecoveryHash(User $user) {
		$hash = md5($user->getEmail() . date('Y-m-d'));
		$user->setRecoreryHash($hash);
		return $this->save($user);
	}

	/**
	 * Atualiza a senha | É necessário informar a senha atual, ou então o recoveryHash
	 * @param User $user
	 * @param string $currentPassword
	 * @param string $recoveryHash
	 * @return string|null
	 */
	public function updatePassword($user, $currentPassword = null, $recoveryHash = null) {
		$savedUser = $this->fetchById($user->getId());
		if (!is_null($currentPassword) and $savedUser->getPasswordHash() != User::encryptPassword($currentPassword)) {
			return 'A senha atual não está correta.';
		} elseif (!is_null($recoveryHash) and $user->getRecoreryHash() !== $recoveryHash) {
			return 'O link de recuperação é inválido.';
		}
		return $this->save($user);
	}

	/**
	 * Retorna true se já existe este email no sistema 
	 * @return boolean
	 */
	public function emailIsUsed() {
		return $this->numRows(['email = ?' => $this->obj->getEmail(), 'person_id <> ?' => $this->obj->getId()]);
	}

	public function fetchByRecoveryHash($recoveryHash) {
		return $this->fetch(['recovery_hash = ?' => $recoveryHash]);
	}

	public function onDelete() {
		$this->obj->getImage()->remove();
	}

	/**
	 * Insere o primeiro admin
	 * @param User $user
	 * @return string|null
	 */
	public function insertFirst(User $user) {
		$user->setName('Administrador');
		$user->setAccessLevel(User::ACCESS_ADMIN);
		$user->setConfirmEmail($user->getEmail());
		$user->setConfirmPassword($user->getPassword());

		if ($this->numRows() === 0) {
			return $this->save($user);
		}
	}

}


1			<?php
2
3			namespace Win\Authentication;
4
5			use Win\DAO\DAO;
6			use Win\Authentication\User;
7			use Win\Calendar\Date;
8
9			/**
10			* User DAO
11			*/
12			class UserDAO extends DAO implements UserDAOInterface {
13
14			const TABLE = 'person';
15			const ALIAS = 'Usuário';
16
17			/** @var User */
18			protected $obj;
19
20			/**
21			* @return string\|null
22			*/
23			protected function validate() {
24			if (strlen($this->obj->getName()) < 2) {
25			return 'O campo Nome deve possuir pelo menos 2 caracteres.';
26			} elseif (!$this->obj->accessIsDenied() && strlen($this->obj->getEmail()) == 0) {
27			return 'O campo E-mail deve ser preenchido.';
28			} elseif (!$this->obj->accessIsDenied() && !filter_var($this->obj->getEmail(), FILTER_VALIDATE_EMAIL)) {
29			return 'O campo E-mail deve ser um e-mail válido.';
30			} elseif (!$this->obj->accessIsDenied() && $this->obj->getConfirmEmail() !== null && $this->obj->getConfirmEmail() != $this->obj->getEmail()) {
31			return 'O campo E-mail deve ser informado duas vezes iguais.';
32			} elseif (strlen($this->obj->getEmail()) > 0 and $this->emailIsUsed()) {
33			return 'Já existe um usuário com este e-mail.';
34			} elseif (!$this->obj->accessIsDenied() && $this->obj->getPassword() !== null && strlen($this->obj->getPassword()) < 4) {
35			return 'A senha deve possuir pelo menos 4 caracteres.';
36			} elseif ($this->obj->getConfirmPassword() != $this->obj->getPassword()) {
37			return 'O campo Senha deve ser informado duas vezes iguais.';
38			}
39			return null;
40			}
41
42			/**
43			* @param array $row
44			* @return User
45			*/
46			public static function mapObject($row) {
47			$obj = new User();
48			$obj->setId($row['person_id']);
49			$obj->setEnabled($row['is_enabled']);
50			$obj->setAccessLevel($row['access_level']);
51			//$obj->setGroupId($row['group_id']);
			0 ignored issues – show Unused Code Comprehensibility introduced 2017-05-13 17:28 UTC by Report Bug Copy Issue Report `90%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
52			$obj->setName($row['name']);
53			$obj->setEmail($row['email']);
54			$obj->setConfirmEmail($row['email']);
55			$obj->setPasswordHash($row['password_hash']);
56			$obj->setRecoreryHash($row['recovery_hash']);
57			$obj->getImage()->setName($row['image']);
58			$obj->setLoginDate(new Date($row['login_date']));
59			return $obj;
60			}
61
62			/**
63			* @param User $obj
64			* @return mixed[]
65			*/
66			public static function mapRow($obj) {
67			$row['person_id'] = $obj->getId();
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-08-16 02:26 UTC by Report Bug Copy Issue Report `$row` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$row = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
68			$row['is_enabled'] = (int) $obj->isEnabled();
69			$row['access_level'] = $obj->getAccessLevel();
70			$row['name'] = strClear($obj->getName());
71			$row['email'] = strClear($obj->getEmail());
72			$row['image'] = $obj->getImage()->getName();
73			$row['login_date'] = $obj->getLoginDate()->toSql();
74			if (!is_null($obj->getPasswordHash())) {
75			$row['password_hash'] = $obj->getPasswordHash();
76			}
77			if (!is_null($obj->getRecoreryHash())) {
78			$row['recovery_hash'] = $obj->getRecoreryHash();
79			}
80			return $row;
81			}
82
83			/**
84			* Atualiza data ultimo login
85			* @param User $user
86			* @return string\|null
87			*/
88			public function updateLoginDate(User $user) {
89			$now = new Date();
90			$userClone = clone $user;
91			$userClone->setLoginDate($now);
92			return $this->save($userClone);
93			}
94
95			/**
96			* Gera/Atualiza um novo recoveryHash
97			* @param User $user
98			* @return string\|null
99			*/
100			public function updateRecoveryHash(User $user) {
101			$hash = md5($user->getEmail() . date('Y-m-d'));
102			$user->setRecoreryHash($hash);
103			return $this->save($user);
104			}
105
106			/**
107			* Atualiza a senha \| É necessário informar a senha atual, ou então o recoveryHash
108			* @param User $user
109			* @param string $currentPassword
110			* @param string $recoveryHash
111			* @return string\|null
112			*/
113			public function updatePassword($user, $currentPassword = null, $recoveryHash = null) {
114			$savedUser = $this->fetchById($user->getId());
115			if (!is_null($currentPassword) and $savedUser->getPasswordHash() != User::encryptPassword($currentPassword)) {
116			return 'A senha atual não está correta.';
117			} elseif (!is_null($recoveryHash) and $user->getRecoreryHash() !== $recoveryHash) {
118			return 'O link de recuperação é inválido.';
119			}
120			return $this->save($user);
121			}
122
123			/**
124			* Retorna true se já existe este email no sistema
125			* @return boolean
126			*/
127			public function emailIsUsed() {
128			return $this->numRows(['email = ?' => $this->obj->getEmail(), 'person_id <> ?' => $this->obj->getId()]);
129			}
130
131			public function fetchByRecoveryHash($recoveryHash) {
132			return $this->fetch(['recovery_hash = ?' => $recoveryHash]);
133			}
134
135			public function onDelete() {
136			$this->obj->getImage()->remove();
137			}
138
139			/**
140			* Insere o primeiro admin
141			* @param User $user
142			* @return string\|null
143			*/
144			public function insertFirst(User $user) {
145			$user->setName('Administrador');
146			$user->setAccessLevel(User::ACCESS_ADMIN);
147			$user->setConfirmEmail($user->getEmail());
148			$user->setConfirmPassword($user->getPassword());
149
150			if ($this->numRows() === 0) {
151			return $this->save($user);
152			}
153			}
154
155			}
156

winframework / winphp

Push — master ( 774d12...7a0ba2 )

UserDAO::updatePassword() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like