Happyr\Validator\Constraint\EntityExistValidator::validate()

Complexity

Conditions	6
Paths	5

Size

Total Lines	24
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	15
CRAP Score	6

Importance

Changes	2
Bugs	0	Features	0

<?php
    public function validate($value, Constraint $constraint): void
    {
        if (!$constraint instanceof EntityExist) {
            throw new \LogicException(\sprintf('You can only pass %s constraint to this validator.', EntityExist::class));
        }

        if (null === $value || '' === $value) {
            return;
        }

        if (empty($constraint->entity)) {
            throw new \LogicException(\sprintf('Must set "entity" on "%s" validator', EntityExist::class));
        }

        $data = $this->entityManager->getRepository($constraint->entity)->findOneBy([
            $constraint->property => $value,
        ]);

        if (null === $data) {
            $this->context->buildViolation($constraint->message)

It seems like $constraint->message can also be of type mixed; however, parameter $message of Symfony\Component\Validator\Context\ExecutionContextInterface::buildViolation() does only seem to accept string, maybe add an additional type check?

                ->setParameter('%entity%', $constraint->entity)

It seems like $constraint->entity can also be of type mixed; however, parameter $value of Symfony\Component\Validator\Violation\ConstraintViolationBuilderInterface::setParameter() does only seem to accept string, maybe add an additional type check?

                ->setParameter('%property%', $constraint->property)
                ->setParameter('%value%', (string) $value)
                ->addViolation();
        }