CultuurNet\UDB3\Offer\Events\Moderation\AbstractPublished

Complexity

Total Complexity

4

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	getPublicationDate()	0	4	1
A	serialize()	0	6	1
A	deserialize()	0	7	1

<?php

namespace CultuurNet\UDB3\Offer\Events\Moderation;

use CultuurNet\UDB3\Offer\Events\AbstractEvent;
use DateTime;
use DateTimeInterface;

abstract class AbstractPublished extends AbstractEvent
{
    /**
     * @var DateTimeInterface
     */
    private $publicationDate;

    final public function __construct(string $itemId, DateTimeInterface $publicationDate)
    {
        parent::__construct($itemId);

        $this->publicationDate = $publicationDate;
    }

    public function getPublicationDate(): DateTimeInterface
    {
        return $this->publicationDate;
    }

    public function serialize(): array
    {
        return parent::serialize() + [
            'publication_date' => $this->publicationDate->format(DateTime::ATOM),
        ];
    }

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

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

        );
    }
}