CultuurNet\UDB3\Timestamp

Complexity

Total Complexity

9

Size/Duplication

Total Lines	79
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	12	2
A	getStartDate()	0	4	1
A	getEndDate()	0	4	1
A	getEventStatus()	0	4	1
A	deserialize()	0	7	1
A	serialize()	0	17	2
A	fromUdb3ModelDateRange()	0	7	1

<?php

namespace CultuurNet\UDB3;

use Broadway\Serializer\SerializableInterface;
use CultuurNet\UDB3\Event\ValueObjects\EventStatus;
use CultuurNet\UDB3\Event\ValueObjects\EventStatusType;
use CultuurNet\UDB3\Model\ValueObject\Calendar\DateRange;
use DateTime;
use DateTimeInterface;
use InvalidArgumentException;

final class Timestamp implements SerializableInterface
{
    /**
     * @var DateTimeInterface
     */
    private $startDate;

    /**
     * @var DateTimeInterface
     */
    private $endDate;

    /**
     * @var EventStatus|null
     */
    private $eventStatus;

    final public function __construct(
        DateTimeInterface $startDate,
        DateTimeInterface $endDate,
        EventStatus $eventStatus = null

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

    ) {
        if ($endDate < $startDate) {
            throw new InvalidArgumentException('End date can not be earlier than start date.');
        }

        $this->startDate = $startDate;
        $this->endDate = $endDate;
    }

    public function getStartDate(): DateTimeInterface
    {
        return $this->startDate;
    }

    public function getEndDate(): DateTimeInterface
    {
        return $this->endDate;
    }

    public function getEventStatus(): ?EventStatus
    {
        return $this->eventStatus;
    }

    public static function deserialize(array $data): Timestamp
    {
        return new static(
            DateTime::createFromFormat(DateTime::ATOM, $data['startDate']),

It seems like \DateTime::createFromFormat(\DateTime::ATOM, $data['startDate']) targeting DateTime::createFromFormat() can also be of type false; however, CultuurNet\UDB3\Timestamp::__construct() does only seem to accept object<DateTimeInterface>, did you maybe forget to handle an error condition?

            DateTime::createFromFormat(DateTime::ATOM, $data['endDate'])

It seems like \DateTime::createFromFormat(\DateTime::ATOM, $data['endDate']) targeting DateTime::createFromFormat() can also be of type false; however, CultuurNet\UDB3\Timestamp::__construct() does only seem to accept object<DateTimeInterface>, did you maybe forget to handle an error condition?

        );
    }

    public function serialize(): array
    {
        $serialized = [
            'startDate' => $this->startDate->format(DateTime::ATOM),
            'endDate' => $this->endDate->format(DateTime::ATOM),
            'eventStatus' => EventStatusType::scheduled()->toNative()
        ];

        if ($this->eventStatus === null) {
            return $serialized;
        }

        return \array_merge(
            $serialized,
            $this->eventStatus->serialize()
        );
    }

    public static function fromUdb3ModelDateRange(DateRange $dateRange): Timestamp
    {
        return new Timestamp(
            $dateRange->getFrom(),

It seems like $dateRange->getFrom() can be null; however, __construct() does not accept null, maybe add an additional type check?

            $dateRange->getTo()

It seems like $dateRange->getTo() can be null; however, __construct() does not accept null, maybe add an additional type check?

        );
    }
}