Hirak\Prestissimo\FetchRequest

Complexity

Total Complexity

15

Size/Duplication

Total Lines	97
Duplicated Lines	11.34 %

Coupling/Cohesion

Components	2
Dependencies	3

Test Coverage

Coverage

83.72%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getCurlOptions()	0	7	1
A	getCurl()	0	13	2
A	fetch()	0	19	4
A	getLastError()	0	8	3
A	getLastHeaders()	0	4	1
A	__construct()	11	11	3
A	headerCallback()	0	6	1

<?php
/*
 * hirak/prestissimo
 * @author Hiraku NAKANO
 * @license MIT https://github.com/hirak/prestissimo
 */
namespace Hirak\Prestissimo;

use Composer\IO;
use Composer\Config;

class FetchRequest extends BaseRequest
{
    protected static $defaultCurlOptions = array(
        CURLOPT_HTTPGET => true,
        CURLOPT_FOLLOWLOCATION => true,
        CURLOPT_MAXREDIRS => 20,
        CURLOPT_ENCODING => '',
        CURLOPT_RETURNTRANSFER => true,
    );

    private $headers = array();

Consider using a different property name as you override a private property of the parent class.

    private $errno;
    private $error;

    /**
     * @param string $url
     * @param IO\IOInterface $io
     * @param Config $config
     */
    public function __construct($url, IO\IOInterface $io, Config $config)

This method seems to be duplicated in your project.

    {
        $this->setURL($url);
        $this->setCA($config->get('capath'), $config->get('cafile'));
        $this->setupAuthentication(
            $io,
            false,
            $config->get('github-domains') ?: array(),
            $config->get('gitlab-domains') ?: array()
        );
    }

    public function getCurlOptions()
    {
        $curlOpts = parent::getCurlOptions();
        $curlOpts[CURLOPT_RETURNTRANSFER] = true;
        $curlOpts[CURLOPT_HEADERFUNCTION] = array($this, 'headerCallback');
        return $curlOpts;
    }

    private static function getCurl($key)
    {
        static $curlCache = array();

        if (isset($curlCache[$key])) {
            return $curlCache[$key];
        }

        $ch = curl_init();
        Share::setup($ch);

        return $curlCache[$key] = $ch;
    }

    /**
     * @return string|false
     */
    public function fetch()
    {
        $ch = self::getCurl($this->getOriginURL());
        curl_setopt_array($ch, $this->getCurlOptions());

        $result = curl_exec($ch);

        $this->errno = $errno = curl_errno($ch);
        $this->error = curl_error($ch);
        $info = curl_getinfo($ch);

        if (!$this->isHTTP()) {
            return $result;
        } elseif ($errno === CURLE_OK && in_array($info['http_code'], array(200, 304))) {
            return $result;
        } else {
            return false;
        }
    }

    public function getLastError()
    {
        if ($this->errno || $this->error) {
            return array($this->errno, $this->error);
        } else {
            return array();
        }
    }

    public function getLastHeaders()
    {
        return $this->headers;
    }

    public function headerCallback($ch, $headerString)
    {
        $len = strlen($headerString);
        $this->headers[] = $headerString;
        return $len;
    }
}