SmoothPhp\EventSourcing\Entity

Complexity

Total Complexity

11

Size/Duplication

Total Lines	80
Duplicated Lines	10 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

89.66%

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	handleRecursively()	0	9	2
A	registerAggregateRoot()	0	8	3
A	addChildEntity()	0	4	1
A	apply()	0	7	2
A	handle()	8	8	2
A	getApplyMethod()	0	6	1

<?php namespace SmoothPhp\EventSourcing;

use SmoothPhp\Contracts\EventSourcing\Event;
use SmoothPhp\Contracts\EventSourcing\AggregateRoot as AggregateRootContract;

/**
 * Abstract class Entity
 *
 * @author jrdn hannah <jrdn@jrdnhannah.co.uk>
 */
abstract class Entity implements \SmoothPhp\Contracts\EventSourcing\Entity
{
    /** @var AggregateRoot */
    private $aggregateRoot;

    /** @var \SmoothPhp\Contracts\EventSourcing\Entity[] */
    private $children = [];

    /**
     * @param Event $event
     * @throws Exception\AggregateRootAlreadyRegistered
     */
    public function handleRecursively(Event $event)
    {
        $this->handle($event);

        foreach ($this->children as $entity) {
            $entity->registerAggregateRoot($this->aggregateRoot);
            $entity->handleRecursively($event);
        }
    }

    /**
     * @param AggregateRootContract $aggregateRoot
     * @throws Exception\AggregateRootAlreadyRegistered
     */
    public function registerAggregateRoot(AggregateRootContract $aggregateRoot)
    {
        if (null !== $this->aggregateRoot && $this->aggregateRoot !== $aggregateRoot) {
            throw new Exception\AggregateRootAlreadyRegistered;
        }

        $this->aggregateRoot = $aggregateRoot;

$aggregateRoot is of type object<SmoothPhp\Contracts\EventSourcing\AggregateRoot>, but the property $aggregateRoot was declared to be of type object<SmoothPhp\EventSourcing\AggregateRoot>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

    }

    /**
     * @param \SmoothPhp\Contracts\EventSourcing\Entity $entity
     */
    public function addChildEntity(\SmoothPhp\Contracts\EventSourcing\Entity $entity)
    {
        $this->children[] = $entity;
    }

    /**
     * @param Event $event
     * @throws Exception\NoAggregateRootRegistered
     */
    protected function apply(Event $event)
    {
        if (!$this->aggregateRoot) {
            throw new Exception\NoAggregateRootRegistered;
        }
        $this->aggregateRoot->apply($event);
    }

    /**
     * Handles event if capable.
     *
     * @param $event
     */
    protected function handle(Event $event)

This method seems to be duplicated in your project.

    {
        $method = $this->getApplyMethod($event);
        if (!method_exists($this, $method)) {
            return;
        }
        $this->$method($event);
    }

    /**
     * @param Event $event
     * @return string
     */
    private function getApplyMethod(Event $event)
    {
        $classParts = explode('\\', get_class($event));

        return 'apply' . end($classParts);
    }
}