Kunstmaan\PagePartBundle\Helper\FormWidgets\PageTemplateWidget

Complexity

Total Complexity

25

Size/Duplication

Total Lines	215
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	12

Test Coverage

Coverage

0%

Importance

Changes

0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	26	3
A	processRegion()	0	10	3
A	loadWidgets()	0	14	4
A	getPageTemplate()	0	4	1
A	getPageTemplates()	0	4	1
A	getPage()	0	4	1
A	buildForm()	0	6	2
A	bindRequest()	0	8	2
A	persist()	0	7	2
A	getFormErrors()	0	10	2
A	getTemplate()	0	4	1
A	getFormWidget()	0	8	2
A	getExtraParams()	0	4	1

<?php

namespace Kunstmaan\PagePartBundle\Helper\FormWidgets;

use Doctrine\ORM\EntityManager;
use Doctrine\ORM\EntityManagerInterface;
use Kunstmaan\AdminBundle\Helper\FormWidgets\FormWidget;
use Kunstmaan\NodeBundle\Entity\PageInterface;
use Kunstmaan\PagePartBundle\Entity\PageTemplateConfiguration;
use Kunstmaan\PagePartBundle\Helper\HasPageTemplateInterface;
use Kunstmaan\PagePartBundle\PagePartAdmin\PagePartAdmin;
use Kunstmaan\PagePartBundle\PagePartAdmin\PagePartAdminConfiguratorInterface;
use Kunstmaan\PagePartBundle\PagePartAdmin\PagePartAdminFactory;
use Kunstmaan\PagePartBundle\PagePartConfigurationReader\PagePartConfigurationReaderInterface;
use Kunstmaan\PagePartBundle\PageTemplate\PageTemplateConfigurationReaderInterface;
use Kunstmaan\PagePartBundle\PageTemplate\PageTemplateConfigurationService;
use Kunstmaan\PagePartBundle\PageTemplate\PageTemplateInterface;
use Kunstmaan\PagePartBundle\PageTemplate\Region;
use Symfony\Component\Form\FormBuilderInterface;
use Symfony\Component\Form\FormView;
use Symfony\Component\HttpFoundation\Request;

/**
 * PageTemplateWidget
 */
class PageTemplateWidget extends FormWidget
{
    /**
     * @var EntityManagerInterface
     */
    private $em;

    /**
     * @var PagePartAdminFactory
     */
    private $pagePartAdminFactory;

    /**
     * @var PageInterface
     */
    private $page;

    /**
     * @var Request
     */
    private $request;

    /**
     * @var PagePartWidget[]
     */
    private $widgets = array();

    /**
     * @var PageTemplateInterface[]
     */
    private $pageTemplates = array();

    /**
     * @var PagePartAdminConfiguratorInterface[]
     */
    private $pagePartAdminConfigurations = array();

    /**
     * @var PageTemplateConfiguration
     */
    protected $pageTemplateConfiguration;

    /**
     * @param HasPageTemplateInterface                 $page
     * @param Request                                  $request
     * @param EntityManagerInterface                   $em
     * @param PagePartAdminFactory                     $pagePartAdminFactory
     * @param PageTemplateConfigurationReaderInterface $templateReader
     * @param PagePartConfigurationReaderInterface     $pagePartReader
     * @param PageTemplateConfigurationService         $pageTemplateConfigurationService
     */
    public function __construct(
        HasPageTemplateInterface $page,
        Request $request,

You have injected the Request via parameter $request. This is generally not recommended as there might be multiple instances during a request cycle (f.e. when using sub-requests). Instead, it is recommended to inject the RequestStack and retrieve the current request each time you need it via getCurrentRequest().

        EntityManagerInterface $em,
        PagePartAdminFactory $pagePartAdminFactory,
        PageTemplateConfigurationReaderInterface $templateReader,
        PagePartConfigurationReaderInterface $pagePartReader,
        PageTemplateConfigurationService $pageTemplateConfigurationService
    ) {
        parent::__construct();

        $this->page = $page;

It seems like $page of type object<Kunstmaan\PagePartBundle\Helper\HasPageTemplateInterface> is incompatible with the declared type object<Kunstmaan\NodeBundle\Entity\PageInterface> of property $page.

        $this->em = $em;
        $this->request = $request;
        $this->pagePartAdminFactory = $pagePartAdminFactory;

        $this->pageTemplates = $templateReader->getPageTemplates($page);
        $this->pagePartAdminConfigurations = $pagePartReader->getPagePartAdminConfigurators($page);
        $this->pageTemplateConfiguration = $pageTemplateConfigurationService->findOrCreateFor($page);

        foreach ($this->getPageTemplate()->getRows() as $row) {
            foreach ($row->getRegions() as $region) {
                $this->processRegion($region);
            }
        }
    }

