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

Complexity

Conditions	8
Paths	11

Size

Total Lines	42
Code Lines	24

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    private function configureFields(\ReflectionClass $class, FormMapper $formMapper, string $action): void
    {
        $propertiesWithPosition = [];
        $propertiesWithoutPosition = [];

        foreach ($class->getProperties() as $property) {
            foreach ($this->getPropertyAnnotations($property) as $annotation) {
                if (!$annotation instanceof FormField || $annotation->action === $action) {
                    continue;
                }

                if (!$annotation->hasPosition()) {
                    $propertiesWithoutPosition[] = [
                        'name' => $property->getName(),
                        'settings' => $annotation->getSettings(),
                    ];

                    continue;
                }

                if (\array_key_exists($annotation->position, $propertiesWithPosition)) {
                    throw new \InvalidArgumentException(sprintf(
                        'Position "%s" is already in use by "%s", try setting a different position for "%s".',
                        $annotation->position,
                        $propertiesWithPosition[$annotation->position]['name'],
                        $property->getName()
                    ));
                }

                $propertiesWithPosition[$annotation->position] = [
                    'name' => $property->getName(),
                    'settings' => $annotation->getSettings(),
                ];
            }
        }

        \ksort($propertiesWithPosition);

        $properties = \array_merge($propertiesWithPosition, $propertiesWithoutPosition);

        foreach ($properties as $property) {
            $formMapper->add($property['name'], ...$property['settings']);

$property['settings'] is expanded, but the parameter $type of Sonata\AdminBundle\Form\FormMapper::add() does not expect variable arguments.

        }