Kreta\Notifier\Domain\Model\Inbox\Notification\Notification

Complexity

Total Complexity

11

Size/Duplication

Total Lines	91
Duplicated Lines	10.99 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	broadcast()	0	15	1
A	markAsRead()	0	9	1
A	markAsUnread()	0	9	1
A	applyNotificationPublished()	0	7	1
A	applyNotificationMarkedAsRead()	0	5	1
A	applyNotificationMarkedAsUnread()	0	5	1
A	reconstitute()	10	10	2
A	id()	0	4	1
A	__toString()	0	4	1

<?php

/*
 * This file is part of the Kreta package.
 *
 * (c) Beñat Espiña <benatespina@gmail.com>
 * (c) Gorka Laucirica <gorka.lauzirika@gmail.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 Kreta\Notifier\Domain\Model\Inbox\Notification;

use Kreta\Notifier\Domain\Model\Inbox\UserId;
use Kreta\SharedKernel\Domain\Model\AggregateRoot;
use Kreta\SharedKernel\Domain\Model\EventSourcedAggregateRoot;
use Kreta\SharedKernel\Event\EventStream;

class Notification extends AggregateRoot implements EventSourcedAggregateRoot
{
    private $id;
    private $body;
    private $publishedOn;
    private $readOn;
    private $status;
    private $userId;

    private function __construct(NotificationId $id)
    {
        $this->id = $id;
    }

    public static function broadcast(NotificationId $id, UserId $userId, NotificationBody $body) : self
    {
        $notification = new self($id);

        $notification->publish(
            new NotificationPublished(
                $id,
                $userId,
                $body,
                NotificationStatus::unread()

The call to NotificationPublished::__construct() has too many arguments starting with \Kreta\Notifier\Domain\Model\Inbox\Notification\NotificationStatus::unread().

            )
        );

        return $notification;
    }

    public function markAsRead() : void
    {
        $this->publish(
            new NotificationMarkedAsRead(
                $this->id,
                $this->userId
            )
        );
    }

    public function markAsUnread() : void
    {
        $this->publish(
            new NotificationMarkedAsUnread(
                $this->id,
                $this->userId
            )
        );
    }

    protected function applyNotificationPublished(NotificationPublished $event) : void
    {
        $this->body = $event->body();
        $this->userId = $event->userId();
        $this->publishedOn = $event->occurredOn();
        $this->status = $event->status();
    }

    protected function applyNotificationMarkedAsRead(NotificationMarkedAsRead $event) : void
    {
        $this->readOn = $event->occurredOn();
        $this->status = $event->status();
    }

    protected function applyNotificationMarkedAsUnread(NotificationMarkedAsUnread $event) : void
    {
        $this->readOn = null;
        $this->status = $event->status();
    }

    public static function reconstitute(EventStream $stream) : EventSourcedAggregateRoot

This method seems to be duplicated in your project.

    {
        $notification = new self($stream->aggregateRootId());

$stream->aggregateRootId() of type object<Kreta\SharedKernel\Domain\Model\Identity\BaseId> is not a sub-type of object<Kreta\Notifier\Domain\Model\Inbox\Notification\NotificationId>. It seems like you assume a concrete implementation of the interface Kreta\SharedKernel\Domain\Model\Identity\BaseId to be always present.

        $events = $stream->events()->toArray();
        foreach ($events as $event) {
            $notification->apply($event);
        }

        return $notification;
    }

    public function id() : NotificationId
    {
        return $this->id;
    }

    public function __toString() : string
    {
        return (string) $this->id()->id();
    }
}