ClickHouseDB\Transport\CurlerRequest

Complexity

Total Complexity

90

Size/Duplication

Total Lines	759
Duplicated Lines	0 %

Test Coverage

Coverage

72.82%

Importance

Changes	2
Bugs	1	Features	0

52 Methods

Rating	Name	Duplication	Size	Complexity
A	keepAlive()	0	4	1
A	httpCompression()	0	8	2
A	getUniqHash()	0	3	1
A	isResultFile()	0	3	2
A	parameters_json()	0	23	5
A	setReadFunction()	0	3	1
A	isPersistent()	0	3	1
A	dump()	0	15	2
F	prepareRequest()	0	76	18
A	setInfile()	0	21	3
A	GET()	0	3	1
A	setStdErrOut()	0	4	2
A	url()	0	4	1
A	id()	0	7	2
A	getId()	0	3	1
A	getResultFileHandle()	0	3	1
A	setResponse()	0	3	1
A	execute()	0	4	1
A	OPTIONS()	0	3	1
A	timeOut()	0	3	1
A	timeOutMs()	0	4	1
A	persistent()	0	4	1
A	__construct()	0	23	1
A	authByBasicAuth()	0	4	1
A	setFunctionProgress()	0	8	2
A	setWriteFunction()	0	3	1
A	getUrl()	0	3	1
A	isResponseExists()	0	3	1
A	getInfileHandle()	0	3	1
A	getDetails()	0	7	1
A	header()	0	4	1
A	onCallback()	0	10	4
A	getHeaders()	0	7	2
A	parameters()	0	4	1
A	connectTimeOut()	0	4	1
A	attachFiles()	0	11	2
A	setRequestExtendedInfo()	0	4	1
A	option()	0	4	1
A	setSslCa()	0	4	1
A	handle()	0	4	1
A	POST()	0	3	1
A	getRequestExtendedInfo()	0	7	3
A	setCallback()	0	4	1
A	response()	0	7	2
A	verbose()	0	4	1
A	authByHeaders()	0	5	1
A	getDnsCache()	0	3	1
A	setDnsCache()	0	4	1
A	setCallbackFunction()	0	3	1
A	setHeaderFunction()	0	3	1
A	PUT()	0	3	1
A	setResultFileHandle()	0	8	2

<?php

Missing declare(strict_types=1).

namespace ClickHouseDB\Transport;

use const CURLOPT_HTTPGET;
use const CURLOPT_POST;

class CurlerRequest
{
    /**

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$extendinfo with single line content, use one-line comment instead.

     * @var array

@var annotation of property \ClickHouseDB\Transport\CurlerRequest::$extendinfo does not specify type hint for its items.

     */
    public $extendinfo = [];

    /**

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$parameters with single line content, use one-line comment instead.

     * @var string|array

@var annotation of property \ClickHouseDB\Transport\CurlerRequest::$parameters does not specify type hint for its items.

     */
    private $parameters = '';

    /**

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$options with single line content, use one-line comment instead.

     * @var array

@var annotation of property \ClickHouseDB\Transport\CurlerRequest::$options does not specify type hint for its items.

     */
    private $options;

    /**

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$headers with single line content, use one-line comment instead.

     * @var array

@var annotation of property \ClickHouseDB\Transport\CurlerRequest::$headers does not specify type hint for its items.

     */
    private $headers; // Parsed reponse header object.

    /**

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$url with single line content, use one-line comment instead.

     * @var string
     */
    private $url;

    /**

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$method with single line content, use one-line comment instead.

     * @var string
     */
    private $method;

    /**

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$id with single line content, use one-line comment instead.

     * @var bool
     */
    private $id;

    /**

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$handle with single line content, use one-line comment instead.

     * @var resource|null
     */
    private $handle;

    /** @var CurlerResponse */
    private $response;

    /** @var bool */
    private $_persistent = false;

    /**

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$_attachFiles with single line content, use one-line comment instead.

     * @var bool
     */
    private $_attachFiles = false;

    /**

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$callback_class with single line content, use one-line comment instead.

     * @var string
     */
    private $callback_class = '';

Member variable "callback_class" is not in valid camel caps format

    /**

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$callback_functionName with single line content, use one-line comment instead.

     * @var string
     */
    private $callback_functionName = '';

Member variable "callback_functionName" is not in valid camel caps format

    /**

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$_httpCompression with single line content, use one-line comment instead.

     * @var bool
     */
    private $_httpCompression = false;

    /**

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$callback_function with single line content, use one-line comment instead.

     * @var callable
     */
    private $callback_function = null;

Member variable "callback_function" is not in valid camel caps format

    /**

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$infile_handle with single line content, use one-line comment instead.

     * @var bool|resource
     */
    private $infile_handle = false;

Member variable "infile_handle" is not in valid camel caps format

    /**

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$_dns_cache with single line content, use one-line comment instead.

     * @var int
     */
    private $_dns_cache = 120;

Member variable "_dns_cache" is not in valid camel caps format

    /**

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$resultFileHandle with single line content, use one-line comment instead.

     * @var resource
     */
    private $resultFileHandle = null;

    /**

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$sslCa with single line content, use one-line comment instead.

     * @var string
     */
    private $sslCa = null;

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

    /**

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$stdErrOut with single line content, use one-line comment instead.

     * @var null|resource

Null type hint should be on last position in "null|resource".

     */
    private $stdErrOut = null;
    /**
     * @param bool $id
     */
    public function __construct($id = false, $handle = null)

Method \ClickHouseDB\Transport\CurlerRequest::__construct() does not have native type hint for its parameter $id but it should be possible to add it based on @param annotation "bool".

Expected 1 blank line before function; 0 found

Method \ClickHouseDB\Transport\CurlerRequest::__construct() does not have parameter type hint nor @param annotation for its parameter $handle.

