WebhookChannel::send() - Code Metrics - Inspection of "Return full response from the request rather than..." - laravel-notification-channels/webhook - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#19)

by Adam

created 2018-03-09 02:06 UTC

WebhookChannel::send() A

↳ Parent: WebhookChannel

Complexity

Conditions	2
Paths	2

Size

Total Lines	14
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	0
loc	14
ccs	0
cts	12
cp	0
rs	9.4285
cc	2
eloc	8
nc	2
nop	2
crap	6

<?php

namespace NotificationChannels\Webhook;

use GuzzleHttp\Client;
use Illuminate\Support\Arr;
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
     *
     * @return \GuzzleHttp\Psr7\Response
     */
    public function send($notifiable, Notification $notification)
    {
        if (! $url = $notifiable->routeNotificationFor('Webhook')) {
            return;
        }

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


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


1			<?php
2
3			namespace NotificationChannels\Webhook;
4
5			use GuzzleHttp\Client;
6			use Illuminate\Support\Arr;
7			use Illuminate\Notifications\Notification;
8
9			class WebhookChannel
10			{
11			/** @var Client */
12			protected $client;
13
14			/**
15			* @param Client $client
16			*/
17			public function __construct(Client $client)
18			{
19			$this->client = $client;
20			}
21
22			/**
23			* Send the given notification.
24			*
25			* @param mixed $notifiable
26			* @param \Illuminate\Notifications\Notification $notification
27			*
28			* @return \GuzzleHttp\Psr7\Response
29			*/
30			public function send($notifiable, Notification $notification)
31			{
32			if (! $url = $notifiable->routeNotificationFor('Webhook')) {
33			return;
34			}
35
36			$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...
37
38			return $this->client->post($url, [
39			'body' => json_encode(Arr::get($webhookData, 'data')),
40			'verify' => false,
41			'headers' => Arr::get($webhookData, 'headers'),
42			]);
43			}
44			}
45

laravel-notification-channels / webhook

Pull Request — master (#19)

WebhookChannel::send() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like