IPub\Forms\Forms\EntityForm

Complexity

Total Complexity

2

Size/Duplication

Total Lines	47
Duplicated Lines	14.89 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	1
A	addContainer()	7	7	1

<?php
/**
 * EntityForm.php
 *
 * @copyright      More in license.md
 * @license        http://www.ipublikuj.eu
 * @author         Adam Kadlec http://www.ipublikuj.eu
 * @package        iPublikuj:Forms!
 * @subpackage     Forms
 * @since          1.0.0
 *
 * @date           26.05.13
 */

declare(strict_types = 1);

namespace IPub\Forms\Forms;

use Doctrine\ORM;

use Nette;
use Nette\Application;
use Nette\ComponentModel;

class EntityForm extends Application\UI\Form
{
	/**
	 * Implement entity handling in form
	 */
	use TEntityContainer;

	/**
	 * Implement form methods
	 */
	use TForm;

	/**
	 * @var ORM\EntityManager
	 */
	private $entityManager;

	/**
	 * @param ORM\EntityManager $entityManager
	 * @param ComponentModel\IContainer|NULL $parent
	 * @param string|NULL $name
	 */
	public function __construct(
		ORM\EntityManager $entityManager,

You have injected the EntityManager via parameter $entityManager. This is generally not recommended as it might get closed and become unusable. Instead, it is recommended to inject the ManagerRegistry and retrieve the EntityManager via getManager() each time you need it.

		ComponentModel\IContainer $parent = NULL,
		string $name = NULL
	) {
		parent::__construct($parent, $name);

		$this->entityManager = $entityManager;
	}

	/**
	 * Adds naming container to the form
	 *
	 * @param string $name
	 *
	 * @return Container
	 */
	public function addContainer($name) : Container

This method seems to be duplicated in your project.

	{
		$control = new Container($this->entityManager);
		$control->setCurrentGroup($this->currentGroup);

		return $this[$name] = $control;
	}
}