IlGala\SMSFactor\Adapters\GuzzleHttpAdapter

Complexity

Total Complexity

18

Size/Duplication

Total Lines	122
Duplicated Lines	50.82 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	15	4
A	get()	10	10	2
A	delete()	10	10	2
A	put()	12	12	3
A	post()	12	12	3
A	getLatestResponseHeaders()	11	11	2
A	handleError()	7	7	2

<?php

/*
 * This file is part of Laravel SMSFactor.
 *
 * (c) Filippo Galante <filippo.galante@b-ground.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace IlGala\SMSFactor\Adapters;

use GuzzleHttp\Client;
use GuzzleHttp\ClientInterface;
use GuzzleHttp\Exception\RequestException;
use GuzzleHttp\Message\ResponseInterface;
use GuzzleHttp\Psr7\Response;
use IlGala\SMSFactor\Exceptions\HttpException;

/**
 * @author Filippo Galante <filippo.galante@b-ground.com>
 */
class GuzzleHttpAdapter implements AdapterInterface
{

    /**
     * @var ClientInterface
     */
    protected $client;

    /**
     * @var Response|ResponseInterface
     */
    protected $response;

    /**
     * @param string               $username
     * @param string               $password
     * @param string               $accept
     * @param ClientInterface|null $client
     */
    public function __construct($username, $password, $accept, ClientInterface $client = null)
    {
        if (version_compare(ClientInterface::VERSION, '6') === 1) {
            $this->client = $client ?: new Client(['headers' => [

It seems like $client ?: new \GuzzleHttp\Client(array('headers' => array('sfusername' => $username, 'sfpassword' => $password, 'Accept' => $accept))) can also be of type object<GuzzleHttp\Client>. However, the property $client is declared as type object<GuzzleHttp\ClientInterface>. Maybe add an additional type check?

                    'sfusername' => $username,
                    'sfpassword' => $password,
                    'Accept' => $accept
            ]]);
        } else {
            $this->client = $client ?: new Client();

It seems like $client ?: new \GuzzleHttp\Client() can also be of type object<GuzzleHttp\Client>. However, the property $client is declared as type object<GuzzleHttp\ClientInterface>. Maybe add an additional type check?

            $this->client->setDefaultOption('headers/sfusername', $username);
            $this->client->setDefaultOption('headers/sfpassword', $password);
            $this->client->setDefaultOption('headers/Accept', $accept);
        }
    }

    /**
     * {@inheritdoc}
     */
    public function get($url)

This method seems to be duplicated in your project.

    {
        try {
            $this->response = $this->client->get($url);
        } catch (RequestException $e) {

The class GuzzleHttp\Exception\RequestException does not exist. Did you forget a USE statement, or did you not list all dependencies?

            $this->response = $e->getResponse();
            $this->handleError();
        }
        return $this->response->getBody();
    }

    /**
     * {@inheritdoc}
     */
    public function delete($url)

This method seems to be duplicated in your project.

    {
        try {
            $this->response = $this->client->delete($url);
        } catch (RequestException $e) {

The class GuzzleHttp\Exception\RequestException does not exist. Did you forget a USE statement, or did you not list all dependencies?

            $this->response = $e->getResponse();
            $this->handleError();
        }
        return $this->response->getBody();
    }

    /**
     * {@inheritdoc}
     */
    public function put($url, $content = '')

This method seems to be duplicated in your project.

    {
        $options = [];
        $options[is_array($content) ? 'json' : 'body'] = $content;
        try {
            $this->response = $this->client->put($url, $options);
        } catch (RequestException $e) {

The class GuzzleHttp\Exception\RequestException does not exist. Did you forget a USE statement, or did you not list all dependencies?

            $this->response = $e->getResponse();
            $this->handleError();
        }
        return $this->response->getBody();
    }

    /**
     * {@inheritdoc}
     */
    public function post($url, $content = '')

This method seems to be duplicated in your project.

    {
        $options = [];
        $options[is_array($content) ? 'json' : 'body'] = $content;
        try {
            $this->response = $this->client->post($url, $options);
        } catch (RequestException $e) {

The class GuzzleHttp\Exception\RequestException does not exist. Did you forget a USE statement, or did you not list all dependencies?

            $this->response = $e->getResponse();
            $this->handleError();
        }
        return $this->response->getBody();
    }

    /**
     * {@inheritdoc}
     */
    public function getLatestResponseHeaders()

This method seems to be duplicated in your project.

    {
        if (null === $this->response) {
            return;
        }
        return [
            'reset' => (int) (string) $this->response->getHeader('RateLimit-Reset'),
            'remaining' => (int) (string) $this->response->getHeader('RateLimit-Remaining'),
            'limit' => (int) (string) $this->response->getHeader('RateLimit-Limit'),
        ];
    }

    /**
     * @throws HttpException
     */
    protected function handleError()

This method seems to be duplicated in your project.

    {
        $body = (string) $this->response->getBody();
        $code = (int) $this->response->getStatusCode();
        $content = json_decode($body);
        throw new HttpException(isset($content->message) ? $content->message : 'Request not processed.', $code);
    }

}