    {
        $this->id = $id;

        $this->header('Cache-Control', 'no-cache, no-store, must-revalidate');
        $this->header('Expires', '0');
        $this->header('Pragma', 'no-cache');

        $this->options = array(

Short array syntax must be used to define arrays

            CURLOPT_SSL_VERIFYHOST => 0,

Constant CURLOPT_SSL_VERIFYHOST should not be referenced via a fallback global name, but via a use statement.

            CURLOPT_SSL_VERIFYPEER => false,

Constant CURLOPT_SSL_VERIFYPEER should not be referenced via a fallback global name, but via a use statement.

            CURLOPT_TIMEOUT => 10,

Constant CURLOPT_TIMEOUT should not be referenced via a fallback global name, but via a use statement.

            CURLOPT_CONNECTTIMEOUT => 5, // Количество секунд ожидания при попытке соединения

Constant CURLOPT_CONNECTTIMEOUT should not be referenced via a fallback global name, but via a use statement.

            CURLOPT_HTTP_VERSION => CURL_HTTP_VERSION_1_1,

Constant CURLOPT_HTTP_VERSION should not be referenced via a fallback global name, but via a use statement.

Constant CURL_HTTP_VERSION_1_1 should not be referenced via a fallback global name, but via a use statement.

            CURLOPT_MAXREDIRS => 10,

Constant CURLOPT_MAXREDIRS should not be referenced via a fallback global name, but via a use statement.

            CURLOPT_HEADER => TRUE,

Constant CURLOPT_HEADER should not be referenced via a fallback global name, but via a use statement.

TRUE, FALSE and NULL must be lowercase; expected true, but found TRUE.

            CURLOPT_FOLLOWLOCATION => TRUE,

Constant CURLOPT_FOLLOWLOCATION should not be referenced via a fallback global name, but via a use statement.

TRUE, FALSE and NULL must be lowercase; expected true, but found TRUE.

            CURLOPT_AUTOREFERER => 1, // при редиректе подставлять в «Referer:» значение из «Location:»

Constant CURLOPT_AUTOREFERER should not be referenced via a fallback global name, but via a use statement.

            CURLOPT_BINARYTRANSFER => 1, // передавать в binary-safe

The constant CURLOPT_BINARYTRANSFER has been deprecated: 5.1.3

Constant CURLOPT_BINARYTRANSFER should not be referenced via a fallback global name, but via a use statement.

            CURLOPT_RETURNTRANSFER => TRUE,

Constant CURLOPT_RETURNTRANSFER should not be referenced via a fallback global name, but via a use statement.

TRUE, FALSE and NULL must be lowercase; expected true, but found TRUE.

            CURLOPT_USERAGENT => 'smi2/PHPClickHouse/client',

Constant CURLOPT_USERAGENT should not be referenced via a fallback global name, but via a use statement.

        );
        $this->handle = $handle;

Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space

    }

    /**
     * @param array $attachFiles

@param annotation of method \ClickHouseDB\Transport\CurlerRequest::attachFiles() does not specify type hint for items of its traversable parameter $attachFiles.

     */
    public function attachFiles($attachFiles)

Method \ClickHouseDB\Transport\CurlerRequest::attachFiles() does not have native type hint for its parameter $attachFiles but it should be possible to add it based on @param annotation "array".

Method \ClickHouseDB\Transport\CurlerRequest::attachFiles() does not have void return type hint.

    {
        $this->header("Content-Type", "multipart/form-data");

The string literal Content-Type does not require double quotes, as per coding-style, please use single quotes.

The string literal multipart/form-data does not require double quotes, as per coding-style, please use single quotes.

        $out = [];
        foreach ($attachFiles as $post_name => $file_path) {

The variable $post_name should be in camel caps format.

The variable $file_path should be in camel caps format.

            $out[$post_name] = new \CURLFile($file_path);

The variable $post_name should be in camel caps format.

Class \CURLFile should not be referenced via a fully qualified name, but via a use statement.

The variable $file_path should be in camel caps format.

        }

        $this->_attachFiles = true;
        $this->parameters($out);
    }

Expected 1 blank line after function; 2 found

    /**
     * @param bool $set

Incorrect annotations group.

     * @return $this
     */
    public function id($set = false)

Method \ClickHouseDB\Transport\CurlerRequest::id() does not have native type hint for its parameter $set but it should be possible to add it based on @param annotation "bool".

    {
        if ($set) {
            $this->id = $set;
        }

        return $this;
    }

    /**
     * @param array $params

Incorrect annotations group.

@param annotation of method \ClickHouseDB\Transport\CurlerRequest::setRequestExtendedInfo() does not specify type hint for items of its traversable parameter $params.

     * @return $this
     */
    public function setRequestExtendedInfo($params)

Method \ClickHouseDB\Transport\CurlerRequest::setRequestExtendedInfo() does not have native type hint for its parameter $params but it should be possible to add it based on @param annotation "array".

    {
        $this->extendinfo = $params;
        return $this;

Expected 1 lines before "return", found 0.

    }

    /**
     * @param string|integer|null $key

Incorrect annotations group.

Expected "int" but found "integer" in @param annotation.

     * @return mixed
     */
    public function getRequestExtendedInfo($key = null)
    {
        if ($key) {
            return isset($this->extendinfo[$key]) ? $this->extendinfo[$key] : false;

Use null coalesce operator instead of ternary operator.

        }

        return $this->extendinfo;
    }

