NotificationChannels\Apn\ApnChannel

Complexity

Total Complexity

5

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	send()	0	14	2
A	sendNotifications()	0	12	2

<?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);

The method toApn() does not seem to exist on object<Illuminate\Notifications\Notification>.

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

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

$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.

The call to ApnChannel::sendNotifications() has too many arguments starting with $tokens.

    }

    /**
     * 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();
    }
}