EventStore\StreamFeed\Event

Complexity

Total Complexity

6

Size/Duplication

Total Lines	83
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Test Coverage

Coverage

88.24%

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	getType()	0	4	1
A	getVersion()	0	4	1
A	getData()	0	4	1
A	getMetadata()	0	4	1
A	getEventId()	0	4	1

<?php
namespace EventStore\StreamFeed;

use EventStore\ValueObjects\Identity\UUID;

/**
 * Class Event
 * @package EventStore\StreamFeed
 */
final class Event
{
    /**
     * @var string
     */
    private $type;

    /**
     * @var integer
     */
    private $version;

    /**
     * @var array
     */
    private $data;

    /**
     * @var array
     */
    private $metadata;

    /**
     * @var UUID
     */
    private $eventId;

    /**
     * @param string  $type
     * @param integer $version
     * @param array   $data
     * @param array   $metadata
     * @param UUID    $eventId
     */
    public function __construct($type, $version, array $data, array $metadata = null, UUID $eventId = null)
    {
        $this->type = $type;
        $this->version = (integer) $version;
        $this->data = $data;
        $this->metadata = $metadata;

It seems like $metadata can be null. However, the property $metadata is declared as array. Maybe change the type of the property to array|null or add a type check?

        $this->eventId = $eventId;
    }

    /**
     * @return string
     */
    public function getType()
    {
        return $this->type;
    }

    /**
     * @return integer
     */
    public function getVersion()
    {
        return $this->version;
    }

    /**
     * @return array
     */
    public function getData()
    {
        return $this->data;
    }

    /**
     * @return array
     */
    public function getMetadata()
    {
        return $this->metadata;
    }

    /**
     * @return UUID
     */
    public function getEventId()
    {
        return $this->eventId;
    }
}