WebhookChannel::send() - Code Metrics - Inspection of "remove events since they are now fired in laravel..." - laravel-notification-channels/webhook - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 2a4379...5cdeb5 )

by Mohamed

created 2016-08-16 00:15 UTC

WebhookChannel::send() A

↳ Parent: WebhookChannel

Complexity

Conditions	3
Paths	3

Size

Total Lines	18
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	0
loc	18
rs	9.4285
cc	3
eloc	10
nc	3
nop	2

<?php

namespace NotificationChannels\Webhook;

use GuzzleHttp\Client;
use Illuminate\Support\Arr;
use NotificationChannels\Webhook\Exceptions\CouldNotSendNotification;
use NotificationChannels\Webhook\Events\MessageWasSent;
use NotificationChannels\Webhook\Events\SendingMessage;
use Illuminate\Notifications\Notification;

class WebhookChannel
{
    /** @var Client */
    protected $client;

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

    /**
     * Send the given notification.
     *
     * @param mixed $notifiable
     * @param \Illuminate\Notifications\Notification $notification
     *
     * @throws \NotificationChannels\Webhook\Exceptions\CouldNotSendNotification
     */
    public function send($notifiable, Notification $notification)
    {
        if (! $url = $notifiable->routeNotificationFor('Webhook')) {
            return;
        }

        $webhookData = $notification->toWebhook($notifiable)->toArray();


        $response = $this->client->post($url, [
            'body' => json_encode(Arr::get($webhookData, 'data')),
            'verify' => false,
            'headers' => Arr::get($webhookData, 'headers'),
        ]);

        if ($response->getStatusCode() !== 200) {
            throw CouldNotSendNotification::serviceRespondedWithAnError($response);
        }
    }
}


1			<?php
2
3			namespace NotificationChannels\Webhook;
4
5			use GuzzleHttp\Client;
6			use Illuminate\Support\Arr;
7			use NotificationChannels\Webhook\Exceptions\CouldNotSendNotification;
8			use NotificationChannels\Webhook\Events\MessageWasSent;
9			use NotificationChannels\Webhook\Events\SendingMessage;
10			use Illuminate\Notifications\Notification;
11
12			class WebhookChannel
13			{
14			/** @var Client */
15			protected $client;
16
17			/**
18			* @param Client $client
19			*/
20			public function __construct(Client $client)
21			{
22			$this->client = $client;
23			}
24
25			/**
26			* Send the given notification.
27			*
28			* @param mixed $notifiable
29			* @param \Illuminate\Notifications\Notification $notification
30			*
31			* @throws \NotificationChannels\Webhook\Exceptions\CouldNotSendNotification
32			*/
33			public function send($notifiable, Notification $notification)
34			{
35			if (! $url = $notifiable->routeNotificationFor('Webhook')) {
36			return;
37			}
38
39			$webhookData = $notification->toWebhook($notifiable)->toArray();
			0 ignored issues – show Bug introduced 2016-08-14 20:58 UTC by Report Bug Copy Issue Report The method `toWebhook()` does not seem to exist on `object<Illuminate\Notifications\Notification>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
40
41			$response = $this->client->post($url, [
42			'body' => json_encode(Arr::get($webhookData, 'data')),
43			'verify' => false,
44			'headers' => Arr::get($webhookData, 'headers'),
45			]);
46
47			if ($response->getStatusCode() !== 200) {
48			throw CouldNotSendNotification::serviceRespondedWithAnError($response);
49			}
50			}
51			}
52

laravel-notification-channels / webhook

Push — master ( 2a4379...5cdeb5 )

WebhookChannel::send() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like