Hab\MeModule\Fetch2

Complexity

Total Complexity

9

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	1
A	fetch()	0	30	5
A	setUrl()	0	3	1
A	setMethod()	0	3	1
A	getMethod()	0	3	1

<?php

namespace Hab\MeModule;

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

class Fetch2 implements ContainerInjectableInterface
{
    use ContainerInjectableTrait;

    private $curl;

The private property $curl is not used, and could be removed.

    private $url;
    private $method;
    private $data;
    private $sweChars;

    public function __construct(String $method = "GET", String $url = "", Array $data = [])
    {
        $this->method = $method;
        $this->url = $url;
        $this->data = $data;
        $this->sweChars = [
            "å" => "a",
            "ä" => "a",
            "ö" => "o"
        ];
    }

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

    public function setUrl(String $url)
    {
        $this->url = $url;
    }

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

    public function fetch(String $method, String $url, Array $params = [], Array $data = [])

The parameter $params is not used and could be removed.

    {
        $curl = curl_init();
        $url = str_replace(array_keys($this->sweChars), $this->sweChars, $url);;
        switch ($method) {
            case "GET":
                break;
            case "POST":
                curl_setopt($curl, CURLOPT_POSTFIELDS, $data);

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

                curl_setopt($curl, CURLOPT_POST, true);
                break;    
            case "PUT":
                curl_setopt($curl, CURLOPT_POSTFIELDS, $data);
                curl_setopt($curl, CURLOPT_CUSTOMREQUEST, "PUT");
                break;    
            case "DELETE":
                curl_setopt($curl, CURLOPT_POSTFIELDS, $data);
                curl_setopt($curl, CURLOPT_CUSTOMREQUEST, "DELETE");
                break;    
            default:
                curl_close($curl);

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

                return false;
                // break;
            }
        curl_setopt($curl, CURLOPT_URL, $url);
        curl_setopt($curl, CURLOPT_RETURNTRANSFER, 1);
        $res = curl_exec($curl);

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

        curl_close($curl);

        return $res;
    }

    // public function multiFetch($urls = [])
    // {
    //     $mc = curl_multi_init();
    //     $curls = [];
    //     foreach ($urls as $url) {
    //         // var_dump($urls);
    //         // die();
    //         $curl = $this->prep($url[0], $url[1], $url[2]);
    //         curl_multi_add_handle($mc, $curl);
    //         array_push($curls, $curl);
    //     }
    //     do {
    //         $status = curl_multi_exec($mc, $active);
    //         if ($active) {
    //             curl_multi_select($mc);
    //         }
    //     } while ($active && $status == CURLM_OK);
    //     foreach ($curls as $curl) {
    //         curl_multi_remove_handle($mc, $curl);
    //     }
    //     curl_multi_close($mc);

    //     return $mc;
    // }
}