    /**
     * @return bool|resource
     */
    public function getInfileHandle()
    {
        return $this->infile_handle;

Member variable "infile_handle" is not in valid camel caps format

    }

    /**
     * @param string $file_name

Incorrect annotations group.

     * @return bool|resource
     */
    public function setInfile($file_name)

Method \ClickHouseDB\Transport\CurlerRequest::setInfile() does not have native type hint for its parameter $file_name but it should be possible to add it based on @param annotation "string".

The variable $file_name should be in camel caps format.

    {
        $this->header('Expect', '');
        $this->infile_handle = fopen($file_name, 'r');

Member variable "infile_handle" is not in valid camel caps format

Function fopen() should not be referenced via a fallback global name, but via a use statement.

The variable $file_name should be in camel caps format.

        if (is_resource($this->infile_handle)) {

Function is_resource() should not be referenced via a fallback global name, but via a use statement.

Member variable "infile_handle" is not in valid camel caps format

Blank line found at start of control structure

            if ($this->_httpCompression) {
                $this->header('Content-Encoding', 'gzip');
                $this->header('Content-Type', 'application/x-www-form-urlencoded');

                stream_filter_append($this->infile_handle, 'zlib.deflate', STREAM_FILTER_READ, ["window" => 30]);

Function stream_filter_append() should not be referenced via a fallback global name, but via a use statement.

Member variable "infile_handle" is not in valid camel caps format

Constant STREAM_FILTER_READ should not be referenced via a fallback global name, but via a use statement.

The string literal window does not require double quotes, as per coding-style, please use single quotes.

                $this->options[CURLOPT_SAFE_UPLOAD] = 1;

The constant CURLOPT_SAFE_UPLOAD has been deprecated: 7.0 Use <b>CURLFile</b> for uploads instead.

Constant CURLOPT_SAFE_UPLOAD should not be referenced via a fallback global name, but via a use statement.

            } else {
                $this->options[CURLOPT_INFILESIZE] = filesize($file_name);

Constant CURLOPT_INFILESIZE should not be referenced via a fallback global name, but via a use statement.

Function filesize() should not be referenced via a fallback global name, but via a use statement.

The variable $file_name should be in camel caps format.

            }

            $this->options[CURLOPT_INFILE] = $this->infile_handle;

Constant CURLOPT_INFILE should not be referenced via a fallback global name, but via a use statement.

Member variable "infile_handle" is not in valid camel caps format

        }

        return $this->infile_handle;

Member variable "infile_handle" is not in valid camel caps format

    }

    /**
     * @param callable $callback
     */
    public function setCallbackFunction($callback)

Method \ClickHouseDB\Transport\CurlerRequest::setCallbackFunction() does not have native type hint for its parameter $callback but it should be possible to add it based on @param annotation "callable".

Method \ClickHouseDB\Transport\CurlerRequest::setCallbackFunction() does not have void return type hint.

    {
        $this->callback_function = $callback;

Member variable "callback_function" is not in valid camel caps format

    }

    /**
     * @param callable $callback
     */
    public function setWriteFunction($callback)

Method \ClickHouseDB\Transport\CurlerRequest::setWriteFunction() does not have native type hint for its parameter $callback but it should be possible to add it based on @param annotation "callable".

Method \ClickHouseDB\Transport\CurlerRequest::setWriteFunction() does not have void return type hint.

    {
        $this->options[CURLOPT_WRITEFUNCTION] = $callback;

Constant CURLOPT_WRITEFUNCTION should not be referenced via a fallback global name, but via a use statement.

    }

    /**
     * @param callable $callback
     */
    public function setReadFunction($callback)

Method \ClickHouseDB\Transport\CurlerRequest::setReadFunction() does not have native type hint for its parameter $callback but it should be possible to add it based on @param annotation "callable".

Method \ClickHouseDB\Transport\CurlerRequest::setReadFunction() does not have void return type hint.

    {
        $this->options[CURLOPT_READFUNCTION] = $callback;

Constant CURLOPT_READFUNCTION should not be referenced via a fallback global name, but via a use statement.

    }

    public function setHeaderFunction($callback)

Method \ClickHouseDB\Transport\CurlerRequest::setHeaderFunction() does not have parameter type hint nor @param annotation for its parameter $callback.

Method \ClickHouseDB\Transport\CurlerRequest::setHeaderFunction() does not have void return type hint.

    {
        $this->options[CURLOPT_HEADERFUNCTION] = $callback;

Constant CURLOPT_HEADERFUNCTION should not be referenced via a fallback global name, but via a use statement.

    }

    /**
     * @param string $classCallBack
     * @param string $functionName
     */
    public function setCallback($classCallBack, $functionName)

Method \ClickHouseDB\Transport\CurlerRequest::setCallback() does not have native type hint for its parameter $classCallBack but it should be possible to add it based on @param annotation "string".

Method \ClickHouseDB\Transport\CurlerRequest::setCallback() does not have native type hint for its parameter $functionName but it should be possible to add it based on @param annotation "string".

Method \ClickHouseDB\Transport\CurlerRequest::setCallback() does not have void return type hint.

    {
        $this->callback_class = $classCallBack;

Member variable "callback_class" is not in valid camel caps format

Equals sign not aligned with surrounding assignments; expected 8 spaces but found 1 space

        $this->callback_functionName = $functionName;

Member variable "callback_functionName" is not in valid camel caps format

    }

    /**

Empty comment

     *
     */
    public function onCallback()

Method \ClickHouseDB\Transport\CurlerRequest::onCallback() does not have void return type hint.

Method \ClickHouseDB\Transport\CurlerRequest::onCallback() does not need documentation comment.