    /**
     * @param Region $region The region
     */
    private function processRegion($region)
    {
        if (\count($region->getChildren())) {
            foreach ($region->getChildren() as $child) {
                $this->processRegion($child);
            }
        } else {
            $this->loadWidgets($region);
        }
    }

    /**
     * @param Region $region The region
     */
    private function loadWidgets($region)
    {
        $pagePartAdminConfiguration = null;
        foreach ($this->pagePartAdminConfigurations as $ppac) {
            if ($ppac->getContext() == $region->getName()) {
                $pagePartAdminConfiguration = $ppac;
            }
        }

        if ($pagePartAdminConfiguration !== null) {
            $pagePartWidget = new PagePartWidget($this->page, $this->request, $this->em, $pagePartAdminConfiguration, $this->pagePartAdminFactory);

$this->page is of type object<Kunstmaan\NodeBundle\Entity\PageInterface>, but the function expects a object<Kunstmaan\PagePartBundle\Helper\HasPagePartsInterface>.

            $this->widgets[$region->getName()] = $pagePartWidget;
        }
    }

    /**
     * @return PageTemplateInterface
     */
    public function getPageTemplate()
    {
        return $this->pageTemplates[$this->pageTemplateConfiguration->getPageTemplate()];
    }

    /**
     * @return PageTemplateInterface[]
     */
    public function getPageTemplates()
    {
        return $this->pageTemplates;
    }

    /**
     * @return PageInterface|HasPageTemplateInterface
     */
    public function getPage()
    {
        return $this->page;
    }

    /**
     * @param FormBuilderInterface $builder The form builder
     */
    public function buildForm(FormBuilderInterface $builder)
    {
        foreach ($this->widgets as $widget) {
            $widget->buildForm($builder);
        }
    }

    /**
     * @param Request $request
     */
    public function bindRequest(Request $request)
    {
        $configurationname = $request->get('pagetemplate_template');
        $this->pageTemplateConfiguration->setPageTemplate($configurationname);
        foreach ($this->widgets as $widget) {
            $widget->bindRequest($request);
        }
    }

    /**
     * @param EntityManager $em The entity manager
     */
    public function persist(EntityManager $em)
    {
        $em->persist($this->pageTemplateConfiguration);
        foreach ($this->widgets as $widget) {
            $widget->persist($em);
        }
    }

    /**
     * @param FormView $formView
     *
     * @return array
     */
    public function getFormErrors(FormView $formView)
    {
        $errors = array();

        foreach ($this->widgets as $widget) {
            $errors = array_merge($errors, $widget->getFormErrors($formView));
        }

        return $errors;
    }

    /**
     * @return string
     */
    public function getTemplate()
    {
        return '@KunstmaanPagePart/FormWidgets/PageTemplateWidget/widget.html.twig';
    }

    /**
     * @param string $name
     *
     * @return PagePartAdmin

Should the return type not be PagePartWidget|null?

     */
    public function getFormWidget($name)
    {
        if (\array_key_exists($name, $this->widgets)) {
            return $this->widgets[$name];
        }

        return null;
    }

    /**
     * @param Request $request
     *
     * @return array
     */
    public function getExtraParams(Request $request)
    {
        return [];
    }
}