Tzsk\Sms\Drivers\Kavenegar

Complexity

Total Complexity

6

Size/Duplication

Total Lines	76
Duplicated Lines	13.16 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	send()	10	21	3
A	getSmsResponse()	0	14	2

<?php
namespace Tzsk\Sms\Drivers;

use Kavenegar\KavenegarApi;
use Tzsk\Sms\Abstracts\Driver;

class Kavenegar extends Driver
{
    /**
     * Kavenegar Settings.
     *
     * @var null|object
     */
    protected $settings = null;

    /**
     * Kavenegar Client.
     *
     * @var null|Client
     */
    protected $client = null;

    /**
     * Construct the class with the relevant settings.
     *
     * SendSmsInterface constructor.
     * @param $settings object
     */
    public function __construct($settings)
    {
        $this->settings = (object) $settings;
        $this->client = new KavenegarApi($this->settings->apiKey);

It seems like new \Kavenegar\KavenegarApi($this->settings->apiKey) of type object<Kavenegar\KavenegarApi> is incompatible with the declared type null|object<Tzsk\Sms\Drivers\Client> of property $client.

    }

    /**
     * Send text message and return response.
     *
     * @return object
     */
    public function send()
    {
        try {
            $response = ['status' => true, 'data' =>[]];
            foreach ($this->recipients as $recipient) {

This code seems to be duplicated across your project.

                $sms = $this->client->Send(
                    $this->settings->from,
                    $recipient,
                    $this->body
                );
                $response['data'][$recipient] = $this->getSmsResponse(
                    json_decode($sms, true)
                );
            }
        } catch (\Exception $e) {
            $response['status'][$recipient] = false;
            $response['data'][$recipient] = $e->getMessage();
        }

        return (object) $response;
    }

    /**
     * Get the Kavenegar Response.
     *
     * @param $sms
     * @return object
     */
    protected function getSmsResponse($sms)
    {
        $attributes = [
            'messageid', 'message', 'status', 'statustext',
            'sender', 'receptor', 'date', 'cost',
        ];

        $res = [];
        foreach ($attributes as $attribute) {
            $res[$attribute] = $sms->$attribute;
        }

        return (object) $res;
    }
}