    {
        if ($this->callback_function) {

Member variable "callback_function" is not in valid camel caps format

            $x = $this->callback_function;

Member variable "callback_function" is not in valid camel caps format

            $x($this);
        }

        if ($this->callback_class && $this->callback_functionName) {

Use early exit to reduce code nesting.

Member variable "callback_class" is not in valid camel caps format

Member variable "callback_functionName" is not in valid camel caps format

            $c = $this->callback_functionName;

Member variable "callback_functionName" is not in valid camel caps format

            $this->callback_class->$c($this);

Member variable "callback_class" is not in valid camel caps format

        }
    }

    public function getDetails(): array

Method \ClickHouseDB\Transport\CurlerRequest::getDetails() does not have @return annotation for its traversable return value.

    {
        return [
            'url'        => $this->url,
            'method'     => $this->method,
            'parameters' => $this->parameters,
            'headers'    => $this->headers,
        ];
    }

    /**
     * @param resource $stream

Incorrect annotations group.

     * @return void
     */
    public function setStdErrOut($stream)

Method \ClickHouseDB\Transport\CurlerRequest::setStdErrOut() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "void".

    {
        if (is_resource($stream)) {

Expected 0 lines after "if", found 1.

Use early exit to reduce code nesting.

Function is_resource() should not be referenced via a fallback global name, but via a use statement.

            $this->stdErrOut=$stream;

Expected at least 1 space before "="; 0 found

Expected at least 1 space after "="; 0 found

Equals sign not aligned correctly; expected 1 space but found 0 spaces

        }

    }

Function closing brace must go on the next line following the body; found 1 blank lines before brace

    /**
     * @param bool $result

Incorrect annotations group.

     * @return string
     */
    public function dump($result = false)

Method \ClickHouseDB\Transport\CurlerRequest::dump() does not have native type hint for its parameter $result but it should be possible to add it based on @param annotation "bool".

Method \ClickHouseDB\Transport\CurlerRequest::dump() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "string".

    {
        $message = "\n------------  Request ------------\n";

Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space

        $message .= 'URL:' . $this->url . "\n\n";
        $message .= 'METHOD:' . $this->method . "\n\n";
        $message .= 'PARAMS:' . print_r($this->parameters, true) . "\n";

Are you sure print_r($this->parameters, true) of type string|true can be used in concatenation?

Function print_r() should not be referenced via a fallback global name, but via a use statement.

        $message .= 'PARAMS:' . print_r($this->headers, true) . "\n";

Are you sure print_r($this->headers, true) of type string|true can be used in concatenation?

Function print_r() should not be referenced via a fallback global name, but via a use statement.

        $message .= "-----------------------------------\n";

        if ($result) {
            return $message;
        }

        echo $message;
        return '';

Expected 1 lines before "return", found 0.

    }

    /**
     * @return bool
     */
    public function getId()

Method \ClickHouseDB\Transport\CurlerRequest::getId() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "bool".

    {
        return $this->id;
    }

    /**
     * @param integer $key

Incorrect annotations group.

Expected "int" but found "integer" in @param annotation.

     * @param mixed $value

Expected 3 spaces after parameter type; 1 found

     * @return $this
     */
    private function option($key, $value)

Method \ClickHouseDB\Transport\CurlerRequest::option() does not have native type hint for its parameter $key but it should be possible to add it based on @param annotation "integer".

    {
        $this->options[$key] = $value;
        return $this;

Expected 1 lines before "return", found 0.

    }

    /**
     * @return $this
     */
    public function persistent()
    {
        $this->_persistent = true;
        return $this;

Expected 1 lines before "return", found 0.

    }

    /**
     * @return bool
     */
    public function isPersistent()

Method \ClickHouseDB\Transport\CurlerRequest::isPersistent() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "bool".

    {
        return $this->_persistent;
    }

    /**
     * @param int $sec

Incorrect annotations group.

Method \ClickHouseDB\Transport\CurlerRequest::keepAlive() has useless @param annotation for parameter $sec.

     * @return $this
     */
    public function keepAlive(int $sec = 60)
    {
        $this->headers['Connection'] = 'keep-alive';
        return $this;

Expected 1 lines before "return", found 0.

    }

    /**
     * @param bool $flag

Incorrect annotations group.

Method \ClickHouseDB\Transport\CurlerRequest::verbose() has useless @param annotation for parameter $flag.

     * @return $this
     */
    public function verbose(bool $flag = true)
    {
        $this->options[CURLOPT_VERBOSE] = $flag;

Constant CURLOPT_VERBOSE should not be referenced via a fallback global name, but via a use statement.

        return $this;

Expected 1 lines before "return", found 0.

    }

    /**
     * @param string $key

Incorrect annotations group.

Method \ClickHouseDB\Transport\CurlerRequest::header() has useless @param annotation for parameter $key.

     * @param string $value

Method \ClickHouseDB\Transport\CurlerRequest::header() has useless @param annotation for parameter $value.

     * @return $this
     */
    public function header(string $key, string $value)
    {
        $this->headers[$key] = $value;
        return $this;

Expected 1 lines before "return", found 0.

    }

    /**
     * @return array

@return annotation of method \ClickHouseDB\Transport\CurlerRequest::getHeaders() does not specify type hint for items of its traversable return value.

     */
    public function getHeaders():array

There must be exactly one space between return type hint colon and return type hint.

    {
        $head = [];
        foreach ($this->headers as $key => $value) {

Expected 1 lines after "foreach", found 0.

            $head[] = sprintf("%s: %s", $key, $value);

Function sprintf() should not be referenced via a fallback global name, but via a use statement.

The string literal %s: %s does not require double quotes, as per coding-style, please use single quotes.

        }
        return $head;

Expected 1 lines before "return", found 0.

    }

