Knp\RadBundle\Form\DefaultFormCreator

Complexity

Total Complexity

13

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	7

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getSubscribedEvents()	0	6	1
A	create()	0	7	1
A	extractPropertyName()	0	4	1
A	hasRelatedSetter()	0	4	1
A	__construct()	0	6	2
C	preSetData()	0	28	7

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 12 and the first side effect is on line 6.

namespace Knp\RadBundle\Form;

use Symfony\Component\Form\FormFactoryInterface;
use Symfony\Component\Form\FormEvent;;

It is generally recommended to place each PHP statement on a line by itself.

use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Symfony\Component\Form\FormEvents;
use Knp\RadBundle\Reflection\ClassMetadataFetcher;
use Doctrine\Common\Inflector\Inflector;

class DefaultFormCreator implements FormCreatorInterface, EventSubscriberInterface
{
    private $fetcher;
    private $factory;
    private $dataTypeGuesser;

    public function __construct(ClassMetadataFetcher $fetcher = null, FormFactoryInterface $factory, DataTypeGuesser $dataTypeGuesser)

Parameters which have default values should be placed at the end.

    {
        $this->fetcher = $fetcher ?: new ClassMetadataFetcher;
        $this->factory = $factory;
        $this->dataTypeGuesser = $dataTypeGuesser;
    }

    public static function getSubscribedEvents()
    {
        return array(
            FormEvents::PRE_SET_DATA => 'preSetData'
        );
    }

    public function create($object, $purpose = null, array $options = array())
    {
        $builder = $this->factory->createBuilder('form', $object, $options);
        $builder->addEventSubscriber($this);

        return $builder->getForm();
    }

    public function preSetData(FormEvent $event)
    {
        $object = $event->getData();
        $form = $event->getForm();

        if (!is_object($object)) {
            return;
        }

        $this->dataTypeGuesser->setData($object);

        foreach ($this->fetcher->getMethods($object) as $method) {
            if (0 === strpos($method, 'get') || 0 === strpos($method, 'is')) {
                $propertyName = $this->extractPropertyName($method);
                if ($this->hasRelatedSetter($object, $propertyName)) {
                    $form->add($this->factory->createForProperty(get_class($object), $propertyName, null, array(
                        'auto_initialize' => false,
                    )));
                }
            }
        }

        foreach ($this->fetcher->getProperties($object) as $property) {
            $form->add($this->factory->createForProperty(get_class($object), $property, null, array(
                'auto_initialize' => false,
            )));
        }
    }

    private function extractPropertyName($methodName)
    {
        return lcfirst(preg_replace('#is|get#', '', $methodName));
    }

    private function hasRelatedSetter($object, $propertyName)
    {
        return $this->fetcher->hasMethod($object, 'set'.Inflector::classify($propertyName));
    }
}