Romm\Formz\Form\Definition\AbstractFormDefinitionComponent

Complexity

Total Complexity

19

Size/Duplication

Total Lines	128
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	6

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	checkDefinitionFreezeState()	0	8	2
A	isDefinitionFrozen()	0	5	2
A	getDefinitionState()	0	13	2
A	attachParents()	0	7	2
A	attachParent()	0	6	2
B	handlePropertyMagicMethod()	0	13	5
A	forceNameForProperty()	0	10	4

<?php
/*
 * 2017 Romain CANON <romain.hydrocanon@gmail.com>
 *
 * This file is part of the TYPO3 FormZ project.
 * It is free software; you can redistribute it and/or modify it
 * under the terms of the GNU General Public License, either
 * version 3 of the License, or any later version.
 *
 * For the full copyright and license information, see:
 * http://www.gnu.org/licenses/gpl-3.0.html
 */

namespace Romm\Formz\Form\Definition;

use Romm\ConfigurationObject\ConfigurationObjectFactory;
use Romm\ConfigurationObject\Service\Items\Parents\ParentsTrait;
use Romm\ConfigurationObject\Traits\ConfigurationObject\MagicMethodsTrait;
use Romm\Formz\Exceptions\PropertyNotAccessibleException;

class AbstractFormDefinitionComponent
{
    use MagicMethodsTrait {
        handlePropertyMagicMethod as handlePropertyMagicMethodInternal;
    }
    use ParentsTrait {
        attachParent as private attachParentInternal;
        attachParents as private attachParentsInternal;
    }

    /**
     * @var bool
     */
    private $parentsAttached = false;

    /**
     * This method is used by setter methods, and other methods which goal is to
     * modify a property value.
     *
     * It checks that the definition is not frozen, and if it is actually frozen
     * an exception is thrown.
     *
     * @throws PropertyNotAccessibleException
     */
    protected function checkDefinitionFreezeState()
    {
        if ($this->isDefinitionFrozen()) {
            $methodName = debug_backtrace()[1]['function'];

            throw PropertyNotAccessibleException::formDefinitionFrozenMethod(get_class($this), $methodName);
        }
    }

    /**
     * @return bool
     */
    protected function isDefinitionFrozen()
    {
        return $this->getDefinitionState()
            && $this->getDefinitionState()->isDefinitionFrozen();
    }

    /**
     * @return FormDefinitionState
     */
    protected function getDefinitionState()
    {
        if ($this instanceof FormDefinition) {
            return $this->getState();

The method getState does not exist on object<Romm\Formz\Form\Definition\AbstractFormDefinitionComponent>? Since you implemented __call, maybe consider adding a @method annotation.

        } else {
            return $this->withFirstParent(
                FormDefinition::class,
                function (FormDefinition $formDefinition) {
                    return $formDefinition->getState();
                }
            );
        }
    }

    /**
     * Overrides the magic methods handling from the Configuration Object API.
     *
     * Blocks the parents feature once it has been used.
     *
     * @param object[] $parents
     */
    public function attachParents(array $parents)
    {
        if (false === $this->parentsAttached) {
            $this->attachParentsInternal($parents);
            $this->parentsAttached = true;
        }
    }

    /**
     * @see attachParents()
     *
     * @param object $parent
     * @param bool   $direct
     */
    public function attachParent($parent, $direct = true)
    {
        if (false === $this->parentsAttached) {
            $this->attachParentInternal($parent, $direct);
        }
    }

    /**
     * Overrides the magic methods handling from the Configuration Object API.
     *
     * A new check is added: if the definition is frozen, and a setter method is
     * called, an exception must be thrown.
     *
     * @param string $property
     * @param string $type
     * @param array  $arguments
     * @return mixed
     * @throws PropertyNotAccessibleException
     */
    protected function handlePropertyMagicMethod($property, $type, array $arguments)
    {
        if ($type === 'set'
            && $this->isPropertyAccessible($property)
            && ($this->isDefinitionFrozen()
                || false === ConfigurationObjectFactory::getInstance()->isRunning()

The method isRunning() does not seem to exist on object<Romm\ConfigurationObject\ConfigurationObjectFactory>.

            )
        ) {
            throw PropertyNotAccessibleException::formDefinitionFrozenProperty(get_class($this), $property);
        }

        return $this->handlePropertyMagicMethodInternal($property, $type, $arguments);
    }

    /**
     * @param array $data
     * @param string $property
     */
    protected static function forceNameForProperty(&$data, $property)
    {
        if (isset($data[$property])
            && is_array($data[$property])
        ) {
            foreach ($data[$property] as $key => $entry) {
                $data[$property][$key]['name'] = $key;
            }
        }
    }
}