NotificationChannels\AfricasTalking\AfricasTalkingChannel

Complexity

Total Complexity

4

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Importance

Changes	2
Bugs	1	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	send()	0	16	3
A	__construct()	0	3	1

<?php

namespace NotificationChannels\AfricasTalking;

use AfricasTalking\SDK\AfricasTalking as AfricasTalkingSDK;
use Exception;
use Illuminate\Notifications\Notification;
use NotificationChannels\AfricasTalking\Exceptions\CouldNotSendNotification;

class AfricasTalkingChannel
{
    /** @var AfricasTalkingSDK */
    protected $africasTalking;

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

    /**
     * Send the given notification.
     *
     * @param mixed $notifiable
     * @param \Illuminate\Notifications\Notification $notification
     * @throws CouldNotSendNotification
     */
    public function send($notifiable, Notification $notification)
    {
        $message = $notification->toAfricasTalking($notifiable);

The method toAfricasTalking() does not exist on Illuminate\Notifications\Notification. It seems like you code against a sub-type of Illuminate\Notifications\Notification such as NotificationChannels\AfricasTalking\Test\TestNotification.

     
        if (!$phoneNumber = $notifiable->routeNotificationFor('africasTalking')) {
            $phoneNumber = $notifiable->phone_number;
        }

        try {
            $this->africasTalking->send([

The method send() does not exist on AfricasTalking\SDK\AfricasTalking.

                'to' => $phoneNumber,
                'message' => $message->getContent(),
                'from' => $message->getSender(),
            ]);
        } catch (Exception $e) {
            throw CouldNotSendNotification::serviceRespondedWithAnError($e->getMessage());
        }
    }
}