coderkun/questlab
1
0
Fork 0
Code Issues 15 Pull requests Projects Releases Wiki Activity

do not use empty() without write context

Browse source
This commit is contained in:
coderkun 2014-05-01 21:26:36 +02:00
commit d39d915e03
1 changed files with 2 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
controllers/UsersController.inc
View file

@ -433,7 +433,8 @@
if($this->Users->usernameExists($username, $user['id'])) {
$validation = $this->Validation->addValidationResult($validation, 'username', 'exist', true);
}
if(!empty($this->request->getPostParam('password'))) {
$password = $this->request->getPostParam('password');
if(!empty($password)) {
$validation = $this->Validation->addValidationResults($validation,
'password',
$this->Validation->validateParam(