Tzsk\Sms\Drivers\Farazsms

Complexity

Total Complexity

5

Size/Duplication

Total Lines	67
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	5	5	1
A	send()	15	15	3
A	payload()	11	11	1

<?php

namespace Tzsk\Sms\Drivers;

use Tzsk\Sms\Abstracts\Driver;
use \SoapClient;

class Farazsms extends Driver

This class seems to be duplicated in your project.

{
    /**
     * Tsms Settings.
     *
     * @var null|object
     */
    protected $settings;

    /**
     * Smsir Client.
     *
     * @var null|SoapClient
     */
    protected $client;

    /**
     * Construct the class with the relevant settings.
     *
     * @param $settings
     * @throws \SoapFault
     */
    public function __construct($settings)
    {
        $this->settings = (object) $settings;
        $this->client = new Client();

It seems like new \Tzsk\Sms\Drivers\Client() of type object<Tzsk\Sms\Drivers\Client> is incompatible with the declared type null|object<SoapClient> of property $client.

    }

    /**
     * Send text message and return response.
     *
     * @return object
     * @throws \GuzzleHttp\Exception\GuzzleException
     */
    public function send()
    {
        $response = collect();

        foreach ($this->recipients as $recipient) {
            $result = $this->client->request(

The method request() does not exist on SoapClient. Did you maybe mean __doRequest()?

                'POST',
                $this->settings->url,
                $this->payload($recipient)
            );
            $response->put($recipient, json_decode($result));
        }

        return (count($this->recipients) == 1) ? $response->first() : $response;
    }

    /**
     * @param string $recipient
     * @param string $token

There is no parameter named $token. Was it maybe removed?

     * @return array
     */
    protected function payload($recipient)
    {
        return [
            'uname' => $this->settings->username,
            'pass' => $this->settings->password,
            'from' => $this->settings->from,
            'message' => $this->body,
            'to' => json_encode([$recipient]),
            'op'=>'send'
        ];
    }
}