Messenger\SMSProviders\TwilioSMS

Complexity

Total Complexity

7

Size/Duplication

Total Lines	86
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	isValid()	0	13	3
A	send()	0	26	3

<?php

namespace Messenger\SMSProviders;

use Twilio\Rest\Client;

class TwilioSMS implements SMSProvider
{
    /**
     * @var Client
     */
    var $client;

    /**
     * @var string
     */
    var $sid;

    /**
     * @var string
     */
    var $number;

    /**
     * TwilioMessage constructor.
     *
     * @param $sid
     * @param $token
     * @param $number
     * @throws \Twilio\Exceptions\ConfigurationException
     */
    function __construct($sid, $token, $number)

It is generally recommended to explicitly declare the visibility for methods.

    {
        $this->client = new Client($sid, $token);
        $this->sid = $sid;
        $this->number = $number;
    }

    /**
     * Sends an sms message.
     *
     * @param string $to
     * @param string $message
     * @return array
     */
    public function send(string $to, string $message)
    {
        try {
            $result = $this
                ->client
                ->messages
                ->create('+' . $to, ['from' => $this->number, 'body' => $message]);

            $sid = $result->sid;
        } catch (\Exception $e) {
            return [
                'error' => 'Error sending SMS: ' . $e->getMessage(),
                'sent' => false,
            ];
        }

        if (empty($sid)) {
            return [
                'error' => 'Error sending SMS: Unknown error',
                'sent' => false,
            ];
        }

        return [
            'message_id' => $sid,
            'sent' => true,
        ];
    }

    /**
     * Checks providers credentials.
     *
     * @return bool
     */
    public function isValid()
    {
        try {
            $account = $this->client->api->v2010->accounts($this->sid)->fetch();
        } catch (\Exception $e) {
            return false;
        }

        if ($account) {

$account is of type Twilio\Rest\Api\V2010\AccountInstance, thus it always evaluated to true.

            return true;
        }

        return false;
    }
}