Fenerum\ApiClient

Complexity

Total Complexity

15

Size/Duplication

Total Lines	143
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	12	1
A	get()	0	4	1
A	post()	0	4	1
A	put()	0	4	1
A	patch()	0	4	1
A	delete()	0	4	1
A	request()	0	20	3
A	handleException()	0	15	2
A	isEnabled()	0	4	2
A	logger()	0	6	2

<?php

declare(strict_types=1);

namespace Fenerum;

use Fenerum\API\Exceptions\FenerumApiException;
use GuzzleHttp\Client;
use GuzzleHttp\Exception\GuzzleException;
use Illuminate\Support\Facades\Log;

class ApiClient
{
    /** @var \GuzzleHttp\Client */
    protected $client;

    public function __construct()
    {
        $this->client = new Client([
            'base_uri' => config('fenerum.base_uri'),
            'headers' => [
                'Authorization' => 'Token '.config('fenerum.api_token'),
                'Content-Type' => 'application/json',
                'Accept' => 'application/json',
            ],
            'timeout'  => 15.0,
        ]);
    }

    /**
     * @param string $url
     * @return array|null
     * @throws \Fenerum\API\Exceptions\FenerumApiException
     */
    public function get(string $url): ?array
    {
        return $this->request('GET', $url);
    }

    /**
     * @param string $url
     * @param array $data
     * @return array|null
     * @throws \Fenerum\API\Exceptions\FenerumApiException
     */
    public function post(string $url, array $data): ?array
    {
        return $this->request('POST', $url, $data);
    }

    /**
     * @param string $url
     * @param array $data
     * @return array
     * @throws \Fenerum\API\Exceptions\FenerumApiException
     */
    public function put(string $url, array $data): ?array
    {
        return $this->request('PUT', $url, $data);
    }

    /**
     * @param string $url
     * @param array $data
     * @return array|null
     * @throws \Fenerum\API\Exceptions\FenerumApiException
     */
    public function patch(string $url, array $data): ?array
    {
        return $this->request('PATCH', $url, $data);
    }

    /**
     * @param string $url
     * @return array|null
     * @throws \Fenerum\API\Exceptions\FenerumApiException
     */
    public function delete(string $url): ?array
    {
        return $this->request('DELETE', $url);
    }

    /**
     * Send Request.
     *
     * @param string $method
     * @param string $uri
     * @param array|null $payload
     * @param array $headers
     * @return array|null
     * @throws \Fenerum\API\Exceptions\FenerumApiException
     */
    public function request(string $method, string $uri, $payload = null, array $headers = []): ?array
    {
        if (! $this->isEnabled()) {
            $this->logger('is not enabled. Please review config');

            return null;
        }

        try {
            $response = $this->client->request($method, $uri, [
                'json' => $payload,
                'headers' => $headers,
            ]);
            $this->logger("Request url: {$uri}", $payload ?? []);

            return json_decode($response->getBody()->getContents(), true);
        } catch (GuzzleException $exception) {
            $this->handleException($exception, $uri);
        }
    }

    /**
     * @param GuzzleException $exception
     * @param string $uri
     * @throws \Fenerum\API\Exceptions\FenerumApiException
     */
    private function handleException(GuzzleException $exception, string $uri): void
    {
        $response = $exception->getResponse();

It seems like you code against a concrete implementation and not the interface GuzzleHttp\Exception\GuzzleException as the method getResponse() does only exist in the following implementations of said interface: GuzzleHttp\Exception\BadResponseException, GuzzleHttp\Exception\ClientException, GuzzleHttp\Exception\ConnectException, GuzzleHttp\Exception\RequestException, GuzzleHttp\Exception\ServerException, GuzzleHttp\Exception\TooManyRedirectsException.

        $code = $exception->getCode();
        $body = $exception->getMessage();

        if (null !== $response) {
            $code = $response->getStatusCode();
            $body = $response->getBody()->getContents();
        }

        $this->logger("Response Error: url: {$uri} Response: {$body}. Response code: $code.");

        throw new FenerumApiException($body, $code, $exception);
    }

    /**
     * Makes sure that all config objects are set before we can use this.
     * @return bool
     */
    private function isEnabled(): bool
    {
        return null !== config('fenerum.base_uri') && null !== config('fenerum.api_token');
    }

    /**
     * @param string $string
     * @param array|null $context
     */
    private function logger(string $string, ?array $context = []): void
    {
        if (true === config('fenerum.debug')) {
            Log::debug(\get_class($this) . ' ' . $string, $context);

It seems like $context defined by parameter $context on line 148 can also be of type null; however, Illuminate\Support\Facades\Log::debug() does only seem to accept array, maybe add an additional type check?

        }
    }
}