WebinoViewLib\Component\ComponentStateTrait

Complexity

Total Complexity

11

Size/Duplication

Total Lines	123
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
	getRenderEvent()	0	1	?
A	onDispatchState()	0	7	2
B	onStateChange()	0	25	3
A	getState()	0	8	2
A	setState()	0	5	1
A	mergeState()	0	5	1
A	initState()	0	3	1
A	__clone()	0	4	1

<?php

namespace WebinoViewLib\Component;

use WebinoAppLib\Event\AppEvent;
use WebinoAppLib\Event\DispatchEvent;
use WebinoAppLib\Feature\HttpListener;
use WebinoAppLib\Service\Initializer\RoutingAwareTrait;
use WebinoConfigLib\Feature\FeatureInterface;
use WebinoDomLib\Dom\Config\SpecConfigAggregateInterface;
use WebinoDomLib\Dom\NodeInterface;
use WebinoDomLib\Event\RenderEvent;
use WebinoEventLib\ListenerAggregateTrait;
use WebinoViewLib\Feature\NodeView;
use WebinoViewLib\Feature\ViewListener;
use Zend\EventManager\ListenerAggregateInterface;
use Zend\Stdlib\CallbackHandler;

/**
 * Class ComponentStateTrait
 * @TODO concept
 */
trait ComponentStateTrait
{
    use RoutingAwareTrait;

    /**
     * @var \stdClass
     */
    private $state;

    /**
     * @TODO concept
     * @var NodeInterface[]
     */
    private $nodes = [];

    /**
     * @return RenderEvent
     */
    abstract protected function getRenderEvent();

    /**
     * TODO concept
     * @param DispatchEvent $event
     */
    public function onDispatchState(DispatchEvent $event)
    {
        // TODO resolve state
        $state = $event->getRequest()->getQuery()->counter[$this->getIndex()];

It seems like you code against a concrete implementation and not the interface Zend\Stdlib\RequestInterface as the method getQuery() does only exist in the following implementations of said interface: Zend\Http\PhpEnvironment\Request, Zend\Http\Request.

It seems like getIndex() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        empty($state) or $this->mergeState($state);

Using logical operators such as or instead of || is generally not recommended.

    }

    /**
     * @TODO concept
     * @param string $name Node name
     * @param string|\Closure $method
     * @return $this
     */
    public function onStateChange($name, $method)
    {
        // TODO create href concept
        $clone = clone $this;

        // TODO refactor
        if (is_string($method)) {
            $clone->$method();
        } elseif ($method instanceof \Closure) {
            call_user_func($method->bindTo($clone));
        } else {

This else statement is empty and can be removed.

            // TODO invalid argument exception
        }

        // TODO common
        $query = ['counter' => [$this->getIndex() => (array) $clone->getState()]];

It seems like getIndex() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $href = $this->getRouter()->url()->setOption('query', $query);

The call to url() misses a required argument $name.

        // TODO concept
        $node = $this->getRenderEvent()->getNode($name);
        $node->setAttribute('href', $href);

        return $this;
    }

    /**
     * @return \stdClass
     */
    public function getState()
    {
        if (null === $this->state) {
            $this->state = new \stdClass;
            $this->initState($this->state);
        }
        return $this->state;
    }

    /**
     * @param \stdClass $state
     * @return $this
     */
    public function setState(\stdClass $state)
    {
        $this->state = $state;
        return $this;
    }

    /**
     * @param array|\stdClass $state
     * @return $this
     */
    public function mergeState($state)
    {
        $this->state = (object) array_replace_recursive((array) $this->getState(), (array) $state);
        return $this;
    }

    /**
     * @param \stdClass $state
     */
    public function initState(\stdClass $state)

The parameter $state is not used and could be removed.

    {
    }

    /**
     * @param string $name
     * @return ComponentNode
     */
//    protected function getNode($name)

61% of this comment could be valid code. Did you maybe forget this after debugging?

//    {
//        if (empty($this->nodes[$name])) {
//            $this->nodes[$name] = new ComponentNode($this->getRenderEvent()->getNode($name), $this);
//        }
//        return $this->nodes[$name];
//    }

    /**
     * @TODO concept
     */
    public function __clone()
    {
        $this->state = clone $this->getState();
    }
}