SendNotificationHandler::__construct() - Code Metrics - Inspection of "Merge remote-tracking branch 'origin/master'" - shippinno/notification - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( dd3a40...b9ffdf )

by Hirofumi

created 2018-11-01 17:51 UTC

SendNotificationHandler::__construct() A

↳ Parent: SendNotificationHandler

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	1

Importance

Changes

Metric	Value
dl	0
loc	7
ccs	4
cts	4
cp	1
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	2
crap	1

<?php
declare(strict_types=1);

namespace Shippinno\Notification\Application\Command;

use Shippinno\Notification\Domain\Model\GatewayRegistry;
use Shippinno\Notification\Domain\Model\NotificationId;
use Shippinno\Notification\Domain\Model\NotificationNotFoundException;
use Shippinno\Notification\Domain\Model\NotificationRepository;

class SendNotificationHandler
{
    /**
     * @var NotificationRepository
     */
    private $notificationRepository;

    /**
     * @var GatewayRegistry
     */
    private $gatewayRegistry;

    /**
     * @param NotificationRepository $notificationRepository
     * @param GatewayRegistry $gatewayRegistry
     */
    public function __construct(
        NotificationRepository $notificationRepository,
        GatewayRegistry $gatewayRegistry
    ) {
        $this->notificationRepository = $notificationRepository;
        $this->gatewayRegistry = $gatewayRegistry;
    }

    /**
     * @param SendNotification $command
     * @throws NotificationNotFoundException
     */
    public function handle(SendNotification $command): void
    {
        $notificationId = new NotificationId($command->notificationId());
        $notification = $this->notificationRepository->notificationOfId($notificationId);
        if (is_null($notification)) {
            throw new NotificationNotFoundException($notificationId);
        }
        $this->gatewayRegistry->get($notification->destination())->send($notification);
        $notification->markSent();
    }
}


1		<?php
2		declare(strict_types=1);
3
4		namespace Shippinno\Notification\Application\Command;
5
6		use Shippinno\Notification\Domain\Model\GatewayRegistry;
7		use Shippinno\Notification\Domain\Model\NotificationId;
8		use Shippinno\Notification\Domain\Model\NotificationNotFoundException;
9		use Shippinno\Notification\Domain\Model\NotificationRepository;
10
11		class SendNotificationHandler
12		{
13		/**
14		* @var NotificationRepository
15		*/
16		private $notificationRepository;
17
18		/**
19		* @var GatewayRegistry
20		*/
21		private $gatewayRegistry;
22
23		/**
24		* @param NotificationRepository $notificationRepository
25		* @param GatewayRegistry $gatewayRegistry
26		*/
27	2	public function __construct(
28		NotificationRepository $notificationRepository,
29		GatewayRegistry $gatewayRegistry
30		) {
31	2	$this->notificationRepository = $notificationRepository;
32	2	$this->gatewayRegistry = $gatewayRegistry;
33	2	}
34
35		/**
36		* @param SendNotification $command
37		* @throws NotificationNotFoundException
38		*/
39	2	public function handle(SendNotification $command): void
40		{
41	2	$notificationId = new NotificationId($command->notificationId());
42	2	$notification = $this->notificationRepository->notificationOfId($notificationId);
43	2	if (is_null($notification)) {
44	1	throw new NotificationNotFoundException($notificationId);
45		}
46	1	$this->gatewayRegistry->get($notification->destination())->send($notification);
47	1	$notification->markSent();
48	1	}
49		}
50

shippinno / notification

Push — master ( dd3a40...b9ffdf )

SendNotificationHandler::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like