KunicMarko\SonataAnnotationBundle\Reader\ShowReader

Complexity

Total Complexity

5

Size/Duplication

Total Lines	15
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	configureFields()	0	11	5

<?php

declare(strict_types=1);

namespace KunicMarko\SonataAnnotationBundle\Reader;

use KunicMarko\SonataAnnotationBundle\Annotation\ShowField;
use Sonata\AdminBundle\Show\ShowMapper;

/**
 * @author Marko Kunic <kunicmarko20@gmail.com>
 */
class ShowReader
{
    use AnnotationReaderTrait;

    public function configureFields(\ReflectionClass $class, ShowMapper $showMapper): void
    {
        foreach ($class->getProperties() as $property) {
            if ($annotation = $this->getPropertyAnnotation($property, ShowField::class)) {
                $showMapper->add($property->getName(), ...$annotation->getSettings());

$annotation->getSettings() is expanded, but the parameter $type of Sonata\AdminBundle\Show\ShowMapper::add() does not expect variable arguments.

The method getSettings() does not exist on KunicMarko\SonataAnnotationBundle\Annotation\AnnotationInterface. It seems like you code against a sub-type of KunicMarko\SonataAnnotationBundle\Annotation\AnnotationInterface such as KunicMarko\SonataAnnotationBundle\Annotation\AbstractField.

            }
        }

        foreach ($class->getMethods() as $method) {
            if ($annotation = $this->getMethodAnnotation($method, ShowField::class)) {
                $showMapper->add($method->getName(), ...$annotation->getSettings());
            }
        }
    }
}