IPub\Forms\Forms\TEntityContainer

Complexity

Total Complexity

39

Size/Duplication

Total Lines	174
Duplicated Lines	17.24 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	injectEntityManager()	0	6	1
B	setDefaults()	0	13	5
A	setEntity()	0	4	1
A	getEntity()	0	4	1
C	bindEntity()	30	67	24
A	getMetadata()	0	8	2
A	isEntity()	0	4	2
A	iterate()	0	12	3

<?php
/**
 * TEntityContainer.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           10.01.16
 */

declare(strict_types = 1);

namespace IPub\Forms\Forms;

use Doctrine\ORM;

use Nette;
use Nette\Forms;

use IPub;
use IPub\Forms\Exceptions;

/**
 * Class TEntity
 * @package IPub\Forms\Application\UI
 *
 * @property ORM\EntityManager $entityManager
 *
 * @method Nette\Application\UI\Form getForm($need = TRUE)
 * @method setValues($values, $erase = FALSE)
 */
trait TEntityContainer
{
	/**
	 * @var mixed
	 */
	private $entity;

	/**
	 * @param ORM\EntityManager $entityManager
	 *
	 * @return $this
	 */
	public function injectEntityManager(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.

	{
		$this->entityManager = $entityManager;

		return $this;
	}

	/**
	 * @param array|\Traversable $values
	 * @param bool $erase
	 */
	public function setDefaults($values, $erase = FALSE)
	{
		$form = $this->getForm(FALSE);

		if (!$form || !$form->isAnchored() || !$form->isSubmitted()) {
			if ($this->isEntity($values)) {
				$this->bindEntity($this, $values, $erase);

It seems like $values defined by parameter $values on line 59 can also be of type array; however, IPub\Forms\Forms\TEntityContainer::bindEntity() does only seem to accept object, maybe add an additional type check?

			} else {
				$this->setValues($values, $erase);
			}
		}
	}

	/**
	 * @param mixed $entity
	 */
	public function setEntity($entity)
	{
		$this->entity = $entity;
	}

	/**
	 * @return mixed
	 */
	public function getEntity()
	{
		return $this->entity;
	}

	/**
	 * @param Nette\ComponentModel\Component $formElement
	 * @param object $entity
	 * @param bool $erase
	 *
	 * @throws Exceptions\InvalidArgumentException
	 */
	private function bindEntity($formElement, $entity, $erase)
	{
		$classMetadata = $this->getMetadata($entity);

		foreach (self::iterate($formElement) as $name => $component) {

$formElement is of type object<Nette\ComponentModel\Component>, but the function expects a object<Nette\Forms\Controls\BaseControl>|object<Nette\Forms\Container>.

			if ($component instanceof Forms\IControl) {
				if (($classMetadata->hasField($name) || $classMetadata->hasAssociation($name)) && $value = $classMetadata->getFieldValue($entity, $name)) {
					if (is_object($value) && $this->isEntity($value)) {

This code seems to be duplicated across your project.

						$UoW = $this->entityManager->getUnitOfWork();
						$value = $UoW->getSingleIdentifierValue($value);

						$component->setValue($value);

					} else {
						$component->setValue($value);
					}

				} else {
					$methodName = 'get' . ucfirst($name);

					if (method_exists($entity, $methodName) && $value = call_user_func([$entity, $methodName])) {
						if (is_object($value) && $this->isEntity($value)) {

This code seems to be duplicated across your project.

							$UoW = $this->entityManager->getUnitOfWork();
							$value = $UoW->getSingleIdentifierValue($value);

							$component->setValue($value);

						} else {
							$component->setValue($value);
						}

					} else {
						if ($erase) {
							$component->setValue(NULL);
						}
					}
				}

			} else if ($component instanceof Forms\Container) {
				if (($classMetadata->hasField($name) || $classMetadata->hasAssociation($name)) && $value = $classMetadata->getFieldValue($entity, $name)) {
					if (is_object($value) && $this->isEntity($value)) {

This code seems to be duplicated across your project.

						$this->bindEntity($component, $value, $erase);

					} else {
						$component->setValues($value, $erase);
					}

				} else {
					$methodName = 'get' . ucfirst($name);

					if (method_exists($entity, $methodName) && $value = call_user_func([$entity, $methodName])) {
						if (is_object($value) && $this->isEntity($value)) {

This code seems to be duplicated across your project.

							$this->bindEntity($component, $value, $erase);

						} else {
							$component->setValues($value, $erase);
						}

					} else {
						if ($erase) {
							$component->setValues([], $erase);
						}
					}
				}
			}
		}
	}

	/**
	 * @param object $entity
	 *
	 * @return ORM\Mapping\ClassMetadata
	 *
	 * @throws Exceptions\InvalidArgumentException
	 */
	private function getMetadata($entity)
	{
		if (!$this->isEntity($entity)) {
			throw new Exceptions\InvalidArgumentException('Expected object, ' . gettype($entity) . ' given.');
		}

		return $this->entityManager->getClassMetadata(get_class($entity));
	}

	/**
	 * @param mixed $entity
	 *
	 * @return bool
	 */
	private function isEntity($entity)
	{
		return is_object($entity) && $this->entityManager->getMetadataFactory()->hasMetadataFor(get_class($entity));
	}

	/**
	 * @param Nette\Forms\Controls\BaseControl|Nette\Forms\Container $formElement
	 *
	 * @return array|\ArrayIterator
	 *
	 * @throws Exceptions\InvalidArgumentException
	 */
	private static function iterate($formElement)
	{
		if ($formElement instanceof Forms\Container) {
			return $formElement->getComponents();

		} elseif ($formElement instanceof Forms\IControl) {
			return [$formElement];

		} else {
			throw new Exceptions\InvalidArgumentException('Expected Nette\Forms\Container or Nette\Forms\IControl, but ' . get_class($formElement) . ' given');
		}
	}
}