LM\AuthAbstractor\Form\Constraint\ValidNewPasswordValidator

Complexity

Total Complexity

12

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	validate()	0	35	11
A	addError()	0	7	1

<?php

declare(strict_types=1);

namespace LM\AuthAbstractor\Form\Constraint;

use Symfony\Component\Validator\Constraint;
use Symfony\Component\Validator\ConstraintValidator;

/**
 * The validator class associated with ValidNewPassword.
 *
 * @see \LM\AuthAbstractor\Form\Constraint\ValidNewPassword
 */
class ValidNewPasswordValidator extends ConstraintValidator
{
    /**
     * Validates a given password and adds errors to the form if it has errors.
     *
     * @internal
     */
    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;
            }
        }
    }

    /**
     * @internal
     */
    private function addError(string $message, string $password): void
    {
        $this
            ->context
            ->buildViolation($message)
            ->setParameter('{{ string }}', $password)
            ->addViolation()
        ;
    }
}