Hab\MeModule\Fetch

Complexity

Total Complexity

13

Size/Duplication

Total Lines	69
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	applyMethod()	0	20	5
A	__construct()	0	6	1
A	setMethod()	0	3	1
A	applyParams()	0	9	4
A	fetch()	0	7	1
A	getMethod()	0	3	1

<?php

namespace Hab\MeModule;

use Anax\Commons\ContainerInjectableInterface;
use Anax\Commons\ContainerInjectableTrait;

class Fetch implements ContainerInjectableInterface
{
    use ContainerInjectableTrait;

    private $curl;
    private $url;
    private $method;
    private $data;

    public function __construct(String $method, String $url, Array $data = [])
    {
        $this->curl = curl_init();
        $this->method = $method;
        $this->url = $url;
        $this->data = $data;
    }

    public function setMethod(String $method = "GET")
    {
        $this->method = $method;
    }

    public function getMethod() : String
    {
        return $this->method;
    }

    public function applyMethod(String $method)
    {
        switch ($method) {
            case "GET":
                $this->setMethod("GET");
                return "GET";
            case "POST":
                curl_setopt($this->curl, CURLOPT_POST, true);

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

                $this->setMethod("POST");
                return "POST";
            case "PUT":
                curl_setopt($this->curl, CURLOPT_CUSTOMREQUEST, "PUT");
                $this->setMethod("PUT");
                return "PUT";
            case "DELETE":
                curl_setopt($this->curl, CURLOPT_CUSTOMREQUEST, "DELETE");
                $this->setMethod("DELETE");
                return "DELETE";
            default:
                return false;
        }
    }

    public function applyParams(Array $data = [])
    {
        // $post = $this->di->get("request")->getPost();
        switch ($this->getMethod()) {
            case "POST":
            case "PUT":
            case "DELETE":
                curl_setopt($this->curl, CURLOPT_POSTFIELDS, $data);

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

                break;
        }
    }

    public function fetch()
    {
        curl_setopt($this->curl, CURLOPT_URL, $this->url);

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

        curl_setopt($this->curl, CURLOPT_RETURNTRANSFER, 1);
        $res = curl_exec($this->curl);

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

        curl_close($this->curl);

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

        return $res;
    }
}