NotificationChannels\GoSms\GoSmsChannel

Complexity

Total Complexity

8

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	1	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	send()	0	16	3
B	sendMessage()	0	31	4

<?php

namespace NotificationChannels\GoSms;

use Illuminate\Notifications\Notification;
use NotificationChannels\GoSms\Exceptions\CouldNotSendNotification;

class GoSmsChannel
{
    /** @var \NotificationChannels\GoSms\GoSmsApi */
    protected $gosms;

    public function __construct(GoSmsApi $gosms)
    {
        $this->gosms = $gosms;
    }

    /**
     * Send the given notification.
     *
     * @param  mixed  $notifiable
     * @param  \Illuminate\Notifications\Notification  $notification
     *
     * @throws  \NotificationChannels\GoSms\Exceptions\CouldNotSendNotification
     */
    public function send($notifiable, Notification $notification)
    {
        $to = $notifiable->routeNotificationFor('gosms');

        if (empty($to)) {
            throw CouldNotSendNotification::missingRecipient();
        }

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

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

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

        $this->sendMessage($to, $message);
    }

    protected function sendMessage($recipient, GoSmsMessage $message)
    {
        // if (mb_strlen($message->content) > 800) {

53% of this comment could be valid code. Did you maybe forget this after debugging?

        //     throw CouldNotSendNotification::contentLengthLimitExceeded();

56% of this comment could be valid code. Did you maybe forget this after debugging?

        // }

        $message->content = html_entity_decode($message->content, ENT_QUOTES, 'utf-8');
        $message->content = urlencode($message->content);

        //the sms format must start with 6
        $valid_mobile = '';
        if ($recipient[0] == '0') {
            $valid_mobile = '6'.$recipient;
        }

        if ($recipient[0] == '+') {
            $valid_mobile = substr($recipient, 1);
        }

        $params = [
            'hp'        => $valid_mobile,
            'mesg'      => $message->content,
            'sender'    => $message->from,
        ];

        if ($message->sendAt instanceof \DateTimeInterface) {
            $params['time'] = '0'.$message->sendAt->getTimestamp();
        }

        $this->gosms->send($params);
    }
}