LM\AuthAbstractor\Form\Constraint\ValidNewPasswordValidator::validate()

Complexity

Conditions	11
Paths	111

Size

Total Lines	35
Code Lines	25

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function validate($password, Constraint $constraint)
    {
        $pwdValidator = $constraint->getPwdValidator();

The method getPwdValidator() does not exist on Symfony\Component\Validator\Constraint. It seems like you code against a sub-type of Symfony\Component\Validator\Constraint such as LM\AuthAbstractor\Form\Constraint\ValidNewPassword.

        $pwdConfig = $constraint->getConfig()->getPwdSettings();

The method getConfig() does not exist on Symfony\Component\Validator\Constraint. It seems like you code against a sub-type of Symfony\Component\Validator\Constraint such as LM\AuthAbstractor\Form\Constraint\ValidNewPassword.

        if (true === $pwdConfig['enforce_min_length']) {
            $pwdMinLength = $pwdConfig['min_length'];
            if (mb_strlen($password, 'utf-8') < $pwdMinLength) {
                $this->addError("Your password needs to be at least {$pwdMinLength} characters long", $password);
            }
        }
        if (true === $pwdConfig['numbers']) {
            switch (preg_match('/[0-9]/', $password)) {
                case 0:
                    $this->addError('Your password needs to contain numbers.', $password);
                    break;

                case false:

It seems like you are loosely comparing preg_match('/[0-9]/', $password) of type integer to the boolean false. If you are specifically checking for 0, consider using something more explicit like === 0 instead.

                    throw new Exception();

The type LM\AuthAbstractor\Form\Constraint\Exception was not found. Did you mean Exception? If so, make sure to prefix the type with \.

                    break;
            }
        }
        if (true === $pwdConfig['special_chars']) {
            if (false === $pwdValidator->hasSpecialChars($password)) {
                $this->addError('Your password needs to contain special characters', $password);
            }
        }
        if (true === $pwdConfig['uppercase']) {
            switch (preg_match('/[A-Z]/', $password)) {
                case 0:
                    $this->addError('Your password needs to contain uppercase letters.', $password);
                    break;

                case false:

It seems like you are loosely comparing preg_match('/[A-Z]/', $password) of type integer to the boolean false. If you are specifically checking for 0, consider using something more explicit like === 0 instead.

                    throw new Exception();
                    break;
            }