Akibatech\FreeMobileSms\FreeMobileSms

Complexity

Total Complexity

5

Size/Duplication

Total Lines	93
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	3
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	15	2
A	loadCredentialsFromConfig()	0	7	1
A	newClient()	0	10	1
A	send()	0	6	1

<?php

namespace Akibatech\FreeMobileSms;

/**
 * Class FreeMobileSms
 *
 * @package Akibatech\FreeMobileSms
 */
class FreeMobileSms
{
    /**
     * API endpoint.
     *
     * @var string
     */
    protected static $endpoint = 'https://smsapi.free-mobile.fr/sendmsg';

    /**
     * @var array
     */
    private $credentials = [
        'user' => null,
        'pass' => null
    ];

    //-------------------------------------------------------------------------

    /**
     * FreeMobileSms constructor.
     *
     * @param   void
     * @return  self

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct(array $credentials = [])
    {
        $credentials = collect($credentials);

        if ($credentials->isEmpty() === false)
        {
            $this->credentials = collect($credentials)->only([

It seems like collect($credentials)->only(array('user', 'pass')) of type object<Illuminate\Support\Collection> is incompatible with the declared type array of property $credentials.

                'user',
                'pass',
            ]);
        } else
        {
            $this->loadCredentialsFromConfig();
        }
    }

    //-------------------------------------------------------------------------

    /**
     * Load API credentials from Laravel config.
     *
     * @param   void
     * @return  self
     */
    protected function loadCredentialsFromConfig()
    {
        $this->credentials['user'] = config('laravel-free-mobile-sms.user');
        $this->credentials['pass'] = config('laravel-free-mobile-sms.pass');

        return $this;
    }

    //-------------------------------------------------------------------------

    /**
     * Create the CurlClient client.
     *
     * @param   string $message
     * @return  CurlClient
     */
    protected function newClient($message)
    {
        $client = new CurlClient(self::$endpoint, [
            'user' => $this->credentials['user'],
            'pass' => $this->credentials['pass'],
            'msg'  => $message
        ]);

        return $client;
    }

    //-------------------------------------------------------------------------

    /**
     * Send a new message.
     *
     * @param   string $message
     * @return  int
     */
    public function send($message = '')
    {
        $return = $this->newClient($message)->hit();

        return $return;
    }

    //-------------------------------------------------------------------------
}