SlackGateway::doSend() - Code Metrics - Inspection of "redesigned some" - shippinno/notification - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 66351d...335aef )

by Hirofumi

created 2018-11-05 14:03 UTC

SlackGateway::doSend() A

↳ Parent: SlackGateway

Complexity

Conditions	3
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	3.0327

Importance

Changes

Metric	Value
dl	0
loc	18
ccs	11
cts	13
cp	0.8462
rs	9.6666
c	0
b	0
f	0
cc	3
nc	4
nop	1
crap	3.0327

<?php
declare(strict_types=1);

namespace Shippinno\Notification\Infrastructure\Domain\Model;

use Maknz\Slack\Client;
use Maknz\Slack\Message;
use Shippinno\Notification\Domain\Model\Destination;
use Shippinno\Notification\Domain\Model\Gateway;
use Shippinno\Notification\Domain\Model\Notification;
use Shippinno\Notification\Domain\Model\NotificationNotSentException;
use Shippinno\Notification\Domain\Model\SlackChannelDestination;
use Throwable;

class SlackGateway extends Gateway
{
    /**
     * @var Client
     */
    private $client;

    /**
     * @param Client $client
     */
    public function __construct(Client $client)
    {
        $this->client = $client;
    }

    /**
     * {@inheritdoc}
     */
    protected function doSend(Notification $notification): void
    {
        /** @var SlackChannelDestination $destination */
        $destination = $notification->destination();
        $text = $notification->subject()->subject()."\n\n";
        $text .= $notification->body()->body();
        $message = new Message($this->client);
        $message->setAllowMarkdown(true);
        $message->setText($text);
        if ($destination instanceof SlackChannelDestination) {
            $message->setChannel($destination->channel());
        }
        try {
            $message->send();
        } catch (Throwable $e) {

            throw new NotificationNotSentException($notification, $e);
        }
    }

    /**
     * {@inheritdoc}
     */
    public function sendsToDestination(Destination $destination): bool
    {
        return $destination instanceof SlackChannelDestination;
    }
}


1		<?php
2		declare(strict_types=1);
3
4		namespace Shippinno\Notification\Infrastructure\Domain\Model;
5
6		use Maknz\Slack\Client;
7		use Maknz\Slack\Message;
8		use Shippinno\Notification\Domain\Model\Destination;
9		use Shippinno\Notification\Domain\Model\Gateway;
10		use Shippinno\Notification\Domain\Model\Notification;
11		use Shippinno\Notification\Domain\Model\NotificationNotSentException;
12		use Shippinno\Notification\Domain\Model\SlackChannelDestination;
13		use Throwable;
14
15		class SlackGateway extends Gateway
16		{
17		/**
18		* @var Client
19		*/
20		private $client;
21
22		/**
23		* @param Client $client
24		*/
25	1	public function __construct(Client $client)
26		{
27	1	$this->client = $client;
28	1	}
29
30		/**
31		* {@inheritdoc}
32		*/
33	1	protected function doSend(Notification $notification): void
34		{
35		/** @var SlackChannelDestination $destination */
36	1	$destination = $notification->destination();
37	1	$text = $notification->subject()->subject()."\n\n";
38	1	$text .= $notification->body()->body();
39	1	$message = new Message($this->client);
40	1	$message->setAllowMarkdown(true);
41	1	$message->setText($text);
42	1	if ($destination instanceof SlackChannelDestination) {
43	1	$message->setChannel($destination->channel());
44		}
45		try {
46	1	$message->send();
47		} catch (Throwable $e) {
		1 ignored issue – show Bug introduced 2018-11-02 09:36 UTC by Report Bug Copy Issue Report The class `Throwable` does not exist. Is this class maybe located in a folder that is not analyzed, or in a newer version of your dependencies than listed in your composer.lock/composer.json? Loading history...
48		throw new NotificationNotSentException($notification, $e);
49		}
50	1	}
51
52		/**
53		* {@inheritdoc}
54		*/
55	1	public function sendsToDestination(Destination $destination): bool
56		{
57	1	return $destination instanceof SlackChannelDestination;
58		}
59		}
60

shippinno / notification

Push — master ( 66351d...335aef )

SlackGateway::doSend() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like