KunicMarko\SonataAnnotationBundle\Reader\ListReader::configureFields()

Complexity

Conditions	8
Paths	30

Size

Total Lines	28
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function configureFields(\ReflectionClass $class, ListMapper $listMapper): void
    {
        foreach ($class->getProperties() as $property) {
            foreach ($this->getPropertyAnnotations($property) as $annotation) {
                if ($annotation instanceof ListAssociationField) {
                    $this->addField(
                        $property->getName() . '.' . $annotation->field,
                        $annotation,
                        $listMapper
                    );
                    continue;
                }

                if ($annotation instanceof ListField) {
                    $this->addField($property->getName(), $annotation, $listMapper);
                }
            }
        }

        foreach ($class->getMethods() as $method) {
            if ($annotation = $this->getMethodAnnotation($method, ListField::class)) {
                $this->addField($property->getName(), $annotation, $listMapper);

The variable $property seems to be defined by a foreach iteration on line 21. Are you sure the iterator is never empty, otherwise this variable is not defined?

            }
        }

        if ($actions = $this->getListActions($this->getClassAnnotations($class))) {
            $listMapper->add('_action', null, [
                'actions' => $actions
            ]);