    /**
     * @param string $url

Incorrect annotations group.

Method \ClickHouseDB\Transport\CurlerRequest::url() has useless @param annotation for parameter $url.

     * @return $this
     */
    public function url(string $url)
    {
        $this->url = $url;
        return $this;

Expected 1 lines before "return", found 0.

    }

    /**
     * @return string

Method \ClickHouseDB\Transport\CurlerRequest::getUrl() has useless @return annotation.

     */
    public function getUrl():string

Method \ClickHouseDB\Transport\CurlerRequest::getUrl() does not need documentation comment.

There must be exactly one space between return type hint colon and return type hint.

    {
        return $this->url;
    }

Expected 1 blank line after function; 2 found

    /**
     * @param string $id

Incorrect annotations group.

Method \ClickHouseDB\Transport\CurlerRequest::getUniqHash() has useless @param annotation for parameter $id.

     * @return string

Method \ClickHouseDB\Transport\CurlerRequest::getUniqHash() has useless @return annotation.

     */
    public function getUniqHash(string $id):string

Method \ClickHouseDB\Transport\CurlerRequest::getUniqHash() does not need documentation comment.

There must be exactly one space between return type hint colon and return type hint.

    {
        return $id . '.' . microtime() . mt_rand(0, 1000000);

Function microtime() should not be referenced via a fallback global name, but via a use statement.

Function mt_rand() should not be referenced via a fallback global name, but via a use statement.

    }

    /**
     * @param bool $flag

Method \ClickHouseDB\Transport\CurlerRequest::httpCompression() has useless @param annotation for parameter $flag.

     */
    public function httpCompression(bool $flag):void

Method \ClickHouseDB\Transport\CurlerRequest::httpCompression() does not need documentation comment.

There must be exactly one space between return type hint colon and return type hint.

    {
        if ($flag) {
            $this->_httpCompression = $flag;

Equals sign not aligned with surrounding assignments; expected 10 spaces but found 1 space

            $this->options[CURLOPT_ENCODING] = 'gzip';

Constant CURLOPT_ENCODING should not be referenced via a fallback global name, but via a use statement.

        } else {
            $this->_httpCompression = false;
            unset($this->options[CURLOPT_ENCODING]);

Constant CURLOPT_ENCODING should not be referenced via a fallback global name, but via a use statement.

        }
    }

    /**
     * @param string $username

Incorrect annotations group.

     * @param string $password
     * @return $this
     */
    public function authByBasicAuth($username, $password)

Method \ClickHouseDB\Transport\CurlerRequest::authByBasicAuth() does not have native type hint for its parameter $username but it should be possible to add it based on @param annotation "string".

Method \ClickHouseDB\Transport\CurlerRequest::authByBasicAuth() does not have native type hint for its parameter $password but it should be possible to add it based on @param annotation "string".

    {
        $this->options[CURLOPT_USERPWD] = sprintf("%s:%s", $username, $password);

Constant CURLOPT_USERPWD should not be referenced via a fallback global name, but via a use statement.

Function sprintf() should not be referenced via a fallback global name, but via a use statement.

The string literal %s:%s does not require double quotes, as per coding-style, please use single quotes.

        return $this;

Expected 1 lines before "return", found 0.

    }

    public function authByHeaders($username, $password)

Method \ClickHouseDB\Transport\CurlerRequest::authByHeaders() does not have parameter type hint nor @param annotation for its parameter $username.

Method \ClickHouseDB\Transport\CurlerRequest::authByHeaders() does not have parameter type hint nor @param annotation for its parameter $password.

Method \ClickHouseDB\Transport\CurlerRequest::authByHeaders() does not have return type hint nor @return annotation for its return value.

    {
        $this->headers['X-ClickHouse-User'] = $username;
        $this->headers['X-ClickHouse-Key'] = $password;

Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space

        return $this;

Expected 1 lines before "return", found 0.

    }

    /**
     * @param array|string $data

Incorrect annotations group.

@param annotation of method \ClickHouseDB\Transport\CurlerRequest::parameters() does not specify type hint for items of its traversable parameter $data.

     * @return $this
     */
    public function parameters($data)
    {
        $this->parameters = $data;
        return $this;

Expected 1 lines before "return", found 0.

    }

    /**
     * The number of seconds to wait when trying to connect. Use 0 for infinite waiting.
     *
     * @param float $seconds

Incorrect annotations group.

Method \ClickHouseDB\Transport\CurlerRequest::connectTimeOut() has useless @param annotation for parameter $seconds.

     * @return $this
     */
    public function connectTimeOut(float $seconds = 1.0)
    {
        $this->options[CURLOPT_CONNECTTIMEOUT_MS] = (int) ($seconds*1000.0);

Constant CURLOPT_CONNECTTIMEOUT_MS should not be referenced via a fallback global name, but via a use statement.

Expected at least 1 space before "*"; 0 found

Expected at least 1 space after "*"; 0 found

        return $this;

Expected 1 lines before "return", found 0.

    }

    /**
     * The maximum number of seconds (float) allowed to execute cURL functions.
     *
     * @param float $seconds

Incorrect annotations group.

Method \ClickHouseDB\Transport\CurlerRequest::timeOut() has useless @param annotation for parameter $seconds.

     * @return $this
     */
    public function timeOut(float $seconds = 10)
    {
        return $this->timeOutMs((int) ($seconds * 1000.0));
    }

