Tzsk\Sms\Drivers\Nexmo

Complexity

Total Complexity

4

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	send()	0	10	3

<?php

namespace Tzsk\Sms\Drivers;

use Nexmo\Message\Text;
use Nexmo\Application\Client;
use Tzsk\Sms\Abstracts\Driver;
use Nexmo\Client\Credentials\Basic;

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

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

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

The call to Client::__construct() has too many arguments starting with new \Nexmo\Client\Credentials\Basic($this->settings->key, $this->settings->secret).

        );
    }

    /**
     * Send text message and return response.
     *
     * @return object
     */
    public function send()
    {
        $response = collect();
        foreach ($this->recipients as $recipient) {
            $text = new Text($recipient, $this->settings->from, $this->body);
            $response->put($recipient, $this->client->message()->send($text));

The method message() does not seem to exist on object<Nexmo\Application\Client>.

        }

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