MusicBrainz\HttpAdapters\GuzzleFiveAdapter

Complexity

Total Complexity

7

Size/Duplication

Total Lines	68
Duplicated Lines	27.94 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	8	8	2
B	call()	11	33	5

<?php
namespace MusicBrainz\HttpAdapters;

use GuzzleHttp\ClientInterface;
use MusicBrainz\Exception;

class GuzzleFiveAdapter extends AbstractHttpAdapter
{
    /**
     * @var ClientInterface
     */
    private $client;

    /**
     * Initialize class
     *
     * @param ClientInterface $client
     * @param null $endpoint
     */
    public function __construct(ClientInterface $client, $endpoint = NULL)

This method seems to be duplicated in your project.

    {
        $this->client = $client;

        if (filter_var($endpoint, FILTER_VALIDATE_URL)) {
            $this->endpoint = $endpoint;
        }
    }

    /**
     * Perform an HTTP request on MusicBrainz
     *
     * @param  string $path
     * @param  array $params
     * @param  array $options
     * @param  boolean $isAuthRequired
     * @param  boolean $returnArray
     *
     * @throws Exception
     * @return array
     */
    public function call($path, array $params = array(), array $options = array(), $isAuthRequired = FALSE, $returnArray = FALSE)
    {
        if ($options['user-agent'] == '') {
            throw new Exception('You must set a valid User Agent before accessing the MusicBrainz API');
        }

        $requestOptions = [
            'headers'        => [
                'Accept'     => 'application/json',
                'User-Agent' => $options['user-agent']
            ],
            'query' => $params
        ];

        if ($isAuthRequired) {

This code seems to be duplicated across your project.

            if ($options['user'] != NULL && $options['password'] != NULL) {
                $requestOptions['auth'] = [
                    'username' => $options['user'],
                    'password' => $options['password'],
                    CURLAUTH_DIGEST
                ];
            } else {
                throw new Exception('Authentication is required');
            }
        }

        $request = $this->client->createRequest('GET', $this->endpoint . '/' . $path, $requestOptions);

        // musicbrainz throttle
        sleep(1);

        return $this->client->send($request)->json();
    }
}