LoginCidadao\ValidationBundle\Validator\Constraints\AgeValidator

Complexity

Total Complexity

8

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Test Coverage

Coverage

95.24%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
C	validate()	0	32	8

<?php
/**
 * This file is part of the login-cidadao project or it's bundles.
 *
 * (c) Guilherme Donato <guilhermednt on github>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace LoginCidadao\ValidationBundle\Validator\Constraints;

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

class AgeValidator extends ConstraintValidator
{
    public function validate($value, Constraint $constraint)
    {
        if (!$value instanceof \DateTime || !$constraint instanceof Age) {
            return;
        }

        $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 false|string 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();
        }
    }
}