ApnChannel::send() - Code Metrics - Inspection of "Replace Zend with Pushok to support p8 authenticat..." - laravel-notification-channels/apn - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#67)

by Dwight

created 2019-10-04 05:03 UTC

ApnChannel::send() A

↳ Parent: ApnChannel

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes

Metric	Value
dl	0
loc	14
ccs	0
cts	10
cp	0
rs	9.7998
c	0
b	0
f	0
cc	2
nc	2
nop	2
crap	6

<?php

namespace NotificationChannels\Apn;

use Pushok\Client;
use Illuminate\Notifications\Notification;
use Pushok\Notification as PushNotification;

class ApnChannel
{
    /**
     * The sandbox environment identifier.
     *
     * @var int
     */
    const SANDBOX = 0;

    /**
     * The production environment identifier.
     *
     * @var int
     */
    const PRODUCTION = 1;

    /**
     * The APNS client.
     *
     * @var \Pushok\Client
     */
    protected $client;

    /**
     * Create a new channel instance.
     *
     * @param  \Pushok\Client  $client
     */
    public function __construct(Client $client)
    {
        $this->client = $client;
    }

    /**
     * Send the notification to Apple Push Notification Service.
     *
     * @param mixed $notifiable
     * @param \Illuminate\Notifications\Notification $notification
     */
    public function send($notifiable, Notification $notification)
    {
        $tokens = (array) $notifiable->routeNotificationFor('apn', $notification);

        if (empty($tokens)) {
            return;
        }

        $message = $notification->toApn($notifiable);


        $payload = (new ApnAdapter)->adapt($message);

        $this->sendNotifications($notifiable, $notification, $tokens, $payload);

    }

    /**
     * Send the notification to each of the provided tokens.
     *
     * @param  array  $tokens
     * @param  \Pushok\Payload  $payload
     * @return void
     */
    protected function sendNotifications($tokens, $payload)
    {
        $notifications = [];

        foreach ($tokens as $token) {
            $notifications[] = new PushNotification($payload, $token);
        }

        $this->client->addNotifications($notifications);

        $this->client->push();
    }
}


1			<?php
2
3			namespace NotificationChannels\Apn;
4
5			use Pushok\Client;
6			use Illuminate\Notifications\Notification;
7			use Pushok\Notification as PushNotification;
8
9			class ApnChannel
10			{
11			/**
12			* The sandbox environment identifier.
13			*
14			* @var int
15			*/
16			const SANDBOX = 0;
17
18			/**
19			* The production environment identifier.
20			*
21			* @var int
22			*/
23			const PRODUCTION = 1;
24
25			/**
26			* The APNS client.
27			*
28			* @var \Pushok\Client
29			*/
30			protected $client;
31
32			/**
33			* Create a new channel instance.
34			*
35			* @param \Pushok\Client $client
36			*/
37			public function __construct(Client $client)
38			{
39			$this->client = $client;
40			}
41
42			/**
43			* Send the notification to Apple Push Notification Service.
44			*
45			* @param mixed $notifiable
46			* @param \Illuminate\Notifications\Notification $notification
47			*/
48			public function send($notifiable, Notification $notification)
49			{
50			$tokens = (array) $notifiable->routeNotificationFor('apn', $notification);
51
52			if (empty($tokens)) {
53			return;
54			}
55
56			$message = $notification->toApn($notifiable);
			0 ignored issues – show Bug introduced 2017-02-01 22:31 UTC by Report Bug Copy Issue Report The method `toApn()` 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...
57
58			$payload = (new ApnAdapter)->adapt($message);
59
60			$this->sendNotifications($notifiable, $notification, $tokens, $payload);
			0 ignored issues – show Compatibility introduced 2019-10-04 05:08 UTC by Report Bug Copy Issue Report `$notification` of type `object<Illuminate\Notifications\Notification>` is not a sub-type of `object<Pushok\Payload>`. It seems like you assume a child class of the class `Illuminate\Notifications\Notification` to be always present. This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass. Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type. Loading history... Unused Code introduced 2019-10-04 05:08 UTC by Report Bug Copy Issue Report The call to `ApnChannel::sendNotifications()` has too many arguments starting with `$tokens`. This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue. If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. In this case you can add the `@ignore` PhpDoc annotation to the duplicate definition and it will be ignored. Loading history...
61			}
62
63			/**
64			* Send the notification to each of the provided tokens.
65			*
66			* @param array $tokens
67			* @param \Pushok\Payload $payload
68			* @return void
69			*/
70			protected function sendNotifications($tokens, $payload)
71			{
72			$notifications = [];
73
74			foreach ($tokens as $token) {
75			$notifications[] = new PushNotification($payload, $token);
76			}
77
78			$this->client->addNotifications($notifications);
79
80			$this->client->push();
81			}
82			}
83

laravel-notification-channels / apn

GitHub Access Token became invalid

Pull Request — master (#67)

ApnChannel::send() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like