Kunstmaan\NodeBundle\Event\AdaptFormEvent

Complexity

Total Complexity

7

Size/Duplication

Total Lines	95
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	1
A	getNode()	0	4	1
A	getNodeTranslation()	0	4	1
A	getNodeVersion()	0	4	1
A	getPage()	0	4	1
A	getTabPane()	0	4	1
A	getRequest()	0	4	1

<?php

namespace Kunstmaan\NodeBundle\Event;

use Kunstmaan\AdminBundle\Event\BcEvent;
use Kunstmaan\AdminBundle\Helper\FormWidgets\Tabs\TabPane;
use Kunstmaan\NodeBundle\Entity\HasNodeInterface;
use Kunstmaan\NodeBundle\Entity\Node;
use Kunstmaan\NodeBundle\Entity\NodeTranslation;
use Kunstmaan\NodeBundle\Entity\NodeVersion;
use Symfony\Component\HttpFoundation\Request;

/**
 * The event to pass metadata if the adaptForm event is triggered
 */
class AdaptFormEvent extends BcEvent
{
    /**
     * @var TabPane
     */
    private $tabPane;

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

    /**
     * @var Node
     */
    private $node;

    /**
     * @var NodeTranslation
     */
    private $nodeTranslation;

    /**
     * @var NodeVersion
     */
    private $nodeVersion;

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

    /**
     * @param Request          $request
     * @param TabPane          $tabPane         The tab pane
     * @param HasNodeInterface $page            The page

Should the type for parameter $page not be HasNodeInterface|null?

     * @param Node             $node            The node

Should the type for parameter $node not be null|Node?

     * @param NodeTranslation  $nodeTranslation The node translation

Should the type for parameter $nodeTranslation not be null|NodeTranslation?

     * @param NodeVersion      $nodeVersion     The node version

Should the type for parameter $nodeVersion not be null|NodeVersion?

     */
    public function __construct(Request $request, TabPane $tabPane, $page = null, Node $node = null, NodeTranslation $nodeTranslation = null, NodeVersion $nodeVersion = null)

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().

    {
        $this->request = $request;
        $this->tabPane = $tabPane;
        $this->page = $page;
        $this->node = $node;
        $this->nodeTranslation = $nodeTranslation;
        $this->nodeVersion = $nodeVersion;
    }

    /**
     * @return Node
     */
    public function getNode()
    {
        return $this->node;
    }

    /**
     * @return NodeTranslation
     */
    public function getNodeTranslation()
    {
        return $this->nodeTranslation;
    }

    /**
     * @return NodeVersion
     */
    public function getNodeVersion()
    {
        return $this->nodeVersion;
    }

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

    /**
     * @return TabPane
     */
    public function getTabPane()
    {
        return $this->tabPane;
    }

    public function getRequest()
    {
        return $this->request;
    }
}