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

Test Failed

Push — master ( 6f7a22...abd8b8 )

by Hirofumi

created 2018-11-02 09:05 UTC

SlackGateway::doSend() A

↳ Parent: SlackGateway

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	16
c	0
b	0
f	0
rs	9.7333
cc	2
nc	2
nop	1

<?php
declare(strict_types=1);

namespace Shippinno\Notification\Infrastructure\Domain\Model;

use Maknz\Slack\Client;
use Maknz\Slack\Message;
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);
        $message->setChannel($destination->channel());
        try {
            $message->send();
        } catch (Throwable $e) {

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

    /**
     * @return string
     */
    public function destinationType(): string
    {
        return SlackChannelDestination::class;
    }
}


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\Gateway;
9			use Shippinno\Notification\Domain\Model\Notification;
10			use Shippinno\Notification\Domain\Model\NotificationNotSentException;
11			use Shippinno\Notification\Domain\Model\SlackChannelDestination;
12			use Throwable;
13
14			class SlackGateway extends Gateway
15			{
16			/**
17			* @var Client
18			*/
19			private $client;
20
21			/**
22			* @param Client $client
23			*/
24			public function __construct(Client $client)
25			{
26			$this->client = $client;
27			}
28
29			/**
30			* {@inheritdoc}
31			*/
32			protected function doSend(Notification $notification): void
33			{
34			/** @var SlackChannelDestination $destination */
35			$destination = $notification->destination();
36			$text = $notification->subject()->subject()."\n\n";
37			$text .= $notification->body()->body();
38			$message = new Message($this->client);
39			$message->setAllowMarkdown(true);
40			$message->setText($text);
41			$message->setChannel($destination->channel());
42			try {
43			$message->send();
44			} 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...
45			throw new NotificationNotSentException($notification, $e);
46			}
47			}
48
49			/**
50			* @return string
51			*/
52			public function destinationType(): string
53			{
54			return SlackChannelDestination::class;
55			}
56			}
57

shippinno / notification

Push — master ( 6f7a22...abd8b8 )

SlackGateway::doSend() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like