Blast\Bundle\CoreBundle\Admin\Traits\Mapper::addField()

Complexity

Conditions	26
Paths	18444

Size

Total Lines	95
Code Lines	51

Duplication

Lines	12
Ratio	12.63 %

Importance

Changes

0

<?php
    protected function addField(BaseMapper $mapper, $name, $options = [], $fieldDescriptionOptions = [])
    {
        // avoid duplicates
        if ($mapper->has($name)) {
            $mapper->remove($name);
        }

        if (!is_array($options)) {
            $options = [];
        }

        if (isset($options['only_new'])) {

This code seems to be duplicated across your project.

            if ($options['only_new'] && $this->subject && !$this->subject->isNew()) {

The property subject does not exist. Did you maybe forget to declare it?

                return $mapper;
            }
            unset($options['only_new']);
        }

        if (isset($options['only_not_new'])) {

This code seems to be duplicated across your project.

            if ($options['only_not_new'] && (!$this->subject || $this->subject->isNew())) {
                return $mapper;
            }
            unset($options['only_not_new']);
        }

        $type = null;
        if (isset($options['type'])) {
            $type = $options['type'];
            unset($options['type']);
        }

        if (isset($options['constraints'])) {
            foreach ($options['constraints'] as $k => $constraint) {
                $options['constraints'][$k] = new $constraint();
            }
        }

        if (isset($options['required']) && $options['required'] === true) {
            $options['constraints'] = [new NotBlank()];
        }

        if (isset($options['query'])) {
            $this->manageQueryCallback($mapper, $options);
        }

        if (isset($options['choicesCallback'])) {
            $this->manageChoicesCallback($mapper, $options);
        }

        if (isset($options['serviceCallback'])) {
            $this->manageServiceCallback($mapper, $options);
        }

        // save-and-remove CoreBundle-specific options
        $extras = [];
        foreach ([
            'template' => 'setTemplate',
            'initializeAssociationAdmin' => null,
        ] as $extra => $method) {
            if (isset($fieldDescriptionOptions[$extra])) {
                $extras[$extra] = [$method, $fieldDescriptionOptions[$extra]];
                unset($fieldDescriptionOptions[$extra]);
            }
        }

        $mapper->add($name, $type, $options, $fieldDescriptionOptions);

It seems like you code against a specific sub-type and not the parent class Sonata\AdminBundle\Mapper\BaseMapper as the method add() does only exist in the following sub-classes of Sonata\AdminBundle\Mapper\BaseMapper: Sonata\AdminBundle\Datagrid\DatagridMapper, Sonata\AdminBundle\Datagrid\ListMapper, Sonata\AdminBundle\Form\FormMapper, Sonata\AdminBundle\Show\ShowMapper. Maybe you want to instanceof check for one of these explicitly?

        // apply extra options
        foreach ($extras as $extra => $call) {
            if ($call[0]) {
                $mapper->get($name)->{$call[0]}($call[1]);
            } else {
                switch ($extra) {
                    case 'initializeAssociationAdmin':
                        // only if "true"
                        if (!$call[1]) {
                            break;
                        }

                        // initialize the association-admin
                        $mapper->get($name)->getAssociationAdmin()->configureShowFields(new ShowMapper(
                                $mapper->get($name)->getAssociationAdmin()->getShowBuilder(), $mapper->get($name)->getAssociationAdmin()->getShow(), $mapper->get($name)->getAssociationAdmin()
                        ));

                        // set the efficient template
                        if (!isset($extras['template'])) {
                            $mapper->get($name)->setTemplate('BlastCoreBundle:CRUD:show_association_admin.html.twig');
                        }
                        break;
                }
            }
        }

        return $mapper;
    }