Reihenfolge beim Überprüfen neuer Passwörter

[vapita]

Hallo Tom,

vielen Dank für den Input. Ich hatte es dann wohl nicht ganz richtig:

$password = $userData['password'];
$passwordCheck = $userData['password_check'];
if($password == $passwordCheck){
    $passwordHashed = $this->passwordEncoder->hash($password);
    $newUser->setPassword($passwordHashed);
} else {
    $this->flash->add('Die Passwörter stimmen nicht überein!','danger');
    break;
}

Habe es nun nach deiner Empfehlung so geändert:

$password = $userData['password'];
$passwordCheck = $userData['password_check'];
if(!isset($password) or !isset($passwordCheck)){
    $this->flash->add('Es muss ein Passwort angegeben werden!','danger');
    break;
}
if($password === $passwordCheck){
    $passwordHashed = $this->passwordEncoder->hash($password);
    $newUser->setPassword($passwordHashed);
} else {
    $this->flash->add('Die Passwörter stimmen nicht überein!','danger');
    break;
}

Punkt 3 und 4 müsste ich noch erledigen.

Beste Grüße und vielen Dank

vapita