drupol\Yaroc\Http\Client

Complexity

Total Complexity

19

Size/Duplication

Total Lines	145
Duplicated Lines	5.52 %

Coupling/Cohesion

Dependencies

13

Test Coverage

Coverage

61.36%

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	setEndpoint()	0	3	1
A	getEndpoint()	0	3	1
A	getLogger()	0	3	1
A	setLogger()	0	3	1
A	getUriFactory()	0	3	1
A	setUriFactory()	0	3	1
A	__construct()	0	12	4
A	request()	0	9	2
C	validateResponse()	8	24	7

<?php

namespace drupol\Yaroc\Http;

use drupol\Yaroc\Http\Client\Common\Plugin\LoggerPlugin;
use drupol\Yaroc\Http\Message\Formatter\Formatter;
use drupol\Yaroc\Plugin\MethodPluginInterface;
use Http\Client\Common\HttpMethodsClient;
use Http\Client\Common\Plugin\HeaderDefaultsPlugin;
use Http\Client\Common\PluginClient;
use Http\Client\Exception\NetworkException;
use Http\Client\HttpClient;
use Http\Discovery\HttpClientDiscovery;
use Http\Discovery\MessageFactoryDiscovery;
use Http\Discovery\UriFactoryDiscovery;
use Http\Message\UriFactory;
use Psr\Http\Message\ResponseInterface;
use Psr\Http\Message\UriInterface;
use Psr\Log\LoggerInterface;

/**
 * Class Client
 *
 * @package drupol\Yaroc\Http
 */
class Client extends HttpMethodsClient {

The class Client has a coupling between objects value of 21. Consider to reduce the number of dependencies under 13.

  /**
   * The default Random.org endpoint.
   *
   * @var UriInterface
   */
  protected $endpoint;

  /**
   * The URI Factory.
   *
   * @var UriFactory
   */
  protected $uriFactory;

  /**
   * The logger.
   *
   * @var \Psr\Log\LoggerInterface
   */
  protected $logger;

  /**
   * Client constructor.
   *
   * @param \Http\Client\HttpClient|NULL $httpClient
   * @param \Http\Message\UriFactory|NULL $uriFactory
   * @param \Psr\Log\LoggerInterface|NULL $logger
   */
  public function __construct(HttpClient $httpClient = NULL, UriFactory $uriFactory = NULL, LoggerInterface $logger = NULL) {
    $httpClient = $httpClient ?: HttpClientDiscovery::find();

    $plugins = [
      new HeaderDefaultsPlugin(['Content-Type' => 'application/json']),
      new LoggerPlugin($logger ?: new \drupol\Yaroc\Log\Logger(), new Formatter()),
    ];
    $httpClient = new HttpMethodsClient(new PluginClient($httpClient, $plugins), MessageFactoryDiscovery::find());

    $this->setUriFactory($uriFactory ?: UriFactoryDiscovery::find());
    parent::__construct($httpClient, MessageFactoryDiscovery::find());
  }

  /**
   * Set the Random.org endpoint.
   *
   * @param string $uri
   */
  public function setEndpoint($uri) {
    $this->endpoint = $this->getUriFactory()->createUri($uri);
  }

  /**
   * Get the Random.org endpoint.
   *
   * @return UriInterface
   */
  public function getEndpoint() {
    return $this->endpoint;
  }

  /**
   * Request.
   *
   * @param MethodPluginInterface $methodPlugin
   *
   * @return null|ResponseInterface
   */
  public function request(MethodPluginInterface $methodPlugin) {
    try {
      $response = $this->post($this->getEndpoint(), [], json_encode($methodPlugin->getParameters()));
    } catch (NetworkException $e) {
      return NULL;
    }

    return $this->validateResponse($response);
  }

  /**
   * Validate the response.
   *
   * @param \Psr\Http\Message\ResponseInterface $response
   *
   * @return \Exception|ResponseInterface
   */
  public function validateResponse(ResponseInterface $response) {
    if (200 == $response->getStatusCode()) {
      $body = json_decode((string) $response->getBody()->getContents(), TRUE);

      if (isset($body['error']['code'])) {
        switch ($body['error']['code']) {
          case -32600:

This code seems to be duplicated across your project.

            throw new \InvalidArgumentException('Invalid Request: ' . $body['error']['message'], $body['error']['code']);
          case -32601:
            throw new \BadFunctionCallException('Procedure not found: ' . $body['error']['message'], $body['error']['code']);
          case -32602:

This code seems to be duplicated across your project.

            throw new \InvalidArgumentException('Invalid arguments: ' . $body['error']['message'], $body['error']['code']);
          case -32603:

This code seems to be duplicated across your project.

            throw new \RuntimeException('Internal Error: ' . $body['error']['message'], $body['error']['code']);
          default:

This code seems to be duplicated across your project.

            throw new \RuntimeException('Invalid request/response: ' . $body['error']['message'], $body['error']['code']);
        }
      }
    }

    $response->getBody()->rewind();

    return $response;
  }

  /**
   * Get the logger.
   *
   * @return \Psr\Log\LoggerInterface
   */
  public function getLogger() {
    return $this->logger;
  }

  /**
   * Set the logger.
   *
   * @param \Psr\Log\LoggerInterface $logger
   */
  public function setLogger(LoggerInterface $logger) {
    $this->logger = $logger;
  }

  /**
   * Returns the UriFactory.
   *
   * @return \Http\Message\UriFactory
   */
  public function getUriFactory() {
    return $this->uriFactory;
  }

  /**
   * @param \Http\Message\UriFactory $uriFactory
   */
  public function setUriFactory(UriFactory $uriFactory) {
    $this->uriFactory = $uriFactory;
  }

}