Kobas\APIClient\Request\Curl

Complexity

Total Complexity

11

Size/Duplication

Total Lines	98
Duplicated Lines	0 %

Test Coverage

Coverage

69.23%

Importance

Changes	3
Bugs	0	Features	0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	setUrl()	0	3	1
A	setOption()	0	4	1
A	getInfo()	0	3	1
A	init()	0	7	2
A	getHandle()	0	3	1
A	execute()	0	3	1
A	getErrorNumber()	0	3	1
A	getErrorMessage()	0	3	1
A	close()	0	6	1
A	getAllInfo()	0	3	1

<?php

namespace Kobas\APIClient\Request;

/**
 * Class Curl
 *
 * @package Kobas\APIClient\Request
 */
class Curl implements HttpRequest
{
    /**
     * @var null|false|resource
     */
    private $handle = null;

    /**
     * @return $this
     */
    public function init()
    {
        if (is_null($this->handle)) {
            $this->handle = curl_init();
        }

        return $this;
    }

    /**
     * @param $url
     * @return Curl
     */
    public function setUrl($url)
    {
        return $this->setOption(CURLOPT_URL, $url);
    }

    /**
     * @param $name
     * @param $value
     * @return Curl
     */
    public function setOption($name, $value)
    {
        curl_setopt($this->handle, $name, $value);

It seems like $this->handle can also be of type boolean; however, parameter $ch of curl_setopt() does only seem to accept resource, maybe add an additional type check?

        return $this;
    }


    /**
     * @return false| resource
     */
    public function getHandle()
    {
        return $this->handle;

The expression return $this->handle also could return the type boolean which is incompatible with the documented return type false|resource.

    }

    /**
     * @return bool|string
     */
    public function execute()
    {
        return curl_exec($this->handle);

It seems like $this->handle can also be of type boolean; however, parameter $ch of curl_exec() does only seem to accept resource, maybe add an additional type check?

    }

    /**
     * @param $name
     * @return mixed
     */
    public function getInfo($name)
    {
        return curl_getinfo($this->handle, $name);

It seems like $this->handle can also be of type boolean; however, parameter $ch of curl_getinfo() does only seem to accept resource, maybe add an additional type check?

    }

    /**
     * @return array|false
     */
    public function getAllInfo()
    {
        return curl_getinfo($this->handle);

It seems like $this->handle can also be of type boolean; however, parameter $ch of curl_getinfo() does only seem to accept resource, maybe add an additional type check?

    }

    /**
     * @return int|mixed
     */
    public function getErrorNumber()
    {
        return curl_errno($this->handle);

It seems like $this->handle can also be of type boolean; however, parameter $ch of curl_errno() does only seem to accept resource, maybe add an additional type check?

    }

    /**
     * @return mixed|string
     */
    public function getErrorMessage()
    {
        return curl_error($this->handle);

It seems like $this->handle can also be of type boolean; however, parameter $ch of curl_error() does only seem to accept resource, maybe add an additional type check?

    }

    /**
     * @return Curl
     */
    public function close()
    {
        curl_close($this->handle);

It seems like $this->handle can also be of type boolean; however, parameter $ch of curl_close() does only seem to accept resource, maybe add an additional type check?

        $this->handle = null;

        return $this;
    }
}