League\JsonReference\Loader\CurlWebLoader

Complexity

Total Complexity

11

Size/Duplication

Total Lines	93
Duplicated Lines	11.83 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

93.94%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	11	11	3
A	load()	0	17	4
A	getResponseBodyAndStatusCode()	0	7	1
A	getDefaultCurlOptions()	0	8	1
A	setCurlOptions()	0	9	2

<?php

namespace League\JsonReference\Loader;

use League\JsonReference;
use League\JsonReference\DecoderManager;
use League\JsonReference\DecoderInterface;
use League\JsonReference\LoaderInterface;

final class CurlWebLoader implements LoaderInterface
{
    /**
     * @var string
     */
    private $prefix;

    /**
     * @var array
     */
    private $curlOptions;

    /**
     * @var DecoderManager
     */
    private $decoders;

    /**
     * @param string                              $prefix
     * @param array                               $curlOptions
     * @param JsonDecoderInterface|DecoderManager $decoders
     */
    public function __construct($prefix, array $curlOptions = null, $decoders = null)

This method seems to be duplicated in your project.

    {
        $this->prefix = $prefix;
        $this->setCurlOptions($curlOptions);
        
        if ($decoders instanceof DecoderInterface) {
            $this->decoders = new DecoderManager([$decoders]);
        } else {
            $this->decoders = $decoders ?: new DecoderManager();

It seems like $decoders ?: new \League\JsonReference\DecoderManager() can also be of type object<League\JsonReference\Loader\JsonDecoderInterface>. However, the property $decoders is declared as type object<League\JsonReference\DecoderManager>. Maybe add an additional type check?

        }
    }

    /**
     * {@inheritdoc}
     */
    public function load($path, $defaultExtension = 'json')
    {
        $uri = $this->prefix . $path;
        
        $extension = isset(pathinfo($path)['extension']) ? pathinfo($path)['extension'] : $defaultExtension;
        
        $ch = curl_init($uri);
        curl_setopt_array($ch, $this->curlOptions);
        list($response, $statusCode) = $this->getResponseBodyAndStatusCode($ch);
        curl_close($ch);

        if ($statusCode >= 400 || !$response) {
            throw JsonReference\SchemaLoadingException::create($uri);
        }

        return $this->decoders->getDecoder($extension)->decode($response);
    }

    /**
     * @param resource $ch
     *
     * @return array
     */
    private function getResponseBodyAndStatusCode($ch)
    {
        $response   = curl_exec($ch);
        $statusCode = curl_getinfo($ch, CURLINFO_HTTP_CODE);

        return [$response, $statusCode];
    }

    /**
     * @return array
     */
    private function getDefaultCurlOptions()
    {
        return [
            CURLOPT_RETURNTRANSFER => true,
            CURLOPT_FOLLOWLOCATION => true,
            CURLOPT_MAXREDIRS      => 20,
        ];
    }

    /**
     * @param array|null $curlOptions
     */
    private function setCurlOptions($curlOptions)
    {
        if (is_array($curlOptions)) {
            $this->curlOptions = $curlOptions + $this->getDefaultCurlOptions();
            return;
        }

        $this->curlOptions = $this->getDefaultCurlOptions();
    }
}