Completed
Push — master ( f863d7...e85477 )
by Hiraku
11s
created

CurlRemoteFilesystem::getContents()   B

Complexity

Conditions 6
Paths 48

Size

Total Lines 26

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 12
CRAP Score 7.7999

Importance

Changes 0
Metric Value
dl 0
loc 26
ccs 12
cts 19
cp 0.6316
rs 8.8817
c 0
b 0
f 0
cc 6
nc 48
nop 4
crap 7.7999
1
<?php
2
/*
3
 * hirak/prestissimo
4
 * @author Hiraku NAKANO
5
 * @license MIT https://github.com/hirak/prestissimo
6
 */
7
namespace Hirak\Prestissimo;
8
9
use Composer\Config;
10
use Composer\IO;
11
use Composer\Util;
12
13
class CurlRemoteFilesystem extends Util\RemoteFilesystem
14
{
15
    protected $io;
16
    protected $config;
17
    protected $options;
18
    protected $disableTls;
19
20
    private $req;
21
22
    /**
23
     * @inheritDoc
24
     */
25 1
    public function __construct(IO\IOInterface $io, Config $config = null, array $options = array(), $disableTls = false)
26
    {
27 1
        $this->io = $io;
28 1
        $this->config = $config;
29 1
        $this->options = $options;
30 1
        $this->disableTls = $disableTls;
31 1
        parent::__construct($io, $config, $options, $disableTls);
32 1
    }
33
34
    /**
35
     * @inheritDoc
36
     */
37 1
    public function getContents($originUrl, $fileUrl, $progress = true, $options = array())
38
    {
39 1
        $res = null;
40 1
        if (isset($options['http']['header'])) {
41
            $headers = $options['http']['header'];
42 1
        } elseif (isset($options['https']['header'])) {
43
            $headers = $options['https']['header'];
44
        } else {
45 1
            $headers = array();
46
        }
47
        try {
48 1
            $this->req = new FetchRequest($fileUrl, $this->io, $this->config);
0 ignored issues
show
Bug introduced by
It seems like $this->config can be null; however, __construct() does not accept null, maybe add an additional type check?

Unless you are absolutely sure that the expression can never be null because of other conditions, we strongly recommend to add an additional type check to your code:

/** @return stdClass|null */
function mayReturnNull() { }

function doesNotAcceptNull(stdClass $x) { }

// With potential error.
function withoutCheck() {
    $x = mayReturnNull();
    doesNotAcceptNull($x); // Potential error here.
}

// Safe - Alternative 1
function withCheck1() {
    $x = mayReturnNull();
    if ( ! $x instanceof stdClass) {
        throw new \LogicException('$x must be defined.');
    }
    doesNotAcceptNull($x);
}

// Safe - Alternative 2
function withCheck2() {
    $x = mayReturnNull();
    if ($x instanceof stdClass) {
        doesNotAcceptNull($x);
    }
}
Loading history...
49 1
            foreach ($headers as $header) {
50
                list($key, $val) = explode(':', $header, 2);
51
                $this->req->addHeader($key, $val);
52 1
            }
53 1
            $res = $this->req->fetch();
54 1
        } catch (\Exception $e) {
55
            $this->io->writeError((string)$e);
56
        }
57 1
        if (false === $res) {
58 1
            return parent::getContents($originUrl, $fileUrl, $progress, $options);
59
        } else {
60
            return $res;
61
        }
62
    }
63
64 1
    public function getLastHeaders()
65
    {
66 1
        if ($this->req && ($headers = $this->req->getLastHeaders())) {
67
            return $headers;
68
        } else {
69 1
            return parent::getLastHeaders();
70
        }
71
    }
72
73 1
    public function __debugInfo()
74
    {
75 1
        return array();
76
    }
77
}
78