LoginCidadao\ValidationBundle\Validator\Constraints\AgeValidator::validate()

Complexity

Conditions	9
Paths	12

Size

Total Lines	37
Code Lines	24

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function validate($value, Constraint $constraint)
    {
        if (!$constraint instanceof Age) {
            throw new UnexpectedTypeException($constraint, __NAMESPACE__.'\Age');
        }
        if (!$value instanceof \DateTime || !$constraint instanceof Age) {
            throw new UnexpectedTypeException($value, '\DateTime');
        }

        $now = new \DateTime();
        $minAgeDate = new \DateTime("-{$constraint->min} years");
        $maxAgeDate = new \DateTime("-{$constraint->max} years");

        $error = false;
        $limit = 0;

        if ($value > $now) {
            $error = $constraint->futureError;
        } else {
            if ($constraint->min > 0 && $value > $minAgeDate) {
                $error = $constraint->minError;
                $limit = $constraint->min;
            }

            if ($value < $maxAgeDate) {
                $error = $constraint->maxError;
                $limit = $constraint->max;
            }
        }

        if ($error) {

The expression $error of type string|false is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== false instead.

            $this->context
                ->buildViolation($error)
                ->setParameter('{{ limit }}', $limit)
                ->addViolation();
        }
    }
}