BPT\api\request\curl

Complexity

Total Complexity

16

Size/Duplication

Total Lines	70
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	init()	0	17	3
A	parseResult()	0	1	1
A	getInfo()	0	23	4
A	setTimeout()	0	10	3
A	setData()	0	4	5

<?php


namespace BPT\api\request;


use BPT\constants\loggerTypes;

use BPT\logger;

use BPT\settings;

use CurlHandle;


class curl {

    private static CurlHandle $curl_handler;


    public static function init(string $method,array $data) {

        $info = self::getInfo($data);

        $data = $info['data'];

        $token = $info['token'];

        $handler = $info['handler'];

        self::setTimeout($data,$handler,$method);

It seems like $handler can also be of type resource; however, parameter $curl_handler of BPT\api\request\curl::setTimeout() does only seem to accept CurlHandle, maybe add an additional type check?

        self::setData($data);

        $data['method'] = $method;

        curl_setopt($handler, CURLOPT_POSTFIELDS, $data);

        $result = curl_exec($handler);

        if (curl_errno($handler)) {

            logger::write(curl_error($handler),loggerTypes::WARNING);

        }

        if ($token != settings::$token) {

            curl_close($handler);

        }

        return json_decode($result);

It seems like $result can also be of type true; however, parameter $json of json_decode() does only seem to accept string, maybe add an additional type check?

    }


    private static function getInfo(array $data): array {

        if (isset($data['token']) && $data['token'] !== settings::$token) {

            $token = $data['token'];

            unset($data['token']);

            $curl_handler = curl_init(settings::$base_url."$token/");

            curl_setopt($curl_handler, CURLOPT_RETURNTRANSFER, true);

            curl_setopt($curl_handler, CURLOPT_SSL_VERIFYPEER, false);

        }

        else{

            $token = settings::$token;

            if (!isset(self::$curl_handler)){

                self::$curl_handler = curl_init(settings::$base_url."$token/");

                curl_setopt(self::$curl_handler, CURLOPT_RETURNTRANSFER, true);

                curl_setopt(self::$curl_handler, CURLOPT_SSL_VERIFYPEER, false);

                curl_setopt(self::$curl_handler, CURLOPT_TCP_KEEPALIVE, 1);

            }

            $curl_handler = self::$curl_handler;

        }


        return [

            'data'    => $data,

            'token'   => $token,

            'handler' => $curl_handler

        ];

    }


    private static function setTimeout(array &$data , CurlHandle $curl_handler,string $method) {

        if (isset($data['forgot'])) {

            curl_setopt($curl_handler, CURLOPT_TIMEOUT_MS, settings::$forgot_time);

            unset($data['forgot']);

        }

        elseif ($method === 'getUpdates'){

            curl_setopt($curl_handler, CURLOPT_TIMEOUT_MS, 5000);

        }

        else{

            curl_setopt($curl_handler, CURLOPT_TIMEOUT_MS, 300);

        }

    }


    private static function setData(array &$data) {

        foreach ($data as &$value){

            if (is_array($value) || (is_object($value) && !is_a($value,'CURLFile'))){

                $value = json_encode($value);

            }

        }

    }


    private static function parseResult($result) {

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

The method parseResult() is not used, and could be removed.

    }

}