LAG\AdminBundle\Validation\Validator\AdminValidator::validate()

Complexity

Conditions	5
Paths	9

Size

Total Lines	19
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

<?php
    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(),
            ));