GitHub Access Token became invalid

It seems like the GitHub access token used for retrieving details about this repository from GitHub became invalid. This might prevent certain types of inspections from being run (in particular, everything related to pull requests).
Please ask an admin of your repository to re-new the access token on this website.
Completed
Pull Request — master (#32)
by
unknown
01:59
created

ApnChannel   A

Complexity

Total Complexity 13

Size/Duplication

Total Lines 155
Duplicated Lines 0 %

Coupling/Cohesion

Components 1
Dependencies 10

Test Coverage

Coverage 87.93%

Importance

Changes 4
Bugs 1 Features 0
Metric Value
wmc 13
c 4
b 1
f 0
lcom 1
cbo 10
dl 0
loc 155
ccs 51
cts 58
cp 0.8793
rs 10

5 Methods

Rating   Name   Duplication   Size   Complexity  
A __construct() 0 15 1
B send() 0 49 6
A getFeedback() 0 21 3
A openConnection() 0 8 2
A closeConnection() 0 4 1
1
<?php
2
3
namespace NotificationChannels\Apn;
4
5
use Exception;
6
use Illuminate\Events\Dispatcher;
7
use ZendService\Apple\Apns\Message\Alert;
8
use Illuminate\Notifications\Notification;
9
use ZendService\Apple\Apns\Message as Packet;
10
use ZendService\Apple\Apns\Client\Message as Client;
11
use NotificationChannels\Apn\Exceptions\SendingFailed;
12
use Illuminate\Notifications\Events\NotificationFailed;
13
use ZendService\Apple\Apns\Response\Message as Response;
14
use ZendService\Apple\Apns\Client\Feedback as FeedbackClient;
15
use ZendService\Apple\Apns\Response\Feedback as FeedbackResponse;
16
17
class ApnChannel
18
{
19
    const SANDBOX = 0;
20
    const PRODUCTION = 1;
21
22
    /** @var string */
23
    protected $environment;
24
25
    /** @var string */
26
    protected $certificate;
27
28
    /** @var string|null */
29
    protected $passPhrase;
30
31
    /** @var \ZendService\Apple\Apns\Client\Message */
32
    protected $client;
33
34
    /** @var \ZendService\Apple\Apns\Client\Feedback */
35
    protected $feedbackClient;
36
37
    /** @var \Illuminate\Events\Dispatcher */
38
    protected $events;
39
40
    /**
41
     * @param \ZendService\Apple\Apns\Client\Message $client
42
     * @param \ZendService\Apple\Apns\Client\Feedback $feedbackClient
43
     * @param \Illuminate\Events\Dispatcher $events
44
     * @param string $environment
45
     * @param string $certificate
46
     * @param string|null $passPhrase
47
     */
48 2
    public function __construct(
49
        Client $client,
50
        FeedbackClient $feedbackClient,
51
        Dispatcher $events,
52
        $environment,
53
        $certificate,
54
        $passPhrase = null
55
    ) {
56 2
        $this->client = $client;
57 2
        $this->feedbackClient = $feedbackClient;
58 2
        $this->events = $events;
59 2
        $this->environment = $environment;
60 2
        $this->certificate = $certificate;
61 2
        $this->passPhrase = $passPhrase;
62 2
    }
63
64
    /**
65
     * Send the notification to Apple Push Notification Service.
66
     *
67
     * @param mixed $notifiable
68
     * @param \Illuminate\Notifications\Notification $notification
69
     *
70
     * @throws \NotificationChannels\Apn\Exceptions\SendingFailed
71
     */
72 1
    public function send($notifiable, Notification $notification)
73
    {
74 1
        $tokens = (array) $notifiable->routeNotificationFor('apn');
75 1
        if (! $tokens) {
0 ignored issues
show
Bug Best Practice introduced by
The expression $tokens of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent.

Consider making the comparison explicit by using empty(..) or ! empty(...) instead.

Loading history...
76
            return;
77
        }
78
79 1
        $message = $notification->toApn($notifiable);
0 ignored issues
show
Bug introduced by
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...
80 1
        if (! $message) {
81
            return;
82
        }
83
84 1
        $this->openConnection();
85
86 1
        foreach ($tokens as $token) {
87
            try {
88 1
                $alert = new Alert();
89 1
                $alert->setTitle($message->title);
90 1
                $alert->setBody($message->body);
91
92 1
                $packet = new Packet();
93 1
                $packet->setToken($token);
94 1
                $packet->setBadge($message->badge);
95 1
                $packet->setSound($message->sound);
96 1
                $packet->setCategory($message->category);
97 1
                $packet->setContentAvailable($message->contentAvailable);
98 1
                $packet->setAlert($alert);
99 1
                $packet->setCustom($message->custom);
100
101 1
                $response = $this->client->send($packet);
102
103 1
                if ($response->getCode() !== Response::RESULT_OK) {
104 1
                    $this->events->fire(
105 1
                        new NotificationFailed($notifiable, $notification, $this, [
0 ignored issues
show
Documentation introduced by
$this is of type this<NotificationChannels\Apn\ApnChannel>, but the function expects a string.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
106 1
                            'token' => $token,
107 1
                            'error' => $response->getCode(),
108
                        ])
109
                    );
110
                    //connection is useless so create a new connection
111 1
                    $this->closeConnection();
112 1
                    $this->openConnection();
113
                }
114
            } catch (Exception $e) {
115 1
                throw SendingFailed::create($e);
116
            }
117
        }
118
119 1
        $this->closeConnection();
120 1
    }
121
122
    /**
123
     * Get feedback from the Apple Feedback Service about failed deliveries.
124
     *
125
     * @return array|ApnFeedback[]
126
     * @throws Exceptions\ConnectionFailed
127
     */
128 1
    public function getFeedback()
129
    {
130 1
        $client = $this->feedbackClient;
131
132
        try {
133 1
            $client->open($this->environment, $this->certificate, $this->passPhrase);
134
        } catch (Exception $exception) {
135
            throw Exceptions\ConnectionFailed::create($exception);
136
        }
137
138 1
        $feedback = [];
139
140
        /** @var FeedbackResponse $response */
141 1
        foreach ($client->feedback() as $response) {
142 1
            $feedback[] = new ApnFeedback($response->getToken(), $response->getTime());
143
        }
144
145 1
        $client->close();
146
147 1
        return $feedback;
148
    }
149
150
    /**
151
     * Open the connection.
152
     *
153
     * @throws \NotificationChannels\Apn\Exceptions\ConnectionFailed
154
     */
155 1
    private function openConnection()
156
    {
157
        try {
158 1
            $this->client->open($this->environment, $this->certificate, $this->passPhrase);
159
        } catch (Exception $exception) {
160
            throw Exceptions\ConnectionFailed::create($exception);
161
        }
162 1
    }
163
164
    /**
165
     * Close the connection.
166
     */
167 1
    private function closeConnection()
168
    {
169 1
        $this->client->close();
170 1
    }
171
}
172