Palmtree\Curl\Curl

Complexity

Total Complexity

21

Size/Duplication

Total Lines	143
Duplicated Lines	0 %

Test Coverage

Coverage

83.64%

Importance

Changes	3
Bugs	0	Features	0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	getContents()	0	5	1
A	__construct()	0	8	1
A	setOpt()	0	11	3
A	setUrl()	0	5	1
A	__toString()	0	3	2
A	buildCurlOpts()	0	8	1
A	getResponse()	0	7	2
A	getRequest()	0	3	1
A	execute()	0	21	4
A	postJson()	0	10	2
A	getUrl()	0	3	1
A	getOpts()	0	3	1
A	post()	0	5	1

<?php

namespace Palmtree\Curl;

use Palmtree\Curl\Exception\BadMethodCallException;
use Palmtree\Curl\Exception\InvalidArgumentException;

class Curl
{
    public static $defaultCurlOpts = [
        CURLOPT_RETURNTRANSFER => true,
        CURLOPT_FOLLOWLOCATION => true,
        CURLOPT_AUTOREFERER    => true,
        CURLOPT_USERAGENT      => 'Palmtree\Curl',
    ];

    /** @var string */
    private $url;
    /** @var resource */
    private $handle;
    /** @var Request */
    private $request;
    /** @var Response */
    private $response;
    /** @var array */
    private $curlOpts = [];

    const HTTP_NOT_FOUND = 404;
    const HTTP_OK_MIN    = 200;
    const HTTP_OK_MAX    = 299;

    public function __construct(string $url, array $curlOpts = [])
    {
        $this->setUrl($url);

        $this->handle  = \curl_init($url);

It seems like curl_init($url) can also be of type false. However, the property $handle is declared as type resource. Maybe add an additional type check?

        $this->request = new Request();

        $this->buildCurlOpts($curlOpts);
    }

    public static function getContents(string $url, array $curlOpts = []): string
    {
        $curl = new self($url, $curlOpts);

        return $curl->getResponse()->getBody();
    }

    /**
     * @param string|array $data
     */
    public function post($data): Response
    {
        $this->getRequest()->setBody($data);

        return $this->execute();
    }

    public function postJson($json): Response
    {
        if (!\is_string($json)) {
            $json = \json_encode($json);
        }

        $this->getRequest()->addHeader('Content-Type', 'application/json');
        $this->getRequest()->addHeader('Content-Length', \strlen($json));

        return $this->post($json);
    }

    public function execute(): Response
    {
        if ($this->response !== null) {
            throw new BadMethodCallException('Request has already been executed');
        }

        if ($headers = $this->getRequest()->getHeaderStrings()) {
            \curl_setopt($this->handle, CURLOPT_HTTPHEADER, $headers);
        }

        if ($body = $this->getRequest()->getBody()) {
            \curl_setopt($this->handle, CURLOPT_POST, true);
            \curl_setopt($this->handle, CURLOPT_POSTFIELDS, $body);
        }

        $response   = \curl_exec($this->handle);
        $statusCode = \curl_getinfo($this->handle, CURLINFO_HTTP_CODE);

        $this->response = new Response($response, $statusCode);

It seems like $response can also be of type boolean; however, parameter $response of Palmtree\Curl\Response::__construct() does only seem to accept string, maybe add an additional type check?

        return $this->response;
    }

    public function getRequest(): Request
    {
        return $this->request;
    }

    public function getResponse(): Response
    {
        if ($this->response === null) {
            $this->execute();
        }

        return $this->response;
    }

    public function setUrl(string $url): self
    {
        $this->url = $url;

        return $this;
    }

    public function getUrl(): string
    {
        return $this->url;
    }

    public function getOpts(): array
    {
        return $this->curlOpts;
    }

    public function setOpt(int $key, $value): self
    {
        if ($key === CURLOPT_HEADER && !$value) {
            throw new InvalidArgumentException('CURLOPT_HEADER cannot be set to false');
        }

        $this->curlOpts[$key] = $value;

        \curl_setopt($this->handle, $key, $value);

        return $this;
    }

    public function __toString(): string
    {
        return $this->getResponse()->getBody() ?: '';
    }

    private function buildCurlOpts(array $curlOpts)
    {
        $this->curlOpts = \array_replace(self::$defaultCurlOpts, $curlOpts);

        // The Response class always parses headers.
        $this->curlOpts[CURLOPT_HEADER] = true;

        \curl_setopt_array($this->handle, $this->curlOpts);
    }
}