LAG\AdminBundle\Validation\Validator\AdminValidator

Complexity

Total Complexity

5

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

1 Method

Rating	Name	Duplication	Size	Complexity
A	validate()	0	19	5

<?php

declare(strict_types=1);

namespace LAG\AdminBundle\Validation\Validator;

use LAG\AdminBundle\Metadata\AdminResource;
use Symfony\Component\Validator\Constraint;
use Symfony\Component\Validator\ConstraintValidator;
use Symfony\Component\Validator\Exception\UnexpectedTypeException;

class AdminValidator extends ConstraintValidator
{
    public function validate(mixed $value, Constraint $constraint): void
    {
        if (!$value instanceof AdminResource) {
            throw new UnexpectedTypeException($value, AdminResource::class);
        }

        if (!$value->getName()) {
            $this->context->addViolation('The admin has an empty name');
        }

        if (!$value->getTitle()) {
            $this->context->addViolation('The admin has an empty title');
        }

        if (!class_exists($value->getDataClass())) {

It seems like $value->getDataClass() can also be of type null; however, parameter $class of class_exists() does only seem to accept string, maybe add an additional type check?

            $this->context->addViolation(sprintf(
                'The admin "%s" has an invalid data class "%s"',
                $value->getName(),
                $value->getDataClass(),
            ));
        }
    }
}