Tzsk\Sms\Drivers\Textlocal

Complexity

Total Complexity

5

Size/Duplication

Total Lines	63
Duplicated Lines	19.05 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	send()	12	12	3
A	payload()	0	12	1

<?php

namespace Tzsk\Sms\Drivers;

use GuzzleHttp\Client;
use Tzsk\Sms\Abstracts\Driver;

class Textlocal extends Driver
{
    /**
     * Textlocal Settings.
     *
     * @var object
     */
    protected $settings;

    /**
     * Http Client.
     *
     * @var Client
     */
    protected $client;

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

    /**
     * Send text message and return response.
     *
     * @return mixed
     */
    public function send()

This method seems to be duplicated in your project.

    {
        $response = collect();
        foreach ($this->recipients as $recipient) {
            $response->put(
                $recipient,
                $this->client->request('POST', $this->settings->url, $this->payload($recipient))
            );
        }

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

    /**
     * @param string $recipient
     * @return array
     */
    public function payload($recipient)
    {
        return [
            'form_params' => [
                'username' => $this->settings->username,
                'hash' => $this->settings->hash,
                'numbers' => $recipient,
                'sender' => urlencode($this->settings->sender),
                'message' => $this->body,
            ],
        ];
    }
}