Passed
Pull Request — master (#140)
by Dmitriy
06:25
created

CurlerRequest   F

Complexity

Total Complexity 87

Size/Duplication

Total Lines 749
Duplicated Lines 0 %

Test Coverage

Coverage 67.72%

Importance

Changes 2
Bugs 1 Features 0
Metric Value
eloc 211
c 2
b 1
f 0
dl 0
loc 749
ccs 172
cts 254
cp 0.6772
rs 2
wmc 87

53 Methods

Rating   Name   Duplication   Size   Complexity  
A setWriteFunction() 0 3 1
A setCallbackFunction() 0 3 1
A close() 0 6 2
A setReadFunction() 0 3 1
A setInfile() 0 21 3
A id() 0 7 2
A __destruct() 0 3 1
A __construct() 0 21 1
A getInfileHandle() 0 3 1
A onCallback() 0 10 4
A attachFiles() 0 11 2
A setRequestExtendedInfo() 0 4 1
A getRequestExtendedInfo() 0 7 3
A setCallback() 0 4 1
A setHeaderFunction() 0 3 1
A keepAlive() 0 7 1
A httpCompression() 0 8 2
A getUniqHash() 0 3 1
A isResultFile() 0 3 2
A parameters_json() 0 23 5
A isPersistent() 0 3 1
A dump() 0 15 2
F prepareRequest() 0 64 14
A GET() 0 3 1
A url() 0 4 1
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 auth() 0 4 1
A timeOutMs() 0 4 1
A persistent() 0 4 1
A getUrl() 0 3 1
A isResponseExists() 0 3 1
A getDetails() 0 7 1
A header() 0 4 1
A getHeaders() 0 7 2
A parameters() 0 4 1
A connectTimeOut() 0 4 1
A setSslCa() 0 4 1
A handle() 0 4 1
A POST() 0 3 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 PUT() 0 3 1
A setResultFileHandle() 0 8 2
A setFunctionProgress() 0 8 2
A option() 0 4 1

How to fix   Complexity   

Complex Class

Complex classes like CurlerRequest often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes.

Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.

While breaking up the class, it is a good idea to analyze how other classes use CurlerRequest, and based on these observations, apply Extract Interface, too.

1
<?php
0 ignored issues
show
introduced by
Missing declare(strict_types=1).
Loading history...
2
3
namespace ClickHouseDB\Transport;
4
5
use const CURLOPT_HTTPGET;
6
use const CURLOPT_POST;
7
8
class CurlerRequest
9
{
10
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$extendinfo with single line content, use one-line comment instead.
Loading history...
11
     * @var array
0 ignored issues
show
introduced by
@var annotation of property \ClickHouseDB\Transport\CurlerRequest::$extendinfo does not specify type hint for its items.
Loading history...
12
     */
13
    public $extendinfo = array();
0 ignored issues
show
Coding Style introduced by
Short array syntax must be used to define arrays
Loading history...
14
15
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$parameters with single line content, use one-line comment instead.
Loading history...
16
     * @var string|array
0 ignored issues
show
introduced by
@var annotation of property \ClickHouseDB\Transport\CurlerRequest::$parameters does not specify type hint for its items.
Loading history...
17
     */
18
    private $parameters = '';
19
20
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$options with single line content, use one-line comment instead.
Loading history...
21
     * @var array
0 ignored issues
show
introduced by
@var annotation of property \ClickHouseDB\Transport\CurlerRequest::$options does not specify type hint for its items.
Loading history...
22
     */
23
    private $options;
24
25
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$headers with single line content, use one-line comment instead.
Loading history...
26
     * @var array
0 ignored issues
show
introduced by
@var annotation of property \ClickHouseDB\Transport\CurlerRequest::$headers does not specify type hint for its items.
Loading history...
27
     */
28
    private $headers; // Parsed reponse header object.
29
30
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$url with single line content, use one-line comment instead.
Loading history...
31
     * @var string
32
     */
33
    private $url;
34
35
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$method with single line content, use one-line comment instead.
Loading history...
36
     * @var string
37
     */
38
    private $method;
39
40
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$id with single line content, use one-line comment instead.
Loading history...
41
     * @var bool
42
     */
43
    private $id;
44
45
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$handle with single line content, use one-line comment instead.
Loading history...
46
     * @var resource|null
47
     */
48
    private $handle;
49
50
    /** @var CurlerResponse */
51
    private $response;
52
53
    /** @var bool */
54
    private $_persistent = false;
55
56
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$_attachFiles with single line content, use one-line comment instead.
Loading history...
57
     * @var bool
58
     */
59
    private $_attachFiles = false;
60
61
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$callback_class with single line content, use one-line comment instead.
Loading history...
62
     * @var string
63
     */
64
    private $callback_class = '';
65
66
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$callback_functionName with single line content, use one-line comment instead.
Loading history...
67
     * @var string
68
     */
69
    private $callback_functionName = '';
70
71
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$_httpCompression with single line content, use one-line comment instead.
Loading history...
72
     * @var bool
73
     */
74
    private $_httpCompression = false;
75
76
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$callback_function with single line content, use one-line comment instead.
Loading history...
77
     * @var callable
78
     */
79
    private $callback_function = null;
80
81
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$infile_handle with single line content, use one-line comment instead.
Loading history...
82
     * @var bool|resource
83
     */
84
    private $infile_handle = false;
85
86
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$_dns_cache with single line content, use one-line comment instead.
Loading history...
87
     * @var int
88
     */
89
    private $_dns_cache = 120;
90
91
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$resultFileHandle with single line content, use one-line comment instead.
Loading history...
92
     * @var resource
93
     */
94
    private $resultFileHandle = null;
95
96
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$sslCa with single line content, use one-line comment instead.
Loading history...
97
     * @var string
98
     */
99
    private $sslCa = null;
0 ignored issues
show
introduced by
The private property $sslCa is not used, and could be removed.
Loading history...
introduced by
Class CurlerRequest contains unused property $sslCa.
Loading history...
100
101
    /**
102
     * @param bool $id
103
     */
104 52
    public function __construct($id = false)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::__construct() does not have parameter type hint for its parameter $id but it should be possible to add it based on @param annotation "bool".
Loading history...
105
    {
106 52
        $this->id = $id;
107
108 52
        $this->header('Cache-Control', 'no-cache, no-store, must-revalidate');
109 52
        $this->header('Expires', '0');
110 52
        $this->header('Pragma', 'no-cache');
111
112 52
        $this->options = array(
0 ignored issues
show
Coding Style introduced by
Short array syntax must be used to define arrays
Loading history...
113 52
            CURLOPT_SSL_VERIFYHOST => 0,
0 ignored issues
show
introduced by
Constant CURLOPT_SSL_VERIFYHOST should not be referenced via a fallback global name, but via a use statement.
Loading history...
114 52
            CURLOPT_SSL_VERIFYPEER => false,
0 ignored issues
show
introduced by
Constant CURLOPT_SSL_VERIFYPEER should not be referenced via a fallback global name, but via a use statement.
Loading history...
115 52
            CURLOPT_TIMEOUT => 10,
0 ignored issues
show
introduced by
Constant CURLOPT_TIMEOUT should not be referenced via a fallback global name, but via a use statement.
Loading history...
116 52
            CURLOPT_CONNECTTIMEOUT => 5, // Количество секунд ожидания при попытке соединения
0 ignored issues
show
introduced by
Constant CURLOPT_CONNECTTIMEOUT should not be referenced via a fallback global name, but via a use statement.
Loading history...
117 52
            CURLOPT_HTTP_VERSION => CURL_HTTP_VERSION_1_1,
0 ignored issues
show
introduced by
Constant CURLOPT_HTTP_VERSION should not be referenced via a fallback global name, but via a use statement.
Loading history...
introduced by
Constant CURL_HTTP_VERSION_1_1 should not be referenced via a fallback global name, but via a use statement.
Loading history...
118 52
            CURLOPT_MAXREDIRS => 10,
0 ignored issues
show
introduced by
Constant CURLOPT_MAXREDIRS should not be referenced via a fallback global name, but via a use statement.
Loading history...
119 52
            CURLOPT_HEADER => TRUE,
0 ignored issues
show
introduced by
Constant CURLOPT_HEADER should not be referenced via a fallback global name, but via a use statement.
Loading history...
Coding Style introduced by
TRUE, FALSE and NULL must be lowercase; expected true, but found TRUE.
Loading history...
120 52
            CURLOPT_FOLLOWLOCATION => TRUE,
0 ignored issues
show
introduced by
Constant CURLOPT_FOLLOWLOCATION should not be referenced via a fallback global name, but via a use statement.
Loading history...
Coding Style introduced by
TRUE, FALSE and NULL must be lowercase; expected true, but found TRUE.
Loading history...
121 52
            CURLOPT_AUTOREFERER => 1, // при редиректе подставлять в «Referer:» значение из «Location:»
0 ignored issues
show
introduced by
Constant CURLOPT_AUTOREFERER should not be referenced via a fallback global name, but via a use statement.
Loading history...
122 52
            CURLOPT_BINARYTRANSFER => 1, // передавать в binary-safe
0 ignored issues
show
introduced by
Constant CURLOPT_BINARYTRANSFER should not be referenced via a fallback global name, but via a use statement.
Loading history...
123 52
            CURLOPT_RETURNTRANSFER => TRUE,
0 ignored issues
show
introduced by
Constant CURLOPT_RETURNTRANSFER should not be referenced via a fallback global name, but via a use statement.
Loading history...
Coding Style introduced by
TRUE, FALSE and NULL must be lowercase; expected true, but found TRUE.
Loading history...
124 52
            CURLOPT_USERAGENT => 'smi2/PHPClickHouse/client',
0 ignored issues
show
introduced by
Constant CURLOPT_USERAGENT should not be referenced via a fallback global name, but via a use statement.
Loading history...
125
        );
126 52
    }
127
128
    /**
0 ignored issues
show
introduced by
Empty comment
Loading history...
129
     *
130
     */
131 47
    public function __destruct()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::__destruct() does not need documentation comment.
Loading history...
132
    {
133 47
        $this->close();
134 47
    }
135
136
137 47
    public function close()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::close() does not have void return type hint.
Loading history...
138
    {
139 47
        if ($this->handle) {
140 47
            curl_close($this->handle);
0 ignored issues
show
introduced by
Function curl_close() should not be referenced via a fallback global name, but via a use statement.
Loading history...
141
        }
142 47
        $this->handle = null;
143 47
    }
144
145
    /**
146
     * @param array $attachFiles
0 ignored issues
show
introduced by
@param annotation of method \ClickHouseDB\Transport\CurlerRequest::attachFiles() does not specify type hint for items of its traversable parameter $attachFiles.
Loading history...
147
     */
148 1
    public function attachFiles($attachFiles)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::attachFiles() does not have parameter type hint for its parameter $attachFiles but it should be possible to add it based on @param annotation "array".
Loading history...
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::attachFiles() does not have void return type hint.
Loading history...
149
    {
150 1
        $this->header("Content-Type", "multipart/form-data");
0 ignored issues
show
Coding Style Comprehensibility introduced by
The string literal Content-Type does not require double quotes, as per coding-style, please use single quotes.

PHP provides two ways to mark string literals. Either with single quotes 'literal' or with double quotes "literal". The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences.

String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (\') and the backslash (\\). Every other character is displayed as is.

Double quoted string literals may contain other variables or more complex escape sequences.

<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

will print an indented: Single is Value

If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear.

For more information on PHP string literals and available escape sequences see the PHP core documentation.

Loading history...
Coding Style Comprehensibility introduced by
The string literal multipart/form-data does not require double quotes, as per coding-style, please use single quotes.

PHP provides two ways to mark string literals. Either with single quotes 'literal' or with double quotes "literal". The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences.

String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (\') and the backslash (\\). Every other character is displayed as is.

Double quoted string literals may contain other variables or more complex escape sequences.

<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

will print an indented: Single is Value

If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear.

For more information on PHP string literals and available escape sequences see the PHP core documentation.

Loading history...
151
152 1
        $out = [];
153 1
        foreach ($attachFiles as $post_name => $file_path) {
154 1
            $out[$post_name] = new \CURLFile($file_path);
0 ignored issues
show
introduced by
Class \CURLFile should not be referenced via a fully qualified name, but via a use statement.
Loading history...
155
        }
156
157 1
        $this->_attachFiles = true;
158 1
        $this->parameters($out);
159 1
    }
160
161
162
    /**
163
     * @param bool $set
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
164
     * @return $this
165
     */
166
    public function id($set = false)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::id() does not have parameter type hint for its parameter $set but it should be possible to add it based on @param annotation "bool".
Loading history...
167
    {
168
        if ($set) {
169
            $this->id = $set;
170
        }
171
172
        return $this;
173
    }
174
175
    /**
176
     * @param array $params
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
introduced by
@param annotation of method \ClickHouseDB\Transport\CurlerRequest::setRequestExtendedInfo() does not specify type hint for items of its traversable parameter $params.
Loading history...
177
     * @return $this
178
     */
179 43
    public function setRequestExtendedInfo($params)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::setRequestExtendedInfo() does not have parameter type hint for its parameter $params but it should be possible to add it based on @param annotation "array".
Loading history...
180
    {
181 43
        $this->extendinfo = $params;
182 43
        return $this;
183
    }
184
185
    /**
186
     * @param string|integer|null $key
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
introduced by
Expected "int" but found "integer" in @param annotation.
Loading history...
187
     * @return mixed
188
     */
189 43
    public function getRequestExtendedInfo($key = null)
190
    {
191 43
        if ($key) {
192 43
            return isset($this->extendinfo[$key]) ? $this->extendinfo[$key] : false;
0 ignored issues
show
introduced by
Use null coalesce operator instead of ternary operator.
Loading history...
193
        }
194
195
        return $this->extendinfo;
196
    }
197
198
    /**
199
     * @return bool|resource
200
     */
201 8
    public function getInfileHandle()
202
    {
203 8
        return $this->infile_handle;
204
    }
205
206
    /**
207
     * @param string $file_name
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
208
     * @return bool|resource
209
     */
210 8
    public function setInfile($file_name)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::setInfile() does not have parameter type hint for its parameter $file_name but it should be possible to add it based on @param annotation "string".
Loading history...
211
    {
212 8
        $this->header('Expect', '');
213 8
        $this->infile_handle = fopen($file_name, 'r');
0 ignored issues
show
introduced by
Function fopen() should not be referenced via a fallback global name, but via a use statement.
Loading history...
214 8
        if (is_resource($this->infile_handle)) {
0 ignored issues
show
introduced by
Function is_resource() should not be referenced via a fallback global name, but via a use statement.
Loading history...
Coding Style introduced by
Blank line found at start of control structure
Loading history...
215
216 8
            if ($this->_httpCompression) {
217 8
                $this->header('Content-Encoding', 'gzip');
218 8
                $this->header('Content-Type', 'application/x-www-form-urlencoded');
219
220 8
                stream_filter_append($this->infile_handle, 'zlib.deflate', STREAM_FILTER_READ, ["window" => 30]);
0 ignored issues
show
introduced by
Function stream_filter_append() should not be referenced via a fallback global name, but via a use statement.
Loading history...
introduced by
Constant STREAM_FILTER_READ should not be referenced via a fallback global name, but via a use statement.
Loading history...
Coding Style Comprehensibility introduced by
The string literal window does not require double quotes, as per coding-style, please use single quotes.

PHP provides two ways to mark string literals. Either with single quotes 'literal' or with double quotes "literal". The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences.

String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (\') and the backslash (\\). Every other character is displayed as is.

Double quoted string literals may contain other variables or more complex escape sequences.

<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

will print an indented: Single is Value

If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear.

For more information on PHP string literals and available escape sequences see the PHP core documentation.

Loading history...
221
222 8
                $this->options[CURLOPT_SAFE_UPLOAD] = 1;
0 ignored issues
show
introduced by
Constant CURLOPT_SAFE_UPLOAD should not be referenced via a fallback global name, but via a use statement.
Loading history...
223
            } else {
224
                $this->options[CURLOPT_INFILESIZE] = filesize($file_name);
0 ignored issues
show
introduced by
Constant CURLOPT_INFILESIZE should not be referenced via a fallback global name, but via a use statement.
Loading history...
introduced by
Function filesize() should not be referenced via a fallback global name, but via a use statement.
Loading history...
225
            }
226
227 8
            $this->options[CURLOPT_INFILE] = $this->infile_handle;
0 ignored issues
show
introduced by
Constant CURLOPT_INFILE should not be referenced via a fallback global name, but via a use statement.
Loading history...
228
        }
229
230 8
        return $this->infile_handle;
231
    }
232
233
    /**
234
     * @param callable $callback
235
     */
236 8
    public function setCallbackFunction($callback)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::setCallbackFunction() does not have parameter type hint for its parameter $callback but it should be possible to add it based on @param annotation "callable".
Loading history...
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::setCallbackFunction() does not have void return type hint.
Loading history...
237
    {
238 8
        $this->callback_function = $callback;
239 8
    }
240
241
    /**
242
     * @param callable $callback
243
     */
244 1
    public function setWriteFunction($callback)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::setWriteFunction() does not have parameter type hint for its parameter $callback but it should be possible to add it based on @param annotation "callable".
Loading history...
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::setWriteFunction() does not have void return type hint.
Loading history...
245
    {
246 1
        $this->options[CURLOPT_WRITEFUNCTION] = $callback;
0 ignored issues
show
introduced by
Constant CURLOPT_WRITEFUNCTION should not be referenced via a fallback global name, but via a use statement.
Loading history...
247 1
    }
248
249
    /**
250
     * @param callable $callback
251
     */
252 3
    public function setReadFunction($callback)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::setReadFunction() does not have parameter type hint for its parameter $callback but it should be possible to add it based on @param annotation "callable".
Loading history...
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::setReadFunction() does not have void return type hint.
Loading history...
253
    {
254 3
        $this->options[CURLOPT_READFUNCTION] = $callback;
0 ignored issues
show
introduced by
Constant CURLOPT_READFUNCTION should not be referenced via a fallback global name, but via a use statement.
Loading history...
255 3
    }
256
257
    public function setHeaderFunction($callback)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::setHeaderFunction() does not have parameter type hint nor @param annotation for its parameter $callback.
Loading history...
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::setHeaderFunction() does not have void return type hint.
Loading history...
258
    {
259
        $this->options[CURLOPT_HEADERFUNCTION] = $callback;
0 ignored issues
show
introduced by
Constant CURLOPT_HEADERFUNCTION should not be referenced via a fallback global name, but via a use statement.
Loading history...
260
    }
261
262
    /**
263
     * @param string $classCallBack
264
     * @param string $functionName
265
     */
266
    public function setCallback($classCallBack, $functionName)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::setCallback() does not have parameter type hint for its parameter $classCallBack but it should be possible to add it based on @param annotation "string".
Loading history...
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::setCallback() does not have parameter type hint for its parameter $functionName but it should be possible to add it based on @param annotation "string".
Loading history...
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::setCallback() does not have void return type hint.
Loading history...
267
    {
268
        $this->callback_class = $classCallBack;
0 ignored issues
show
Coding Style introduced by
Equals sign not aligned with surrounding assignments; expected 8 spaces but found 1 space

This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line.

To visualize

$a = "a";
$ab = "ab";
$abc = "abc";

will produce issues in the first and second line, while this second example

$a   = "a";
$ab  = "ab";
$abc = "abc";

will produce no issues.

Loading history...
269
        $this->callback_functionName = $functionName;
270
    }
271
272
    /**
0 ignored issues
show
introduced by
Empty comment
Loading history...
273
     *
274
     */
275 10
    public function onCallback()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::onCallback() does not have void return type hint.
Loading history...
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::onCallback() does not need documentation comment.
Loading history...
276
    {
277 10
        if ($this->callback_function) {
278 8
            $x = $this->callback_function;
279 8
            $x($this);
280
        }
281
282 10
        if ($this->callback_class && $this->callback_functionName) {
0 ignored issues
show
introduced by
Use early exit to reduce code nesting.
Loading history...
283
            $c = $this->callback_functionName;
284
            $this->callback_class->$c($this);
285
        }
286 10
    }
287
288
    public function getDetails(): array
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::getDetails() does not have @return annotation for its traversable return value.
Loading history...
introduced by
There must be exactly 1 whitespace between closing parenthesis and return type colon.
Loading history...
289
    {
290
        return [
291
            'url'        => $this->url,
292
            'method'     => $this->method,
293
            'parameters' => $this->parameters,
294
            'headers'    => $this->headers,
295
        ];
296
    }
297
298
    /**
299
     * @param bool $result
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
300
     * @return string
301
     */
302
    public function dump($result = false)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::dump() does not have parameter type hint for its parameter $result but it should be possible to add it based on @param annotation "bool".
Loading history...
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::dump() does not have return type hint for its return value but it should be possible to add it based on @return annotation "string".
Loading history...
303
    {
304
        $message = "\n------------  Request ------------\n";
0 ignored issues
show
Coding Style introduced by
Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space

This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line.

To visualize

$a = "a";
$ab = "ab";
$abc = "abc";

will produce issues in the first and second line, while this second example

$a   = "a";
$ab  = "ab";
$abc = "abc";

will produce no issues.

Loading history...
305
        $message .= 'URL:' . $this->url . "\n\n";
306
        $message .= 'METHOD:' . $this->method . "\n\n";
307
        $message .= 'PARAMS:' . print_r($this->parameters, true) . "\n";
0 ignored issues
show
introduced by
Function print_r() should not be referenced via a fallback global name, but via a use statement.
Loading history...
308
        $message .= 'PARAMS:' . print_r($this->headers, true) . "\n";
0 ignored issues
show
introduced by
Function print_r() should not be referenced via a fallback global name, but via a use statement.
Loading history...
309
        $message .= "-----------------------------------\n";
310
311
        if ($result) {
312
            return $message;
313
        }
314
315
        echo $message;
316
        return '';
317
    }
318
319
    /**
320
     * @return bool
321
     */
322 15
    public function getId()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::getId() does not have return type hint for its return value but it should be possible to add it based on @return annotation "bool".
Loading history...
323
    {
324 15
        return $this->id;
325
    }
326
327
    /**
328
     * @param integer $key
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
introduced by
Expected "int" but found "integer" in @param annotation.
Loading history...
329
     * @param mixed $value
0 ignored issues
show
Coding Style introduced by
Expected 3 spaces after parameter type; 1 found
Loading history...
330
     * @return $this
331
     */
332
    private function option($key, $value)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::option() does not have parameter type hint for its parameter $key but it should be possible to add it based on @param annotation "integer".
Loading history...
333
    {
334
        $this->options[$key] = $value;
335
        return $this;
336
    }
337
338
    /**
339
     * @return $this
340
     */
341 1
    public function persistent()
342
    {
343 1
        $this->_persistent = true;
344 1
        return $this;
345
    }
346
347
    /**
348
     * @return bool
349
     */
350 10
    public function isPersistent()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::isPersistent() does not have return type hint for its return value but it should be possible to add it based on @return annotation "bool".
Loading history...
351
    {
352 10
        return $this->_persistent;
353
    }
354
355
    /**
356
     * @param int $sec
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
357
     * @return $this
358
     */
359
    public function keepAlive($sec = 60)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::keepAlive() does not have parameter type hint for its parameter $sec but it should be possible to add it based on @param annotation "int".
Loading history...
360
    {
361
        $this->options[CURLOPT_FORBID_REUSE] = TRUE;
0 ignored issues
show
introduced by
Constant CURLOPT_FORBID_REUSE should not be referenced via a fallback global name, but via a use statement.
Loading history...
Coding Style introduced by
TRUE, FALSE and NULL must be lowercase; expected true, but found TRUE.
Loading history...
362
        $this->headers['Connection'] = 'Keep-Alive';
0 ignored issues
show
Coding Style introduced by
Equals sign not aligned with surrounding assignments; expected 9 spaces but found 1 space

This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line.

To visualize

$a = "a";
$ab = "ab";
$abc = "abc";

will produce issues in the first and second line, while this second example

$a   = "a";
$ab  = "ab";
$abc = "abc";

will produce no issues.

Loading history...
363
        $this->headers['Keep-Alive'] = $sec;
0 ignored issues
show
Coding Style introduced by
Equals sign not aligned with surrounding assignments; expected 9 spaces but found 1 space

This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line.

To visualize

$a = "a";
$ab = "ab";
$abc = "abc";

will produce issues in the first and second line, while this second example

$a   = "a";
$ab  = "ab";
$abc = "abc";

will produce no issues.

Loading history...
364
365
        return $this;
366
    }
367
368
    /**
369
     * @param bool $flag
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
370
     * @return $this
371
     */
372 52
    public function verbose($flag = true)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::verbose() does not have parameter type hint for its parameter $flag but it should be possible to add it based on @param annotation "bool".
Loading history...
373
    {
374 52
        $this->options[CURLOPT_VERBOSE] = $flag;
0 ignored issues
show
introduced by
Constant CURLOPT_VERBOSE should not be referenced via a fallback global name, but via a use statement.
Loading history...
375 52
        return $this;
376
    }
377
378
    /**
379
     * @param string $key
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
380
     * @param string $value
381
     * @return $this
382
     */
383 52
    public function header($key, $value)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::header() does not have parameter type hint for its parameter $key but it should be possible to add it based on @param annotation "string".
Loading history...
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::header() does not have parameter type hint for its parameter $value but it should be possible to add it based on @param annotation "string".
Loading history...
384
    {
385 52
        $this->headers[$key] = $value;
386 52
        return $this;
387
    }
388
389
    /**
390
     * @return array
0 ignored issues
show
introduced by
@return annotation of method \ClickHouseDB\Transport\CurlerRequest::getHeaders() does not specify type hint for items of its traversable return value.
Loading history...
391
     */
392
    public function getHeaders()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::getHeaders() does not have return type hint for its return value but it should be possible to add it based on @return annotation "array".
Loading history...
393
    {
394
        $head = [];
395
        foreach ($this->headers as $key => $value) {
396
            $head[] = sprintf("%s: %s", $key, $value);
0 ignored issues
show
introduced by
Function sprintf() should not be referenced via a fallback global name, but via a use statement.
Loading history...
Coding Style Comprehensibility introduced by
The string literal %s: %s does not require double quotes, as per coding-style, please use single quotes.

PHP provides two ways to mark string literals. Either with single quotes 'literal' or with double quotes "literal". The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences.

String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (\') and the backslash (\\). Every other character is displayed as is.

Double quoted string literals may contain other variables or more complex escape sequences.

<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

will print an indented: Single is Value

If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear.

For more information on PHP string literals and available escape sequences see the PHP core documentation.

Loading history...
397
        }
398
        return $head;
399
    }
400
401
    /**
402
     * @param string $url
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
403
     * @return $this
404
     */
405 52
    public function url($url)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::url() does not have parameter type hint for its parameter $url but it should be possible to add it based on @param annotation "string".
Loading history...
406
    {
407 52
        $this->url = $url;
408 52
        return $this;
409
    }
410
411
    /**
412
     * @return mixed
413
     */
414
    public function getUrl()
415
    {
416
        return $this->url;
417
    }
418
419
420
    /**
421
     * @param string $id
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
422
     * @return string
423
     */
424 15
    public function getUniqHash($id)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::getUniqHash() does not have parameter type hint for its parameter $id but it should be possible to add it based on @param annotation "string".
Loading history...
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::getUniqHash() does not have return type hint for its return value but it should be possible to add it based on @return annotation "string".
Loading history...
425
    {
426 15
        return $id . '.' . microtime() . mt_rand(0, 1000000);
0 ignored issues
show
introduced by
Function microtime() should not be referenced via a fallback global name, but via a use statement.
Loading history...
introduced by
Function mt_rand() should not be referenced via a fallback global name, but via a use statement.
Loading history...
427
    }
428
429
    /**
430
     * @param bool $flag
431
     */
432 43
    public function httpCompression($flag)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::httpCompression() does not have parameter type hint for its parameter $flag but it should be possible to add it based on @param annotation "bool".
Loading history...
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::httpCompression() does not have void return type hint.
Loading history...
433
    {
434 43
        if ($flag) {
435 43
            $this->_httpCompression = $flag;
0 ignored issues
show
Coding Style introduced by
Equals sign not aligned with surrounding assignments; expected 10 spaces but found 1 space

This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line.

To visualize

$a = "a";
$ab = "ab";
$abc = "abc";

will produce issues in the first and second line, while this second example

$a   = "a";
$ab  = "ab";
$abc = "abc";

will produce no issues.

Loading history...
436 43
            $this->options[CURLOPT_ENCODING] = 'gzip';
0 ignored issues
show
introduced by
Constant CURLOPT_ENCODING should not be referenced via a fallback global name, but via a use statement.
Loading history...
437
        } else {
438
            $this->_httpCompression = false;
439
            unset($this->options[CURLOPT_ENCODING]);
0 ignored issues
show
introduced by
Constant CURLOPT_ENCODING should not be referenced via a fallback global name, but via a use statement.
Loading history...
440
        }
441 43
    }
442
443
    /**
444
     * @param string $username
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
445
     * @param string $password
446
     * @return $this
447
     */
448
    public function auth($username, $password)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::auth() does not have parameter type hint for its parameter $username but it should be possible to add it based on @param annotation "string".
Loading history...
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::auth() does not have parameter type hint for its parameter $password but it should be possible to add it based on @param annotation "string".
Loading history...
449
    {
450
        $this->options[CURLOPT_USERPWD] = sprintf("%s:%s", $username, $password);
0 ignored issues
show
introduced by
Constant CURLOPT_USERPWD should not be referenced via a fallback global name, but via a use statement.
Loading history...
introduced by
Function sprintf() should not be referenced via a fallback global name, but via a use statement.
Loading history...
Coding Style Comprehensibility introduced by
The string literal %s:%s does not require double quotes, as per coding-style, please use single quotes.

PHP provides two ways to mark string literals. Either with single quotes 'literal' or with double quotes "literal". The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences.

String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (\') and the backslash (\\). Every other character is displayed as is.

Double quoted string literals may contain other variables or more complex escape sequences.

<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

will print an indented: Single is Value

If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear.

For more information on PHP string literals and available escape sequences see the PHP core documentation.

Loading history...
451
        return $this;
452
    }
453
454 43
    public function authByHeaders($username, $password)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::authByHeaders() does not have parameter type hint nor @param annotation for its parameter $username.
Loading history...
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::authByHeaders() does not have parameter type hint nor @param annotation for its parameter $password.
Loading history...
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::authByHeaders() does not have return type hint nor @return annotation for its return value.
Loading history...
455
    {
456 43
        $this->headers['X-ClickHouse-User'] = $username;
457 43
        $this->headers['X-ClickHouse-Key'] = $password;
0 ignored issues
show
Coding Style introduced by
Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space

This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line.

To visualize

$a = "a";
$ab = "ab";
$abc = "abc";

will produce issues in the first and second line, while this second example

$a   = "a";
$ab  = "ab";
$abc = "abc";

will produce no issues.

Loading history...
458 43
        return $this;
459
    }
460
461
    /**
462
     * @param array|string $data
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
introduced by
@param annotation of method \ClickHouseDB\Transport\CurlerRequest::parameters() does not specify type hint for items of its traversable parameter $data.
Loading history...
463
     * @return $this
464
     */
465 1
    public function parameters($data)
466
    {
467 1
        $this->parameters = $data;
468 1
        return $this;
469
    }
470
471
    /**
472
     * The number of seconds to wait when trying to connect. Use 0 for infinite waiting.
473
     *
474
     * @param int $seconds
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
475
     * @return $this
476
     */
477 52
    public function connectTimeOut($seconds = 1)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::connectTimeOut() does not have parameter type hint for its parameter $seconds but it should be possible to add it based on @param annotation "int".
Loading history...
478
    {
479 52
        $this->options[CURLOPT_CONNECTTIMEOUT] = $seconds;
0 ignored issues
show
introduced by
Constant CURLOPT_CONNECTTIMEOUT should not be referenced via a fallback global name, but via a use statement.
Loading history...
480 52
        return $this;
481
    }
482
483
    /**
484
     * The maximum number of seconds (float) allowed to execute cURL functions.
485
     *
486
     * @param float $seconds
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
487
     * @return $this
488
     */
489 43
    public function timeOut($seconds = 10)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::timeOut() does not have parameter type hint for its parameter $seconds but it should be possible to add it based on @param annotation "float".
Loading history...
490
    {
491 43
        return $this->timeOutMs(intval($seconds * 1000));
0 ignored issues
show
introduced by
Function intval() should not be referenced via a fallback global name, but via a use statement.
Loading history...
492
    }
493
494
    /**
495
     * The maximum allowed number of milliseconds to perform cURL functions.
496
     *
497
     * @param int $ms millisecond
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
498
     * @return $this
499
     */
500 43
    protected function timeOutMs($ms = 10000)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::timeOutMs() does not have parameter type hint for its parameter $ms but it should be possible to add it based on @param annotation "int".
Loading history...
501
    {
502 43
        $this->options[CURLOPT_TIMEOUT_MS] = $ms;
0 ignored issues
show
introduced by
Constant CURLOPT_TIMEOUT_MS should not be referenced via a fallback global name, but via a use statement.
Loading history...
503 43
        return $this;
504
    }
505
506
507
    /**
508
     * @param array|mixed $data
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
introduced by
@param annotation of method \ClickHouseDB\Transport\CurlerRequest::parameters_json() does not specify type hint for items of its traversable parameter $data.
Loading history...
509
     * @return $this
510
     * @throws \ClickHouseDB\Exception\TransportException
0 ignored issues
show
introduced by
Class \ClickHouseDB\Exception\TransportException should not be referenced via a fully qualified name, but via a use statement.
Loading history...
511
     */
512 43
    public function parameters_json($data)
0 ignored issues
show
Coding Style introduced by
Method name "CurlerRequest::parameters_json" is not in camel caps format
Loading history...
513
    {
0 ignored issues
show
Coding Style introduced by
Expected 0 blank lines after opening function brace; 1 found
Loading history...
514
515 43
        $this->header("Content-Type", "application/json, text/javascript; charset=utf-8");
0 ignored issues
show
Coding Style Comprehensibility introduced by
The string literal Content-Type does not require double quotes, as per coding-style, please use single quotes.

PHP provides two ways to mark string literals. Either with single quotes 'literal' or with double quotes "literal". The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences.

String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (\') and the backslash (\\). Every other character is displayed as is.

Double quoted string literals may contain other variables or more complex escape sequences.

<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

will print an indented: Single is Value

If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear.

For more information on PHP string literals and available escape sequences see the PHP core documentation.

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

PHP provides two ways to mark string literals. Either with single quotes 'literal' or with double quotes "literal". The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences.

String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (\') and the backslash (\\). Every other character is displayed as is.

Double quoted string literals may contain other variables or more complex escape sequences.

<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

will print an indented: Single is Value

If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear.

For more information on PHP string literals and available escape sequences see the PHP core documentation.

Loading history...
516 43
        $this->header("Accept", "application/json, text/javascript, */*; q=0.01");
0 ignored issues
show
Coding Style Comprehensibility introduced by
The string literal Accept does not require double quotes, as per coding-style, please use single quotes.

PHP provides two ways to mark string literals. Either with single quotes 'literal' or with double quotes "literal". The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences.

String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (\') and the backslash (\\). Every other character is displayed as is.

Double quoted string literals may contain other variables or more complex escape sequences.

<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

will print an indented: Single is Value

If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear.

For more information on PHP string literals and available escape sequences see the PHP core documentation.

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

PHP provides two ways to mark string literals. Either with single quotes 'literal' or with double quotes "literal". The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences.

String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (\') and the backslash (\\). Every other character is displayed as is.

Double quoted string literals may contain other variables or more complex escape sequences.

<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

will print an indented: Single is Value

If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear.

For more information on PHP string literals and available escape sequences see the PHP core documentation.

Loading history...
517
518 43
        if ($data === null) {
519
            $this->parameters = '{}';
520
            return $this;
521
        }
522
523 43
        if (is_string($data)) {
0 ignored issues
show
introduced by
Function is_string() should not be referenced via a fallback global name, but via a use statement.
Loading history...
524 43
            $this->parameters = $data;
525 43
            return $this;
526
        }
527
528
        $this->parameters = json_encode($data);
0 ignored issues
show
introduced by
Function json_encode() should not be referenced via a fallback global name, but via a use statement.
Loading history...
529
530
        if (!$this->parameters && $data) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space(s) after NOT operator; 0 found
Loading history...
531
            throw new \ClickHouseDB\Exception\TransportException('Cant json_encode: ' . strval($data));
0 ignored issues
show
introduced by
Class \ClickHouseDB\Exception\TransportException should not be referenced via a fully qualified name, but via a use statement.
Loading history...
introduced by
Function strval() should not be referenced via a fallback global name, but via a use statement.
Loading history...
532
        }
533
534
        return $this;
535
    }
536
537
    /**
538
     * @return resource
539
     */
540
    public function getResultFileHandle()
541
    {
542
        return $this->resultFileHandle;
543
    }
544
545
    /**
546
     * @return bool
547
     */
548
    public function isResultFile()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::isResultFile() does not have return type hint for its return value but it should be possible to add it based on @return annotation "bool".
Loading history...
549
    {
550
        return ($this->resultFileHandle ? true : false);
0 ignored issues
show
introduced by
Usage of language construct "return" with parentheses is disallowed.
Loading history...
551
    }
552
553
    /**
554
     * @param resource $h resource
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
Coding Style introduced by
Expected 4 spaces after parameter name; 1 found
Loading history...
555
     * @param bool $zlib
0 ignored issues
show
Coding Style introduced by
Expected 5 spaces after parameter type; 1 found
Loading history...
556
     * @return $this
557
     */
558 1
    public function setResultFileHandle($h, $zlib = false)
559
    {
560 1
        $this->resultFileHandle = $h;
561 1
        if ($zlib) {
562
            $params = array('level' => 6, 'window' => 15, 'memory' => 9);
0 ignored issues
show
Coding Style introduced by
Short array syntax must be used to define arrays
Loading history...
563
            stream_filter_append($this->resultFileHandle, 'zlib.deflate', STREAM_FILTER_WRITE, $params);
0 ignored issues
show
introduced by
Function stream_filter_append() should not be referenced via a fallback global name, but via a use statement.
Loading history...
introduced by
Constant STREAM_FILTER_WRITE should not be referenced via a fallback global name, but via a use statement.
Loading history...
564
        }
565 1
        return $this;
566
    }
567
568
    /**
569
     * @return CurlerRequest
570
     */
571
    public function PUT()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::PUT() does not have return type hint for its return value but it should be possible to add it based on @return annotation "CurlerRequest".
Loading history...
572
    {
573
        return $this->execute('PUT');
574
    }
575
576
    /**
577
     * @return CurlerRequest
578
     */
579 43
    public function POST()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::POST() does not have return type hint for its return value but it should be possible to add it based on @return annotation "CurlerRequest".
Loading history...
580
    {
581 43
        return $this->execute('POST');
582
    }
583
584
    /**
585
     * @return CurlerRequest
586
     */
587
    public function OPTIONS()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::OPTIONS() does not have return type hint for its return value but it should be possible to add it based on @return annotation "CurlerRequest".
Loading history...
588
    {
589
        return $this->execute('OPTIONS');
590
    }
591
592
    /**
593
     * @return CurlerRequest
594
     */
595 37
    public function GET()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::GET() does not have return type hint for its return value but it should be possible to add it based on @return annotation "CurlerRequest".
Loading history...
596
    {
597 37
        return $this->execute('GET');
598
    }
599
600
    /**
601
     * The number of seconds that DNS records are stored in memory. By default this parameter is 120 (2 minutes).
602
     *
603
     * @param integer $set
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
introduced by
Expected "int" but found "integer" in @param annotation.
Loading history...
604
     * @return $this
605
     */
606
    public function setDnsCache($set)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::setDnsCache() does not have parameter type hint for its parameter $set but it should be possible to add it based on @param annotation "integer".
Loading history...
607
    {
608
        $this->_dns_cache = $set;
609
        return $this;
610
    }
611
612
    /**
613
     * The number of seconds that DNS records are stored in memory. By default this parameter is 120 (2 minutes).
614
     *
615
     * @return int
616
     */
617 47
    public function getDnsCache()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::getDnsCache() does not have return type hint for its return value but it should be possible to add it based on @return annotation "int".
Loading history...
618
    {
619 47
        return $this->_dns_cache;
620
    }
621
622
    /**
623
     * Sets client certificate
624
     *
625
     * @param string $filePath
626
     */
627
    public function setSslCa($filePath)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::setSslCa() does not have parameter type hint for its parameter $filePath but it should be possible to add it based on @param annotation "string".
Loading history...
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::setSslCa() does not have void return type hint.
Loading history...
628
    {
629
        $this->option(CURLOPT_SSL_VERIFYPEER, true);
0 ignored issues
show
introduced by
Constant CURLOPT_SSL_VERIFYPEER should not be referenced via a fallback global name, but via a use statement.
Loading history...
630
        $this->option(CURLOPT_CAINFO, $filePath);
0 ignored issues
show
introduced by
Constant CURLOPT_CAINFO should not be referenced via a fallback global name, but via a use statement.
Loading history...
631
    }
632
633
    /**
634
     * @param string $method
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
635
     * @return $this
636
     */
637 52
    private function execute($method)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::execute() does not have parameter type hint for its parameter $method but it should be possible to add it based on @param annotation "string".
Loading history...
638
    {
639 52
        $this->method = $method;
640 52
        return $this;
641
    }
642
643
    /**
644
     * @return CurlerResponse
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
645
     * @throws \ClickHouseDB\Exception\TransportException
0 ignored issues
show
introduced by
Class \ClickHouseDB\Exception\TransportException should not be referenced via a fully qualified name, but via a use statement.
Loading history...
646
     */
647 47
    public function response()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::response() does not have return type hint for its return value but it should be possible to add it based on @return annotation "CurlerResponse".
Loading history...
648
    {
649 47
        if (!$this->response) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space(s) after NOT operator; 0 found
Loading history...
650
            throw new \ClickHouseDB\Exception\TransportException('Can`t fetch response - is empty');
0 ignored issues
show
introduced by
Class \ClickHouseDB\Exception\TransportException should not be referenced via a fully qualified name, but via a use statement.
Loading history...
651
        }
652
653 47
        return $this->response;
654
    }
655
656 31
    public function isResponseExists(): bool
0 ignored issues
show
introduced by
There must be exactly 1 whitespace between closing parenthesis and return type colon.
Loading history...
657
    {
658 31
        return $this->response !== null;
659
    }
660
661 47
    public function setResponse(CurlerResponse $response): void
0 ignored issues
show
introduced by
There must be exactly 1 whitespace between closing parenthesis and return type colon.
Loading history...
662
    {
663 47
        $this->response = $response;
664 47
    }
665
666
    /**
667
     * @return mixed
668
     */
669 47
    public function handle()
670
    {
671 47
        $this->prepareRequest();
672 47
        return $this->handle;
673
    }
674
675
    /**
676
     * @param callable $callback
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::setFunctionProgress() has useless @param annotation for parameter $callback.
Loading history...
677
     * @throws \Exception
0 ignored issues
show
introduced by
Class \Exception should not be referenced via a fully qualified name, but via a use statement.
Loading history...
678
     */
679
    public function setFunctionProgress(callable $callback)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::setFunctionProgress() does not have void return type hint.
Loading history...
680
    {
681
        if (!is_callable($callback)) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space(s) after NOT operator; 0 found
Loading history...
introduced by
Function is_callable() should not be referenced via a fallback global name, but via a use statement.
Loading history...
682
            throw new \Exception('setFunctionProgress not is_callable');
0 ignored issues
show
introduced by
Class \Exception should not be referenced via a fully qualified name, but via a use statement.
Loading history...
683
        }
684
685
        $this->option(CURLOPT_NOPROGRESS, false);
0 ignored issues
show
introduced by
Constant CURLOPT_NOPROGRESS should not be referenced via a fallback global name, but via a use statement.
Loading history...
686
        $this->option(CURLOPT_PROGRESSFUNCTION, $callback); // version 5.5.0
0 ignored issues
show
introduced by
Constant CURLOPT_PROGRESSFUNCTION should not be referenced via a fallback global name, but via a use statement.
Loading history...
687
    }
688
689
690
    /**
691
     * @return bool
692
     */
693 47
    private function prepareRequest()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Transport\CurlerRequest::prepareRequest() does not have return type hint for its return value but it should be possible to add it based on @return annotation "bool".
Loading history...
694
    {
695 47
        if (!$this->handle) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space(s) after NOT operator; 0 found
Loading history...
696 47
            $this->handle = curl_init();
0 ignored issues
show
Documentation Bug introduced by
It seems like curl_init() can also be of type false. However, the property $handle is declared as type null|resource. Maybe add an additional type check?

Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a mixed type is assigned to a property that is type hinted more strictly.

For example, imagine you have a variable $accountId that can either hold an Id object or false (if there is no account id yet). Your code now assigns that value to the id property of an instance of the Account class. This class holds a proper account, so the id value must no longer be false.

Either this assignment is in error or a type check should be added for that assignment.

class Id
{
    public $id;

    public function __construct($id)
    {
        $this->id = $id;
    }

}

class Account
{
    /** @var  Id $id */
    public $id;
}

$account_id = false;

if (starsAreRight()) {
    $account_id = new Id(42);
}

$account = new Account();
if ($account instanceof Id)
{
    $account->id = $account_id;
}
Loading history...
introduced by
Function curl_init() should not be referenced via a fallback global name, but via a use statement.
Loading history...
697
        }
698
699 47
        $curl_opt = $this->options;
700 47
        $method = $this->method;
0 ignored issues
show
Coding Style introduced by
Equals sign not aligned with surrounding assignments; expected 3 spaces but found 1 space

This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line.

To visualize

$a = "a";
$ab = "ab";
$abc = "abc";

will produce issues in the first and second line, while this second example

$a   = "a";
$ab  = "ab";
$abc = "abc";

will produce no issues.

Loading history...
701
702 47
        if ($this->_attachFiles) {
703 1
            $curl_opt[CURLOPT_SAFE_UPLOAD] = true;
0 ignored issues
show
introduced by
Constant CURLOPT_SAFE_UPLOAD should not be referenced via a fallback global name, but via a use statement.
Loading history...
704
        }
705
0 ignored issues
show
Coding Style introduced by
Functions must not contain multiple empty lines in a row; found 2 empty lines
Loading history...
706
707 47
        if (strtoupper($method) == 'GET') {
0 ignored issues
show
introduced by
Function strtoupper() should not be referenced via a fallback global name, but via a use statement.
Loading history...
introduced by
Operator == is disallowed, use === instead.
Loading history...
708 37
            $curl_opt[CURLOPT_HTTPGET] = true;
0 ignored issues
show
Coding Style introduced by
Equals sign not aligned with surrounding assignments; expected 7 spaces but found 1 space

This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line.

To visualize

$a = "a";
$ab = "ab";
$abc = "abc";

will produce issues in the first and second line, while this second example

$a   = "a";
$ab  = "ab";
$abc = "abc";

will produce no issues.

Loading history...
709 37
            $curl_opt[CURLOPT_CUSTOMREQUEST] = strtoupper($method);
0 ignored issues
show
introduced by
Constant CURLOPT_CUSTOMREQUEST should not be referenced via a fallback global name, but via a use statement.
Loading history...
introduced by
Function strtoupper() should not be referenced via a fallback global name, but via a use statement.
Loading history...
710 37
            $curl_opt[CURLOPT_POSTFIELDS] = false;
0 ignored issues
show
introduced by
Constant CURLOPT_POSTFIELDS should not be referenced via a fallback global name, but via a use statement.
Loading history...
Coding Style introduced by
Equals sign not aligned with surrounding assignments; expected 4 spaces but found 1 space

This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line.

To visualize

$a = "a";
$ab = "ab";
$abc = "abc";

will produce issues in the first and second line, while this second example

$a   = "a";
$ab  = "ab";
$abc = "abc";

will produce no issues.

Loading history...
711
        } else {
712 38
            if (strtoupper($method) === 'POST') {
0 ignored issues
show
introduced by
Function strtoupper() should not be referenced via a fallback global name, but via a use statement.
Loading history...
713 38
                $curl_opt[CURLOPT_POST] = true;
714
            }
715
716 38
            $curl_opt[CURLOPT_CUSTOMREQUEST] = strtoupper($method);
0 ignored issues
show
introduced by
Constant CURLOPT_CUSTOMREQUEST should not be referenced via a fallback global name, but via a use statement.
Loading history...
introduced by
Function strtoupper() should not be referenced via a fallback global name, but via a use statement.
Loading history...
717
718 38
            if ($this->parameters) {
719 38
                $curl_opt[CURLOPT_POSTFIELDS] = $this->parameters;
0 ignored issues
show
introduced by
Constant CURLOPT_POSTFIELDS should not be referenced via a fallback global name, but via a use statement.
Loading history...
720
721 38
                if (!is_array($this->parameters)) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space(s) after NOT operator; 0 found
Loading history...
introduced by
Function is_array() should not be referenced via a fallback global name, but via a use statement.
Loading history...
722 38
                    $this->header('Content-Length', strlen($this->parameters));
0 ignored issues
show
introduced by
Function strlen() should not be referenced via a fallback global name, but via a use statement.
Loading history...
723
                }
724
            }
725
        }
726
        // CURLOPT_DNS_CACHE_TIMEOUT - Количество секунд, в течение которых в памяти хранятся DNS-записи.
727 47
        $curl_opt[CURLOPT_DNS_CACHE_TIMEOUT] = $this->getDnsCache();
0 ignored issues
show
introduced by
Constant CURLOPT_DNS_CACHE_TIMEOUT should not be referenced via a fallback global name, but via a use statement.
Loading history...
728 47
        $curl_opt[CURLOPT_URL] = $this->url;
0 ignored issues
show
introduced by
Constant CURLOPT_URL should not be referenced via a fallback global name, but via a use statement.
Loading history...
Coding Style introduced by
Equals sign not aligned with surrounding assignments; expected 15 spaces but found 1 space

This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line.

To visualize

$a = "a";
$ab = "ab";
$abc = "abc";

will produce issues in the first and second line, while this second example

$a   = "a";
$ab  = "ab";
$abc = "abc";

will produce no issues.

Loading history...
729
730 47
        if (!empty($this->headers) && sizeof($this->headers)) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space(s) after NOT operator; 0 found
Loading history...
introduced by
Function sizeof() should not be referenced via a fallback global name, but via a use statement.
Loading history...
Coding Style introduced by
The use of function sizeof() is forbidden; use count() instead
Loading history...
731 47
            $curl_opt[CURLOPT_HTTPHEADER] = array();
0 ignored issues
show
introduced by
Constant CURLOPT_HTTPHEADER should not be referenced via a fallback global name, but via a use statement.
Loading history...
Coding Style introduced by
Short array syntax must be used to define arrays
Loading history...
732
733 47
            foreach ($this->headers as $key => $value) {
734 47
                $curl_opt[CURLOPT_HTTPHEADER][] = sprintf("%s: %s", $key, $value);
0 ignored issues
show
introduced by
Constant CURLOPT_HTTPHEADER should not be referenced via a fallback global name, but via a use statement.
Loading history...
introduced by
Function sprintf() should not be referenced via a fallback global name, but via a use statement.
Loading history...
Coding Style Comprehensibility introduced by
The string literal %s: %s does not require double quotes, as per coding-style, please use single quotes.

PHP provides two ways to mark string literals. Either with single quotes 'literal' or with double quotes "literal". The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences.

String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (\') and the backslash (\\). Every other character is displayed as is.

Double quoted string literals may contain other variables or more complex escape sequences.

<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

will print an indented: Single is Value

If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear.

For more information on PHP string literals and available escape sequences see the PHP core documentation.

Loading history...
735
            }
736
        }
737
738 47
        if (!empty($curl_opt[CURLOPT_INFILE])) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space(s) after NOT operator; 0 found
Loading history...
introduced by
Constant CURLOPT_INFILE should not be referenced via a fallback global name, but via a use statement.
Loading history...
Coding Style introduced by
Blank line found at start of control structure
Loading history...
739
740 8
            $curl_opt[CURLOPT_PUT] = true;
0 ignored issues
show
introduced by
Constant CURLOPT_PUT should not be referenced via a fallback global name, but via a use statement.
Loading history...
741
        }
742
743 47
        if (!empty($curl_opt[CURLOPT_WRITEFUNCTION])) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space(s) after NOT operator; 0 found
Loading history...
introduced by
Constant CURLOPT_WRITEFUNCTION should not be referenced via a fallback global name, but via a use statement.
Loading history...
744 1
            $curl_opt[CURLOPT_HEADER] = false;
0 ignored issues
show
introduced by
Constant CURLOPT_HEADER should not be referenced via a fallback global name, but via a use statement.
Loading history...
745
        }
746
747 47
        if ($this->resultFileHandle) {
748 1
            $curl_opt[CURLOPT_FILE] = $this->resultFileHandle;
0 ignored issues
show
introduced by
Constant CURLOPT_FILE should not be referenced via a fallback global name, but via a use statement.
Loading history...
Coding Style introduced by
Equals sign not aligned with surrounding assignments; expected 3 spaces but found 1 space

This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line.

To visualize

$a = "a";
$ab = "ab";
$abc = "abc";

will produce issues in the first and second line, while this second example

$a   = "a";
$ab  = "ab";
$abc = "abc";

will produce no issues.

Loading history...
749 1
            $curl_opt[CURLOPT_HEADER] = false;
0 ignored issues
show
introduced by
Constant CURLOPT_HEADER should not be referenced via a fallback global name, but via a use statement.
Loading history...
750
        }
751
752 47
        if ($this->options[CURLOPT_VERBOSE]) {
0 ignored issues
show
introduced by
Constant CURLOPT_VERBOSE should not be referenced via a fallback global name, but via a use statement.
Loading history...
753
            echo "\n-----------BODY REQUEST----------\n" . $curl_opt[CURLOPT_POSTFIELDS] . "\n------END--------\n";
0 ignored issues
show
introduced by
Constant CURLOPT_POSTFIELDS should not be referenced via a fallback global name, but via a use statement.
Loading history...
754
        }
755 47
        curl_setopt_array($this->handle, $curl_opt);
0 ignored issues
show
Bug introduced by
It seems like $this->handle can also be of type false; however, parameter $ch of curl_setopt_array() does only seem to accept resource, maybe add an additional type check? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

755
        curl_setopt_array(/** @scrutinizer ignore-type */ $this->handle, $curl_opt);
Loading history...
introduced by
Function curl_setopt_array() should not be referenced via a fallback global name, but via a use statement.
Loading history...
756 47
        return true;
757
    }
758
}
759