    /**
     * The maximum allowed number of milliseconds to perform cURL functions.
     *
     * @param int $ms millisecond

Incorrect annotations group.

     * @return $this
     */
    protected function timeOutMs(int $ms = 10000)
    {
        $this->options[CURLOPT_TIMEOUT_MS] = $ms;

Constant CURLOPT_TIMEOUT_MS should not be referenced via a fallback global name, but via a use statement.

        return $this;

Expected 1 lines before "return", found 0.

    }

Expected 1 blank line after function; 2 found

    /**
     * @param array|mixed $data

Incorrect annotations group.

@param annotation of method \ClickHouseDB\Transport\CurlerRequest::parameters_json() does not specify type hint for items of its traversable parameter $data.

     * @return $this
     * @throws \ClickHouseDB\Exception\TransportException

Class \ClickHouseDB\Exception\TransportException should not be referenced via a fully qualified name, but via a use statement.

     */
    public function parameters_json($data)

Method name "CurlerRequest::parameters_json" is not in camel caps format

    {

Expected 0 blank lines after opening function brace; 1 found

        $this->header("Content-Type", "application/json, text/javascript; charset=utf-8");

The string literal Content-Type does not require double quotes, as per coding-style, please use single quotes.

The string literal application/json, text/javascript; charset=utf-8 does not require double quotes, as per coding-style, please use single quotes.

        $this->header("Accept", "application/json, text/javascript, */*; q=0.01");

The string literal Accept does not require double quotes, as per coding-style, please use single quotes.

The string literal application/json, text/javascript, */*; q=0.01 does not require double quotes, as per coding-style, please use single quotes.

        if ($data === null) {
            $this->parameters = '{}';
            return $this;

Expected 1 lines before "return", found 0.

        }

        if (is_string($data)) {

Function is_string() should not be referenced via a fallback global name, but via a use statement.

            $this->parameters = $data;
            return $this;

Expected 1 lines before "return", found 0.

        }

        $this->parameters = json_encode($data);

Function json_encode() should not be referenced via a fallback global name, but via a use statement.

        if (!$this->parameters && $data) {

Expected 1 space after NOT operator; 0 found

            throw new \ClickHouseDB\Exception\TransportException('Cant json_encode: ' . strval($data));

Class \ClickHouseDB\Exception\TransportException should not be referenced via a fully qualified name, but via a use statement.

Function strval() should not be referenced via a fallback global name, but via a use statement.

        }

        return $this;
    }

    /**
     * @return resource
     */
    public function getResultFileHandle()
    {
        return $this->resultFileHandle;
    }

    /**
     * @return bool
     */
    public function isResultFile()

Method \ClickHouseDB\Transport\CurlerRequest::isResultFile() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "bool".

    {
        return ($this->resultFileHandle ? true : false);

Usage of language construct "return" with parentheses is disallowed.

Useless ternary operator.

    }

    /**
     * @param resource $h resource

Incorrect annotations group.

Expected 4 spaces after parameter name; 1 found

     * @param bool $zlib

Expected 5 spaces after parameter type; 1 found

     * @return $this
     */
    public function setResultFileHandle($h, $zlib = false)

Method \ClickHouseDB\Transport\CurlerRequest::setResultFileHandle() does not have native type hint for its parameter $zlib but it should be possible to add it based on @param annotation "bool".

    {
        $this->resultFileHandle = $h;
        if ($zlib) {

Expected 1 lines after "if", found 0.

            $params = array('level' => 6, 'window' => 15, 'memory' => 9);

Short array syntax must be used to define arrays

            stream_filter_append($this->resultFileHandle, 'zlib.deflate', STREAM_FILTER_WRITE, $params);

Function stream_filter_append() should not be referenced via a fallback global name, but via a use statement.

Constant STREAM_FILTER_WRITE should not be referenced via a fallback global name, but via a use statement.

        }
        return $this;

Expected 1 lines before "return", found 0.

    }

    /**
     * @return CurlerRequest
     */
    public function PUT()

Method \ClickHouseDB\Transport\CurlerRequest::PUT() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "CurlerRequest".

    {
        return $this->execute('PUT');
    }

    /**
     * @return CurlerRequest
     */
    public function POST()

Method \ClickHouseDB\Transport\CurlerRequest::POST() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "CurlerRequest".

    {
        return $this->execute('POST');
    }

    /**
     * @return CurlerRequest
     */
    public function OPTIONS()

Method \ClickHouseDB\Transport\CurlerRequest::OPTIONS() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "CurlerRequest".

    {
        return $this->execute('OPTIONS');
    }

    /**
     * @return CurlerRequest
     */
    public function GET()

Method \ClickHouseDB\Transport\CurlerRequest::GET() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "CurlerRequest".

    {
        return $this->execute('GET');
    }

    /**
     * The number of seconds that DNS records are stored in memory. By default this parameter is 120 (2 minutes).
     *
     * @param integer $set

Incorrect annotations group.

Expected "int" but found "integer" in @param annotation.

     * @return $this
     */
    public function setDnsCache($set)

Method \ClickHouseDB\Transport\CurlerRequest::setDnsCache() does not have native type hint for its parameter $set but it should be possible to add it based on @param annotation "integer".

    {
        $this->_dns_cache = $set;

Member variable "_dns_cache" is not in valid camel caps format

        return $this;

Expected 1 lines before "return", found 0.

    }

    /**
     * The number of seconds that DNS records are stored in memory. By default this parameter is 120 (2 minutes).
     *
     * @return int
     */
    public function getDnsCache()

Method \ClickHouseDB\Transport\CurlerRequest::getDnsCache() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "int".

    {
        return $this->_dns_cache;

Member variable "_dns_cache" is not in valid camel caps format

    }

    /**
     * Sets client certificate
     *
     * @param string $filePath
     */
    public function setSslCa($filePath)

Method \ClickHouseDB\Transport\CurlerRequest::setSslCa() does not have native type hint for its parameter $filePath but it should be possible to add it based on @param annotation "string".

Method \ClickHouseDB\Transport\CurlerRequest::setSslCa() does not have void return type hint.

    {
        $this->option(CURLOPT_SSL_VERIFYPEER, true);

Constant CURLOPT_SSL_VERIFYPEER should not be referenced via a fallback global name, but via a use statement.

        $this->option(CURLOPT_CAINFO, $filePath);

Constant CURLOPT_CAINFO should not be referenced via a fallback global name, but via a use statement.

    }

    /**
     * @param string $method

Incorrect annotations group.

     * @return $this
     */
    private function execute($method)

Method \ClickHouseDB\Transport\CurlerRequest::execute() does not have native type hint for its parameter $method but it should be possible to add it based on @param annotation "string".

    {
        $this->method = $method;
        return $this;

Expected 1 lines before "return", found 0.

    }

    /**
     * @return CurlerResponse

Incorrect annotations group.

     * @throws \ClickHouseDB\Exception\TransportException

Class \ClickHouseDB\Exception\TransportException should not be referenced via a fully qualified name, but via a use statement.

     */
    public function response()

Method \ClickHouseDB\Transport\CurlerRequest::response() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "CurlerResponse".

    {
        if (!$this->response) {

Expected 1 space after NOT operator; 0 found

            throw new \ClickHouseDB\Exception\TransportException('Can`t fetch response - is empty');

Class \ClickHouseDB\Exception\TransportException should not be referenced via a fully qualified name, but via a use statement.

        }

        return $this->response;
    }

    public function isResponseExists(): bool
    {
        return $this->response !== null;
    }

    public function setResponse(CurlerResponse $response): void
    {
        $this->response = $response;
    }

    /**
     * @return mixed
     */
    public function handle()
    {
        $this->prepareRequest();
        return $this->handle;

Expected 1 lines before "return", found 0.

    }

    /**
     * @param callable $callback

Incorrect annotations group.

Method \ClickHouseDB\Transport\CurlerRequest::setFunctionProgress() has useless @param annotation for parameter $callback.

     * @throws \Exception

Class \Exception should not be referenced via a fully qualified name, but via a use statement.

     */
    public function setFunctionProgress(callable $callback)

Method \ClickHouseDB\Transport\CurlerRequest::setFunctionProgress() does not have void return type hint.

    {
        if (!is_callable($callback)) {

Expected 1 space after NOT operator; 0 found

Function is_callable() should not be referenced via a fallback global name, but via a use statement.

            throw new \Exception('setFunctionProgress not is_callable');

Class \Exception should not be referenced via a fully qualified name, but via a use statement.

        }

        $this->option(CURLOPT_NOPROGRESS, false);

Constant CURLOPT_NOPROGRESS should not be referenced via a fallback global name, but via a use statement.

        $this->option(CURLOPT_PROGRESSFUNCTION, $callback); // version 5.5.0

Constant CURLOPT_PROGRESSFUNCTION should not be referenced via a fallback global name, but via a use statement.

    }

Expected 1 blank line after function; 2 found

    /**
     * @return bool
     */
    private function prepareRequest()

Method \ClickHouseDB\Transport\CurlerRequest::prepareRequest() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "bool".

    {
        if (!$this->handle) {

Expected 1 space after NOT operator; 0 found

            $this->handle = curl_init();

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

Function curl_init() should not be referenced via a fallback global name, but via a use statement.

        }

        $curl_opt = $this->options;

The variable $curl_opt should be in camel caps format.

        $method = $this->method;

Equals sign not aligned with surrounding assignments; expected 3 spaces but found 1 space

        if ($this->_attachFiles) {

Expected 1 lines after "if", found 2.

            $curl_opt[CURLOPT_SAFE_UPLOAD] = true;

The constant CURLOPT_SAFE_UPLOAD has been deprecated: 7.0 Use <b>CURLFile</b> for uploads instead.

The variable $curl_opt should be in camel caps format.

Constant CURLOPT_SAFE_UPLOAD should not be referenced via a fallback global name, but via a use statement.

        }

Functions must not contain multiple empty lines in a row; found 2 empty lines

        if (strtoupper($method) == 'GET') {

Expected 1 lines after "if", found 0.

Function strtoupper() should not be referenced via a fallback global name, but via a use statement.

Operator == is disallowed, use === instead.

            $curl_opt[CURLOPT_HTTPGET] = true;

The variable $curl_opt should be in camel caps format.

Equals sign not aligned with surrounding assignments; expected 7 spaces but found 1 space

            $curl_opt[CURLOPT_CUSTOMREQUEST] = strtoupper($method);

The variable $curl_opt should be in camel caps format.

Constant CURLOPT_CUSTOMREQUEST should not be referenced via a fallback global name, but via a use statement.

Function strtoupper() should not be referenced via a fallback global name, but via a use statement.

            $curl_opt[CURLOPT_POSTFIELDS] = false;

The variable $curl_opt should be in camel caps format.

Constant CURLOPT_POSTFIELDS should not be referenced via a fallback global name, but via a use statement.

Equals sign not aligned with surrounding assignments; expected 4 spaces but found 1 space

        } else {
            if (strtoupper($method) === 'POST') {

Function strtoupper() should not be referenced via a fallback global name, but via a use statement.

                $curl_opt[CURLOPT_POST] = true;

The variable $curl_opt should be in camel caps format.

            }

            $curl_opt[CURLOPT_CUSTOMREQUEST] = strtoupper($method);

The variable $curl_opt should be in camel caps format.

Constant CURLOPT_CUSTOMREQUEST should not be referenced via a fallback global name, but via a use statement.

Function strtoupper() should not be referenced via a fallback global name, but via a use statement.

            if ($this->parameters) {
                $curl_opt[CURLOPT_POSTFIELDS] = $this->parameters;

The variable $curl_opt should be in camel caps format.

Constant CURLOPT_POSTFIELDS should not be referenced via a fallback global name, but via a use statement.

                if (!is_array($this->parameters)) {

Expected 1 space after NOT operator; 0 found

Function is_array() should not be referenced via a fallback global name, but via a use statement.

                    $this->header('Content-Length',  mb_strlen($this->parameters, '8bit'));

Expected 1 space after comma in argument list; 2 found

Function mb_strlen() should not be referenced via a fallback global name, but via a use statement.

                }
            }
        }
        // CURLOPT_DNS_CACHE_TIMEOUT - Количество секунд, в течение которых в памяти хранятся DNS-записи.
        $curl_opt[CURLOPT_DNS_CACHE_TIMEOUT] = $this->getDnsCache();

The variable $curl_opt should be in camel caps format.

Constant CURLOPT_DNS_CACHE_TIMEOUT should not be referenced via a fallback global name, but via a use statement.

        $curl_opt[CURLOPT_URL] = $this->url;

The variable $curl_opt should be in camel caps format.

Constant CURLOPT_URL should not be referenced via a fallback global name, but via a use statement.

Equals sign not aligned with surrounding assignments; expected 15 spaces but found 1 space

        if (!empty($this->headers) && sizeof($this->headers)) {

Expected 1 space after NOT operator; 0 found

Function sizeof() should not be referenced via a fallback global name, but via a use statement.

The use of function sizeof() is forbidden; use count() instead

            $curl_opt[CURLOPT_HTTPHEADER] = [];

The variable $curl_opt should be in camel caps format.

Constant CURLOPT_HTTPHEADER should not be referenced via a fallback global name, but via a use statement.

            foreach ($this->headers as $key => $value) {
                $curl_opt[CURLOPT_HTTPHEADER][] = sprintf("%s: %s", $key, $value);

The variable $curl_opt should be in camel caps format.

Constant CURLOPT_HTTPHEADER should not be referenced via a fallback global name, but via a use statement.

Function sprintf() should not be referenced via a fallback global name, but via a use statement.

The string literal %s: %s does not require double quotes, as per coding-style, please use single quotes.

            }
        }

        if (!empty($curl_opt[CURLOPT_INFILE])) {

The variable $curl_opt should be in camel caps format.

Expected 1 space after NOT operator; 0 found

Constant CURLOPT_INFILE should not be referenced via a fallback global name, but via a use statement.

Blank line found at start of control structure

            $curl_opt[CURLOPT_PUT] = true;

The variable $curl_opt should be in camel caps format.

Constant CURLOPT_PUT should not be referenced via a fallback global name, but via a use statement.

        }

        if (!empty($curl_opt[CURLOPT_WRITEFUNCTION])) {

Expected 1 space after NOT operator; 0 found

The variable $curl_opt should be in camel caps format.

Constant CURLOPT_WRITEFUNCTION should not be referenced via a fallback global name, but via a use statement.

            $curl_opt[CURLOPT_HEADER] = false;

The variable $curl_opt should be in camel caps format.

Constant CURLOPT_HEADER should not be referenced via a fallback global name, but via a use statement.

        }

        if ($this->resultFileHandle) {
            $curl_opt[CURLOPT_FILE] = $this->resultFileHandle;

The variable $curl_opt should be in camel caps format.

Constant CURLOPT_FILE should not be referenced via a fallback global name, but via a use statement.

Equals sign not aligned with surrounding assignments; expected 3 spaces but found 1 space

            $curl_opt[CURLOPT_HEADER] = false;

The variable $curl_opt should be in camel caps format.

Constant CURLOPT_HEADER should not be referenced via a fallback global name, but via a use statement.

        }

        if ($this->options[CURLOPT_VERBOSE]) {

Constant CURLOPT_VERBOSE should not be referenced via a fallback global name, but via a use statement.

            $msg="\n-----------BODY REQUEST----------\n" . $curl_opt[CURLOPT_POSTFIELDS] . "\n------END--------\n";

The variable $curl_opt should be in camel caps format.

Expected at least 1 space before "="; 0 found

Expected at least 1 space after "="; 0 found

Equals sign not aligned correctly; expected 1 space but found 0 spaces

Constant CURLOPT_POSTFIELDS should not be referenced via a fallback global name, but via a use statement.

            if ($this->stdErrOut && is_resource($this->stdErrOut)) {

Function is_resource() should not be referenced via a fallback global name, but via a use statement.

                fwrite($this->stdErrOut,$msg);

Function fwrite() should not be referenced via a fallback global name, but via a use statement.

            } else {
                echo $msg;
            }
        }

        if ($this->stdErrOut) {
            if (is_resource($this->stdErrOut)) {

Function is_resource() should not be referenced via a fallback global name, but via a use statement.

                $curl_opt[CURLOPT_STDERR]=$this->stdErrOut;

The variable $curl_opt should be in camel caps format.

Constant CURLOPT_STDERR should not be referenced via a fallback global name, but via a use statement.

Expected at least 1 space before "="; 0 found

Expected at least 1 space after "="; 0 found

Equals sign not aligned correctly; expected 1 space but found 0 spaces

            }
        }

        curl_setopt_array($this->handle, $curl_opt);

The variable $curl_opt should be in camel caps format.

Function curl_setopt_array() should not be referenced via a fallback global name, but via a use statement.

        return true;

Expected 1 lines before "return", found 0.

    }
}