Xigen\ESM

Complexity

Total Complexity

7

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

96%

Importance

Changes	5
Bugs	1	Features	2

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	testConnection()	0	4	1
A	getRequestData()	0	8	2
A	__toArray()	0	4	1
A	get()	0	13	2

<?php
namespace Xigen;

use GuzzleHttp\Client;

class ESM
{
    use Shortcuts\Banner;
    use Shortcuts\Client;
    use Shortcuts\Templates;

    public $responseData;
    public $options;
    private $client;
    private $response;

The property $response is not used and could be removed.

    public function __construct($options)
    {
        $this->options = $options;
        $this->client = new Client([
            'base_url' => [$this->options['URL'], ['version' => 'v2']]
        ]);
    }

    public function testConnection()
    {
        $this->get('/ping');
    }

    public function getRequestData($JsonDecode = true)
    {
        if ($JsonDecode == true) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

            $this->responseData = $this->__toArray($this->responseData);
        }

        return $this->responseData;
    }

    public function __toArray($data)
    {
        return @json_decode($data);
    }

    private function get($endpoint)
    {
        try {
            $response = $this->client->post($this->options['URL'] . $endpoint, [
                'body' => [
                    'APIKEY' => $this->options['APIKEY']
                ]
            ]);
            $this->responseData = (string) $response->getBody();
        } catch (\GuzzleHttp\Exception\RequestException $e) {
            var_dump($e);

var_dump($e); looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data.

        }
    }
}