WSW\SiftScience\Client\Client

Complexity

Total Complexity

7

Size/Duplication

Total Lines	78
Duplicated Lines	34.62 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	2
A	addHeader()	0	6	1
A	post()	14	14	2
A	get()	13	13	2

<?php

namespace WSW\SiftScience\Client;

use GuzzleHttp\ClientInterface;
use GuzzleHttp\Client as HttpClient;
use GuzzleHttp\Exception\RequestException;
use Psr\Http\Message\ResponseInterface;
use WSW\SiftScience\Collections\Headers;
use WSW\SiftScience\Exceptions\SiftScienceRequestException;

/**
 * Class Client
 *
 * @package WSW\SiftScience\Client
 * @author Ronaldo Matos Rodrigues <ronaldo@whera.com.br>
 */
class Client
{
    /**
     * @var \GuzzleHttp\Client
     */
    private $client;

    /**
     * @var Headers
     */
    private $headers;

    /**
     * @param ClientInterface $client
     */
    public function __construct(ClientInterface $client = null)
    {
        $this->client = $client ?: new HttpClient();

$client ?: new \GuzzleHttp\Client() is of type object<GuzzleHttp\ClientInterface>, but the property $client was declared to be of type object<GuzzleHttp\Client>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        $this->headers = new Headers();
        $this->addHeader('Content-Type', 'application/json');
        $this->addHeader('Accept', 'application/json');
    }

    /**
     * @param string $key
     * @param string $value
     *
     * @return $this
     */
    public function addHeader($key, $value)
    {
        $this->headers->add([$key => $value]);

        return $this;
    }

    /**
     * @param string $url
     * @param string $body
     * @throws SiftScienceRequestException
     *
     * @return ResponseInterface
     */
    public function post($url, $body)

This method seems to be duplicated in your project.

    {
        try {
            $response = $this->client->request('POST', $url, [
                'headers' => $this->headers->getValues(),
                'body'    => $body,
                'verify'  => false
            ]);

            return $response;
        } catch (RequestException $e) {
            throw new SiftScienceRequestException($e);
        }
    }

    /**
     * @param string $url
     * @throws SiftScienceRequestException
     *
     * @return ResponseInterface
     */
    public function get($url)

This method seems to be duplicated in your project.

    {
        try {
            $response = $this->client->request('GET', $url, [
                'headers' => $this->headers->getValues(),
                'verify' => false
            ]);

            return $response;
        } catch (RequestException $e) {
            throw new SiftScienceRequestException($e);
        }
    }
}