Daikon\EventSourcing\Aggregate\Command

Complexity

Total Complexity

5

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

84.62%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getAggregateId()	0	4	1
A	hasKnownAggregateRevision()	0	4	1
A	__construct()	0	5	1
A	toArray()	0	7	1
A	getKnownAggregateRevision()	0	4	1

<?php
/**
 * This file is part of the daikon-cqrs/cqrs project.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace Daikon\EventSourcing\Aggregate;

abstract class Command implements CommandInterface
{
    /** @var AggregateIdInterface */
    private $aggregateId;

    /** @var AggregateRevision */
    private $knownAggregateRevision;

    public function getAggregateId(): AggregateIdInterface
    {
        return $this->aggregateId;
    }

    public function getKnownAggregateRevision(): AggregateRevision
    {
        return $this->knownAggregateRevision;
    }

    public function hasKnownAggregateRevision(): bool
    {
        return !$this->knownAggregateRevision->isEmpty();
    }

    public function toArray(): array
    {
        return [
            'aggregateId' => $this->aggregateId->toNative(),
            'knownAggregateRevision' => $this->knownAggregateRevision->toNative()
        ];
    }

    protected function __construct(AggregateIdInterface $aggregateId, AggregateRevision $knownAggregateRevision = null)
    {
        $this->aggregateId = $aggregateId;
        $this->knownAggregateRevision = $knownAggregateRevision ?? AggregateRevision::makeEmpty();

$knownAggregateRevision ?? \Daikon\EventSourcing\Aggregate\AggregateRevision::makeEmpty() is of type object<Daikon\Entity\ValueObject\ValueObjectInterface>, but the property $knownAggregateRevision was declared to be of type object<Daikon\EventSourcing\Aggregate\AggregateRevision>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

    }
}