LIN3S\SharedKernel\Event\StoredEvent

Complexity

Total Complexity

7

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	fromDomainEvent()	0	7	1
A	setPayload()	0	17	3
A	__construct()	0	7	1
A	setOccurredOn()	0	5	1
A	toArray()	0	9	1

<?php

/*
 * This file is part of the Shared Kernel library.
 *
 * Copyright (c) 2016-present LIN3S <info@lin3s.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace LIN3S\SharedKernel\Event;

use LIN3S\SharedKernel\Domain\Model\DomainEvent;

/**
 * @author Beñat Espiña <benatespina@gmail.com>
 */
class StoredEvent

Since you have declared the constructor as private, maybe you should also declare the class as final.

{
    private $id;
    private $type;
    private $payload;
    private $occurredOn;
    private $stream;

    public static function fromDomainEvent(DomainEvent $event, StreamName $stream) : self
    {
        $instance = new self(get_class($event), $event->occurredOn(), $stream);
        $instance->setPayload($event);

        return $instance;
    }

    private function setPayload(DomainEvent $event) : void
    {
        $eventReflection = new \ReflectionClass($event);
        foreach ($eventReflection->getProperties() as $property) {
            if ('occurredOn' === $property->name) {
                continue;
            }
            $property->setAccessible(true);
            // TODO: the following cast is not valid
            // what happens when the given property (Value object)
            // contains more than one attribute or it has not got implemented
            // the __toString() method ??

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

            // Furthermore, in the decodification process we need the class this property
            $this->payload[$property->getName()] = (string) $property->getValue($event);
        }
        $this->payload = json_encode($this->payload);
    }

    private function __construct(string $type, \DateTimeInterface $occurredOn, StreamName $stream)
    {
        $this->type = $type;
        $this->payload = [];
        $this->setOccurredOn($occurredOn);
        $this->stream = $stream->name();
    }

    private function setOccurredOn(\DateTimeInterface $occurredOn) : void
    {
        $occurredOn->setTimezone(new \DateTimeZone('UTC'));
        $this->occurredOn = $occurredOn->getTimestamp();
    }

    public function toArray() : array
    {
        return [
            $this->type,
            $this->payload,
            $this->occurredOn,
            $this->stream,
        ];
    }
}