NotificationChannels\Jusibe\JusibeChannel

Complexity

Total Complexity

7

Size/Duplication

Total Lines	63
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
B	send()	0	28	6

<?php

namespace NotificationChannels\Jusibe;

use DomainException;
use Illuminate\Notifications\Notification;
use NotificationChannels\Jusibe\Exceptions\CouldNotSendNotification;
use Unicodeveloper\Jusibe\Jusibe as JusibeClient;

class JusibeChannel
{
    /**
     * The Jusibe client instance.
     *
     * @var \Jusibe\Jusibe
     */
    protected $jusibe;

    /**
     * The phone number notifications should be sent from.
     *
     * @var string
     */
    protected $from;

    /**
     * Create a new Jusibe channel instance.
     *
     * @param  JusibeClient  $jusibe
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct(JusibeClient $jusibe)
    {
        $this->jusibe = $jusibe;

It seems like $jusibe of type object<Unicodeveloper\Jusibe\Jusibe> is incompatible with the declared type object<Jusibe\Jusibe> of property $jusibe.

    }

    /**
     * Send the given notification.
     *
     * @param  mixed  $notifiable
     * @param  \Illuminate\Notifications\Notification  $notification
     * @return string
     */
    public function send($notifiable, Notification $notification)
    {
        if (! $to = $notifiable->routeNotificationFor('jusibe')) {
            throw CouldNotSendNotification::missingTo();
        }

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

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

        if (is_string($message)) {
            $message = new JusibeMessage($message);
        }

        if (! $from = $message->from ?: config('services.jusibe.sms_from')) {
            throw CouldNotSendNotification::missingFrom();
        }

        try {
            $response = $this->jusibe->sendSMS([
                'to' => $to,
                'from' => $from,
                'message' => trim($message->content)
            ])->getResponse();

            return $response;
        } catch(DomainException $e) {
            throw CouldNotSendNotification::serviceRespondedWithAnError($exception);
        }
    }
}