smi2 / phpClickHouse

src/Transport/CurlerRequest.php

<?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 = array();

Short array syntax must be used to define arrays

    /**

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.

Class CurlerRequest contains unused property $sslCa.

    /**
     * @param bool $id
     */
    public function __construct($id = false)

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".

    {
        $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

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

The constant CURLOPT_BINARYTRANSFER has been deprecated: 5.1.3

            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.

        );
    }

    /**

Empty comment

     *
     */
    public function __destruct()

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

    {
        $this->close();
    }

Expected 1 blank line after function; 2 found

    public function close()

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

    {
        if ($this->handle) {

Expected 1 lines after "if", found 0.

            curl_close($this->handle);

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

        }
        $this->handle = null;
    }

    /**
     * @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);

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

The variable $post_name should be in camel caps format.

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;

The variable $infile_handle should be in 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');

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

The variable $infile_handle should be in camel caps format.

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.

Blank line found at start of control structure

The variable $infile_handle should be in camel caps format.

            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.

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.

The variable $infile_handle should be in camel caps format.

                $this->options[CURLOPT_SAFE_UPLOAD] = 1;

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

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

            } 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.

The variable $infile_handle should be in camel caps format.

        }

        return $this->infile_handle;

The variable $infile_handle should be in 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;

The variable $callback_function should be in 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;

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

The variable $callback_class should be in camel caps format.

        $this->callback_functionName = $functionName;

The variable $callback_functionName should be in 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) {

The variable $callback_function should be in camel caps format.

            $x = $this->callback_function;

The variable $callback_function should be in camel caps format.

            $x($this);
        }

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

Use early exit to reduce code nesting.

The variable $callback_class should be in camel caps format.

The variable $callback_functionName should be in camel caps format.

            $c = $this->callback_functionName;

The variable $callback_functionName should be in camel caps format.

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

The variable $callback_class should be in 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 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";

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

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

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

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

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

        $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.

     * @return $this
     */
    public function keepAlive($sec = 60)

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

    {
        $this->options[CURLOPT_FORBID_REUSE] = TRUE;

Constant CURLOPT_FORBID_REUSE 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.

        $this->headers['Connection'] = 'Keep-Alive';

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

        $this->headers['Keep-Alive'] = $sec;

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

        return $this;
    }

    /**
     * @param bool $flag

Incorrect annotations group.

     * @return $this
     */
    public function verbose($flag = true)

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

    {
        $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.

     * @param string $value
     * @return $this
     */
    public function header($key, $value)

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

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

    {
        $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()

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

    {
        $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.

     * @return $this
     */
    public function url($url)

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

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

Expected 1 lines before "return", found 0.

    }

    /**
     * @return mixed
     */
    public function getUrl()
    {
        return $this->url;
    }

Expected 1 blank line after function; 2 found

    /**
     * @param string $id

Incorrect annotations group.

     * @return string
     */
    public function getUniqHash($id)

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

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

    {
        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
     */
    public function httpCompression($flag)

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

Method \ClickHouseDB\Transport\CurlerRequest::httpCompression() does not have void 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 int $seconds

Incorrect annotations group.

     * @return $this
     */
    public function connectTimeOut($seconds = 1)

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

    {
        $this->options[CURLOPT_CONNECTTIMEOUT] = $seconds;

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

        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.

     * @return $this
     */
    public function timeOut($seconds = 10)

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

    {
        return $this->timeOutMs(intval($seconds * 1000));

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

    }

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

Incorrect annotations group.

     * @return $this
     */
    protected function timeOutMs($ms = 10000)

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

    {
        $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(s) 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;

The variable $_dns_cache should be in 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;

The variable $_dns_cache should be in 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(s) 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(s) 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(s) after NOT operator; 0 found

            $this->handle = curl_init();

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

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?

        }

        $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;

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

The variable $curl_opt should be in camel caps format.

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

        }

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

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

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

Expected 1 lines after "if", found 0.

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

            $curl_opt[CURLOPT_HTTPGET] = true;

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

The variable $curl_opt should be in camel caps format.

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

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.

The variable $curl_opt should be in camel caps format.

            $curl_opt[CURLOPT_POSTFIELDS] = false;

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

The variable $curl_opt should be in camel caps format.

        } 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);

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.

The variable $curl_opt should be in camel caps format.

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

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

The variable $curl_opt should be in camel caps format.

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

Expected 1 space(s) 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', strlen($this->parameters));

Function 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();

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

The variable $curl_opt should be in camel caps format.

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

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

The variable $curl_opt should be in camel caps format.

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

Expected 1 space(s) 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] = array();

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

Short array syntax must be used to define arrays

The variable $curl_opt should be in camel caps format.

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

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.

The variable $curl_opt should be in camel caps format.

            }
        }

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

Expected 1 space(s) 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

The variable $curl_opt should be in camel caps format.

            $curl_opt[CURLOPT_PUT] = true;

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

The variable $curl_opt should be in camel caps format.

        }

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

Expected 1 space(s) after NOT operator; 0 found

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

The variable $curl_opt should be in camel caps format.

            $curl_opt[CURLOPT_HEADER] = false;

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

The variable $curl_opt should be in camel caps format.

        }

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

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

The variable $curl_opt should be in camel caps format.

            $curl_opt[CURLOPT_HEADER] = false;

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

The variable $curl_opt should be in camel caps format.

        }

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

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

Expected 1 lines after "if", found 0.

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

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

The variable $curl_opt should be in camel caps format.

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

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

The variable $curl_opt should be in camel caps format.

        return true;

Expected 1 lines before "return", found 0.

    }
}