Failed Conditions
Pull Request — master (#222)
by
unknown
06:27
created

Statement::current()   A

Complexity

Conditions 2
Paths 2

Size

Total Lines 6
Code Lines 4

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 0
CRAP Score 6

Importance

Changes 1
Bugs 0 Features 0
Metric Value
cc 2
eloc 4
c 1
b 0
f 0
nc 2
nop 0
dl 0
loc 6
ccs 0
cts 0
cp 0
crap 6
rs 10
1
<?php
0 ignored issues
show
introduced by
Missing declare(strict_types=1).
Loading history...
2
3
namespace ClickHouseDB;
4
5
use ClickHouseDB\Exception\ClickHouseUnavailableException;
6
use ClickHouseDB\Exception\DatabaseException;
7
use ClickHouseDB\Exception\QueryException;
8
use ClickHouseDB\Query\Query;
9
use ClickHouseDB\Transport\CurlerRequest;
10
use ClickHouseDB\Transport\CurlerResponse;
11
12
class Statement implements \Iterator
0 ignored issues
show
introduced by
Class \Iterator should not be referenced via a fully qualified name, but via a use statement.
Loading history...
13
{
14
    private const CLICKHOUSE_ERROR_REGEX = "%Code:\s(\d+)\.\s*DB::Exception\s*:\s*(.*)(?:,\s*e\.what|\(version).*%ius";
0 ignored issues
show
Coding Style Comprehensibility introduced by
The string literal %Code:\s(\d+)\.\s*DB::Ex...\.what|\(version).*%ius 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...
15
16
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Statement::$_rawData with single line content, use one-line comment instead.
Loading history...
17
     * @var string|mixed
18
     */
19
    private $_rawData;
20
21
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Statement::$_http_code with single line content, use one-line comment instead.
Loading history...
22
     * @var int
23
     */
24
    private $_http_code = -1;
0 ignored issues
show
Coding Style introduced by
Member variable "_http_code" is not in valid camel caps format
Loading history...
25
26
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Statement::$_request with single line content, use one-line comment instead.
Loading history...
27
     * @var CurlerRequest
28
     */
29
    private $_request = null;
30
31
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Statement::$_init with single line content, use one-line comment instead.
Loading history...
32
     * @var bool
33
     */
34
    private $_init = false;
35
36
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Statement::$query with single line content, use one-line comment instead.
Loading history...
37
     * @var Query
38
     */
39
    private $query;
40
41
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Statement::$format with single line content, use one-line comment instead.
Loading history...
42
     * @var mixed
43
     */
44
    private $format;
45
46
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Statement::$sql with single line content, use one-line comment instead.
Loading history...
47
     * @var string
48
     */
49
    private $sql = '';
50
51
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Statement::$meta with single line content, use one-line comment instead.
Loading history...
52
     * @var array
0 ignored issues
show
introduced by
@var annotation of property \ClickHouseDB\Statement::$meta does not specify type hint for its items.
Loading history...
53
     */
54
    private $meta;
55
56
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Statement::$totals with single line content, use one-line comment instead.
Loading history...
57
     * @var array
0 ignored issues
show
introduced by
@var annotation of property \ClickHouseDB\Statement::$totals does not specify type hint for its items.
Loading history...
58
     */
59
    private $totals;
60
61
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Statement::$extremes with single line content, use one-line comment instead.
Loading history...
62
     * @var array
0 ignored issues
show
introduced by
@var annotation of property \ClickHouseDB\Statement::$extremes does not specify type hint for its items.
Loading history...
63
     */
64
    private $extremes;
65
66
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Statement::$rows with single line content, use one-line comment instead.
Loading history...
67
     * @var int
68
     */
69
    private $rows;
70
71
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Statement::$rows_before_limit_at_least with single line content, use one-line comment instead.
Loading history...
72
     * @var bool|integer
0 ignored issues
show
introduced by
Expected "int" but found "integer" in @var annotation.
Loading history...
73
     */
74
    private $rows_before_limit_at_least = false;
0 ignored issues
show
Coding Style introduced by
Member variable "rows_before_limit_at_least" is not in valid camel caps format
Loading history...
75
76
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Statement::$array_data with single line content, use one-line comment instead.
Loading history...
77
     * @var array
0 ignored issues
show
introduced by
@var annotation of property \ClickHouseDB\Statement::$array_data does not specify type hint for its items.
Loading history...
78
     */
79
    private $array_data = [];
0 ignored issues
show
Coding Style introduced by
Member variable "array_data" is not in valid camel caps format
Loading history...
80
81
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Statement::$statistics with single line content, use one-line comment instead.
Loading history...
82
     * @var array|null
0 ignored issues
show
introduced by
@var annotation of property \ClickHouseDB\Statement::$statistics does not specify type hint for its items.
Loading history...
83
     */
84
    private $statistics = null;
85
86
    /**
0 ignored issues
show
introduced by
Found multi-line comment for property \ClickHouseDB\Statement::$iterator with single line content, use one-line comment instead.
Loading history...
87
     * @var int
88
     */
89
    public $iterator = 0;
90 45
91
92 45
    public function __construct(CurlerRequest $request)
0 ignored issues
show
Coding Style introduced by
Expected 1 blank line before function; 2 found
Loading history...
93 45
    {
94 45
        $this->_request = $request;
95 45
        $this->format = $this->_request->getRequestExtendedInfo('format');
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...
96 45
        $this->query = $this->_request->getRequestExtendedInfo('query');
0 ignored issues
show
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...
97
        $this->sql = $this->_request->getRequestExtendedInfo('sql');
0 ignored issues
show
Coding Style introduced by
Equals sign not aligned with surrounding assignments; expected 6 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...
98
    }
99
100
    /**
101
     * @return CurlerRequest
102
     */
103
    public function getRequest()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Statement::getRequest() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "CurlerRequest".
Loading history...
104
    {
105
        return $this->_request;
106
    }
107
108
    /**
109
     * @return CurlerResponse
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
110 39
     * @throws Exception\TransportException
111
     */
112 39
    private function response()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Statement::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".
Loading history...
113
    {
114
        return $this->_request->response();
115
    }
116
117
    /**
118
     * @return mixed
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
119 1
     * @throws Exception\TransportException
120
     */
121 1
    public function responseInfo()
122
    {
123
        return $this->response()->info();
124
    }
125
126
    /**
127 10
     * @return mixed|string
128
     */
129 10
    public function sql()
130
    {
131
        return $this->sql;
132
    }
133
134
    /**
135
     * @param string $body
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
introduced by
Method \ClickHouseDB\Statement::parseErrorClickHouse() has useless @param annotation for parameter $body.
Loading history...
136 5
     * @return array|bool
0 ignored issues
show
introduced by
@return annotation of method \ClickHouseDB\Statement::parseErrorClickHouse() does not specify type hint for items of its traversable return value.
Loading history...
137
     */
138 5
    private function parseErrorClickHouse(string $body)
139 5
    {
140
        $body = trim($body);
0 ignored issues
show
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...
introduced by
Function trim() should not be referenced via a fallback global name, but via a use statement.
Loading history...
141
        $matches = [];
142
143
        // Code: 115. DB::Exception: Unknown setting readonly[0], e.what() = DB::Exception
144
        // Code: 192. DB::Exception: Unknown user x, e.what() = DB::Exception
145 5
        // Code: 60. DB::Exception: Table default.ZZZZZ doesn't exist., e.what() = DB::Exception
146 4
        // Code: 516. DB::Exception: test_username: Authentication failed: password is incorrect or there is no user with such name. (AUTHENTICATION_FAILED) (version 22.8.3.13 (official build))
147
148
        if (preg_match(self::CLICKHOUSE_ERROR_REGEX, $body, $matches)) {
0 ignored issues
show
introduced by
Function preg_match() should not be referenced via a fallback global name, but via a use statement.
Loading history...
149 1
            return ['code' => $matches[1], 'message' => $matches[2]];
150
        }
151
152
        return false;
153
    }
154
155
    private function hasErrorClickhouse(string $body): bool {
0 ignored issues
show
Coding Style introduced by
Expected 0 blank lines after opening function brace; 1 found
Loading history...
156 8
157
        return preg_match(self::CLICKHOUSE_ERROR_REGEX, $body) === 1;
0 ignored issues
show
introduced by
Expected 0 lines before "return", found 1.
Loading history...
introduced by
Function preg_match() should not be referenced via a fallback global name, but via a use statement.
Loading history...
158 8
    }
159 2
160
    /**
161
     * @return bool
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
162 6
     * @throws Exception\TransportException
163 6
     */
164 6
    public function error()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Statement::error() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "bool".
Loading history...
165
    {
166 6
        if (!$this->isError()) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space after NOT operator; 0 found
Loading history...
167 6
            return false;
168 5
        }
169
170 5
        $body = $this->response()->body();
0 ignored issues
show
Coding Style introduced by
Equals sign not aligned with surrounding assignments; expected 5 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...
171 4
        $error_no = $this->response()->error_no();
0 ignored issues
show
Coding Style introduced by
The variable $error_no should be in camel caps format.
Loading history...
172
        $error = $this->response()->error();
0 ignored issues
show
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...
173 1
174 1
        $dumpStatement = false;
175 1
        if (!$error_no && !$error) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space after NOT operator; 0 found
Loading history...
Coding Style introduced by
The variable $error_no should be in camel caps format.
Loading history...
176
            $parse = $this->parseErrorClickHouse($body);
177
178 1
            if ($parse) {
0 ignored issues
show
Bug Best Practice introduced by
The expression $parse of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent.

Consider making the comparison explicit by using empty(..) or ! empty(...) instead.

Loading history...
introduced by
$parse is a non-empty array, thus is always true.
Loading history...
179 1
                throw new DatabaseException($parse['message'] . "\nIN:" . $this->sql(), $parse['code']);
180
            } else {
0 ignored issues
show
introduced by
Remove useless "else" to reduce code nesting.
Loading history...
181
                $code = $this->response()->http_code();
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...
182 2
                $message = "HttpCode:" . $this->response()->http_code() . " ; " . $this->response()->error() . " ;" . $body;
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...
Coding Style Comprehensibility introduced by
The string literal HttpCode: 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 ; 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 ; 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...
183 2
                $dumpStatement = true;
184
            }
185
        } else {
186
            $code = $error_no;
0 ignored issues
show
Coding Style introduced by
The variable $error_no should be in camel caps format.
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...
187 2
            $message = $this->response()->error();
188 1
        }
189 1
190
        $exception = new QueryException($message, $code);
191
        if ($code === CURLE_COULDNT_CONNECT) {
0 ignored issues
show
introduced by
Constant CURLE_COULDNT_CONNECT should not be referenced via a fallback global name, but via a use statement.
Loading history...
192 2
            $exception = new ClickHouseUnavailableException($message, $code);
193
        }
194
195
        if ($dumpStatement) {
0 ignored issues
show
introduced by
The condition $dumpStatement is always false.
Loading history...
196
            $exception->setRequestDetails($this->_request->getDetails());
197
            $exception->setResponseDetails($this->response()->getDetails());
198
        }
199 39
200
        throw $exception;
201 39
    }
202
203
    /**
204 32
     * @return bool
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
introduced by
Method \ClickHouseDB\Statement::isError() has useless @return annotation.
Loading history...
205
     * @throws Exception\TransportException
206 32
     */
207
    public function isError(): bool
208
    {
209
        if ($this->response()->http_code() !== 200) {
210 32
            return true;
211 1
        }
212
213
        if ($this->response()->error_no()) {
214 31
            return true;
215
        }
216
217
        if ($this->hasErrorClickhouse($this->response()->body())) {
0 ignored issues
show
introduced by
Useless condition.
Loading history...
218
            return true;
219
        }
220
221 30
        return false;
222
    }
223 30
224
    private function check(): bool
225
    {
226
        if (!$this->_request->isResponseExists()) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space after NOT operator; 0 found
Loading history...
227
            throw QueryException::noResponse();
228 30
        }
229
230
        if ($this->isError()) {
231 29
            $this->error();
232
        }
233 29
234
        return true;
235
    }
236
237 29
    /**
238 29
     * @return bool
239
     */
240 29
    public function isInited()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Statement::isInited() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "bool".
Loading history...
241 29
    {
242
        return $this->_init;
243 29
    }
244
245
    /**
246 29
     * @return bool
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
247
     * @throws Exception\TransportException
248
     */
249
    private function init()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Statement::init() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "bool".
Loading history...
250
    {
251
        if ($this->_init) {
252 29
            return false;
253
        }
254
255
        $this->check();
256 29
257 29
        $this->_rawData = $this->response()->rawDataOrJson($this->format);
258 29
259 29
        if (!$this->_rawData) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space after NOT operator; 0 found
Loading history...
260
            $this->_init = true;
261
            return false;
0 ignored issues
show
introduced by
Expected 1 lines before "return", found 0.
Loading history...
262 29
        }
263
264 1
        $data = [];
265
        foreach (['meta', 'data', 'totals', 'extremes', 'rows', 'rows_before_limit_at_least', 'statistics'] as $key) {
0 ignored issues
show
Coding Style introduced by
Blank line found at start of control structure
Loading history...
266
267 28
            if (isset($this->_rawData[$key])) {
0 ignored issues
show
introduced by
Use early exit to reduce code nesting.
Loading history...
268 28
                if ($key=='data') {
0 ignored issues
show
Coding Style introduced by
Expected at least 1 space before "=="; 0 found
Loading history...
Coding Style introduced by
Expected at least 1 space after "=="; 0 found
Loading history...
introduced by
Operator == is disallowed, use === instead.
Loading history...
269
                    $data=$this->_rawData[$key];
0 ignored issues
show
Coding Style introduced by
Expected at least 1 space before "="; 0 found
Loading history...
Coding Style introduced by
Expected at least 1 space after "="; 0 found
Loading history...
Coding Style introduced by
Equals sign not aligned correctly; expected 1 space but found 0 spaces
Loading history...
270
                } else {
271
                    $this->{$key} = $this->_rawData[$key];
272 29
                }
273
            }
274
        }
275
276 29
        if (empty($this->meta)) {
277
            throw new QueryException('Can`t find meta');
278
        }
279
280
        $isJSONCompact=(stripos($this->format,'JSONCompact')!==false?true:false);
0 ignored issues
show
Coding Style introduced by
Expected at least 1 space before "="; 0 found
Loading history...
Coding Style introduced by
Expected at least 1 space after "="; 0 found
Loading history...
Coding Style introduced by
Equals sign not aligned with surrounding assignments; expected 4 spaces but found 0 spaces

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...
introduced by
Function stripos() should not be referenced via a fallback global name, but via a use statement.
Loading history...
Coding Style introduced by
Expected at least 1 space before "!=="; 0 found
Loading history...
Coding Style introduced by
Expected at least 1 space after "!=="; 0 found
Loading history...
Coding Style introduced by
Expected at least 1 space before "?"; 0 found
Loading history...
Coding Style introduced by
Expected at least 1 space after "?"; 0 found
Loading history...
introduced by
Useless ternary operator.
Loading history...
Coding Style introduced by
Expected at least 1 space before ":"; 0 found
Loading history...
Coding Style introduced by
Expected at least 1 space after ":"; 0 found
Loading history...
281
        $this->array_data = [];
0 ignored issues
show
Coding Style introduced by
Member variable "array_data" is not in valid camel caps format
Loading history...
282
        foreach ($data as $rows) {
283
            $r = [];
284
285
            if ($isJSONCompact) {
286
                $r[] = $rows;
287
            } else {
288
                foreach ($this->meta as $meta) {
289
                    $r[$meta['name']] = $meta['type'] === 'Int64'
290
                        ? (int) $rows[$meta['name']]
291
                        : $rows[$meta['name']];
292
                }
293 1
            }
294
295 1
            $this->array_data[] = $r;
0 ignored issues
show
Coding Style introduced by
Member variable "array_data" is not in valid camel caps format
Loading history...
296 1
        }
297
298
        $this->_init = true;
299
300
        return true;
301
    }
302
303
    /**
304 1
     * @return array
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
introduced by
@return annotation of method \ClickHouseDB\Statement::extremes() does not specify type hint for items of its traversable return value.
Loading history...
305
     * @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...
306 1
     */
307
    public function extremes()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Statement::extremes() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "array".
Loading history...
308 1
    {
309
        $this->init();
310
        return $this->extremes;
0 ignored issues
show
introduced by
Expected 1 lines before "return", found 0.
Loading history...
311
    }
312 1
313
    /**
314
     * @return mixed
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
315
     * @throws Exception\TransportException
316
     */
317
    public function totalTimeRequest()
318
    {
319
        $this->check();
320
        return $this->response()->total_time();
0 ignored issues
show
introduced by
Expected 1 lines before "return", found 0.
Loading history...
introduced by
Expected 0 lines after "return", found 1.
Loading history...
321
322
    }
0 ignored issues
show
Coding Style introduced by
Function closing brace must go on the next line following the body; found 1 blank lines before brace
Loading history...
323
324
    /**
325
     * @return array
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
introduced by
@return annotation of method \ClickHouseDB\Statement::extremesMin() does not specify type hint for items of its traversable return value.
Loading history...
326
     * @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...
327
     */
328
    public function extremesMin()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Statement::extremesMin() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "array".
Loading history...
329
    {
330
        $this->init();
331
332
        if (empty($this->extremes['min'])) {
333
            return [];
334 1
        }
335
336 1
        return $this->extremes['min'];
337 1
    }
338
339
    /**
340
     * @return array
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
introduced by
@return annotation of method \ClickHouseDB\Statement::extremesMax() does not specify type hint for items of its traversable return value.
Loading history...
341
     * @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...
342
     */
343
    public function extremesMax()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Statement::extremesMax() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "array".
Loading history...
344
    {
345
        $this->init();
346
347
        if (empty($this->extremes['max'])) {
348
            return [];
349
        }
350
351
        return $this->extremes['max'];
352
    }
353
354
    /**
355
     * @return array
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
introduced by
@return annotation of method \ClickHouseDB\Statement::totals() does not specify type hint for items of its traversable return value.
Loading history...
356
     * @throws Exception\TransportException
357
     */
358
    public function totals()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Statement::totals() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "array".
Loading history...
359
    {
360
        $this->init();
361 2
        return $this->totals;
0 ignored issues
show
introduced by
Expected 1 lines before "return", found 0.
Loading history...
362
    }
363 2
364 2
    /**
0 ignored issues
show
introduced by
Empty comment
Loading history...
365
     *
366
     */
367
    public function dump()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Statement::dump() does not have void return type hint.
Loading history...
introduced by
Method \ClickHouseDB\Statement::dump() does not need documentation comment.
Loading history...
368
    {
369
        $this->_request->dump();
370
        $this->response()->dump();
371
    }
372
373
    /**
374
     * @return bool|int
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
375
     * @throws Exception\TransportException
376
     */
377
    public function countAll()
378
    {
379
        $this->init();
380
        return $this->rows_before_limit_at_least;
0 ignored issues
show
introduced by
Expected 1 lines before "return", found 0.
Loading history...
Coding Style introduced by
Member variable "rows_before_limit_at_least" is not in valid camel caps format
Loading history...
381
    }
382
383
    /**
384
     * @param bool $key
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
385
     * @return array|mixed|null
0 ignored issues
show
introduced by
@return annotation of method \ClickHouseDB\Statement::statistics() does not specify type hint for items of its traversable return value.
Loading history...
386
     * @throws Exception\TransportException
387
     */
388
    public function statistics($key = false)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Statement::statistics() does not have native type hint for its parameter $key but it should be possible to add it based on @param annotation "bool".
Loading history...
389
    {
390
        $this->init();
391
392
        if (!is_array($this->statistics)) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space 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...
393
            return null;
394 8
        }
395
396 8
        if (!$key) return $this->statistics;
0 ignored issues
show
Coding Style introduced by
Expected 1 space after NOT operator; 0 found
Loading history...
Coding Style Best Practice introduced by
It is generally a best practice to always use braces with control structures.

Adding braces to control structures avoids accidental mistakes as your code changes:

// Without braces (not recommended)
if (true)
    doSomething();

// Recommended
if (true) {
    doSomething();
}
Loading history...
introduced by
Expected 1 lines before "return", found -1.
Loading history...
397 8
398
        if (!isset($this->statistics[$key])) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space after NOT operator; 0 found
Loading history...
399
            return null;
400
        }
401
402
        return $this->statistics[$key];
0 ignored issues
show
introduced by
Expected 0 lines after "return", found 1.
Loading history...
403
404 3
    }
0 ignored issues
show
Coding Style introduced by
Function closing brace must go on the next line following the body; found 1 blank lines before brace
Loading history...
405
406 3
    /**
407
     * @return int
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
408
     * @throws Exception\TransportException
409
     */
410 3
    public function count()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Statement::count() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "int".
Loading history...
411
    {
412 3
        $this->init();
413
        return $this->rows;
0 ignored issues
show
introduced by
Expected 1 lines before "return", found 0.
Loading history...
414
    }
415
416
    /**
417
     * @return mixed|string
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
418 2
     * @throws Exception\TransportException
419
     */
420 2
    public function rawData()
421 2
    {
422
        if ($this->_init) {
423 2
            return $this->_rawData;
424
        }
425 2
426
        $this->check();
427 2
428
        return $this->response()->rawDataOrJson($this->format);
429 2
    }
430
431
    /**
0 ignored issues
show
introduced by
Empty comment
Loading history...
432
     *
433 2
     */
434
    public function resetIterator()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Statement::resetIterator() does not have void return type hint.
Loading history...
introduced by
Method \ClickHouseDB\Statement::resetIterator() does not need documentation comment.
Loading history...
435 2
    {
436
        $this->iterator=0;
0 ignored issues
show
Coding Style introduced by
Expected at least 1 space before "="; 0 found
Loading history...
Coding Style introduced by
Expected at least 1 space after "="; 0 found
Loading history...
Coding Style introduced by
Equals sign not aligned correctly; expected 1 space but found 0 spaces
Loading history...
437
    }
438 2
439 1
    public function fetchRow($key = null)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Statement::fetchRow() does not have parameter type hint nor @param annotation for its parameter $key.
Loading history...
introduced by
Method \ClickHouseDB\Statement::fetchRow() does not have return type hint nor @return annotation for its return value.
Loading history...
440
    {
441
        $this->init();
442 2
443
        $position=$this->iterator;
0 ignored issues
show
Coding Style introduced by
Expected at least 1 space before "="; 0 found
Loading history...
Coding Style introduced by
Expected at least 1 space after "="; 0 found
Loading history...
Coding Style introduced by
Equals sign not aligned correctly; expected 1 space but found 0 spaces
Loading history...
444
445
        if (!isset($this->array_data[$position])) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space after NOT operator; 0 found
Loading history...
Coding Style introduced by
Member variable "array_data" is not in valid camel caps format
Loading history...
446
            return null;
447
        }
448
449
        $this->iterator++;
450 18
451
        if (!$key) {
0 ignored issues
show
introduced by
Expected 1 lines after "if", found 0.
Loading history...
Coding Style introduced by
Expected 1 space after NOT operator; 0 found
Loading history...
452 18
            return $this->array_data[$position];
0 ignored issues
show
Coding Style introduced by
Member variable "array_data" is not in valid camel caps format
Loading history...
453 18
        }
454
        if (!isset($this->array_data[$position][$key])) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space after NOT operator; 0 found
Loading history...
Coding Style introduced by
Member variable "array_data" is not in valid camel caps format
Loading history...
455
            return null;
456
        }
457 18
458 3
        return $this->array_data[$position][$key];
0 ignored issues
show
introduced by
Expected 0 lines after "return", found 1.
Loading history...
Coding Style introduced by
Member variable "array_data" is not in valid camel caps format
Loading history...
459
460
    }
0 ignored issues
show
Coding Style introduced by
Function closing brace must go on the next line following the body; found 1 blank lines before brace
Loading history...
Coding Style introduced by
Expected 1 blank line after function; 0 found
Loading history...
461 18
    /**
462 2
     * @param string $key
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
463
     * @return mixed|null
464
     * @throws Exception\TransportException
465 17
     */
466
    public function fetchOne($key = null)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Statement::fetchOne() does not have native type hint for its parameter $key but it should be possible to add it based on @param annotation "string".
Loading history...
467
    {
468
        $this->init();
469
        if (!isset($this->array_data[0])) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space after NOT operator; 0 found
Loading history...
Coding Style introduced by
Member variable "array_data" is not in valid camel caps format
Loading history...
470
            return null;
471
        }
472
473 4
        if (!$key) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space after NOT operator; 0 found
Loading history...
474
            return $this->array_data[0];
0 ignored issues
show
Coding Style introduced by
Member variable "array_data" is not in valid camel caps format
Loading history...
475 4
        }
476
477 4
        if (!isset($this->array_data[0][$key])) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space after NOT operator; 0 found
Loading history...
Coding Style introduced by
Member variable "array_data" is not in valid camel caps format
Loading history...
478 4
            return null;
479 4
        }
480 4
481
        return $this->array_data[0][$key];
0 ignored issues
show
Coding Style introduced by
Member variable "array_data" is not in valid camel caps format
Loading history...
482
    }
483 4
484
    /**
485
     * @param string|null $path
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
486
     * @return array
0 ignored issues
show
introduced by
@return annotation of method \ClickHouseDB\Statement::rowsAsTree() does not specify type hint for items of its traversable return value.
Loading history...
487
     * @throws Exception\TransportException
488
     */
489
    public function rowsAsTree($path)
0 ignored issues
show
introduced by
Method \ClickHouseDB\Statement::rowsAsTree() does not have native type hint for its parameter $path but it should be possible to add it based on @param annotation "string|null".
Loading history...
introduced by
Method \ClickHouseDB\Statement::rowsAsTree() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "array".
Loading history...
490
    {
491
        $this->init();
492
493
        $out = [];
494
        foreach ($this->array_data as $row) {
0 ignored issues
show
Coding Style introduced by
Member variable "array_data" is not in valid camel caps format
Loading history...
495
            $d = $this->array_to_tree($row, $path);
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...
496
            $out = array_replace_recursive($d, $out);
0 ignored issues
show
introduced by
Function array_replace_recursive() should not be referenced via a fallback global name, but via a use statement.
Loading history...
497
        }
498
499
        return $out;
500
    }
501
502
    /**
503
     * Return size_upload,upload_content,speed_upload,time_request
504
     *
505
     * @return array
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
introduced by
@return annotation of method \ClickHouseDB\Statement::info_upload() does not specify type hint for items of its traversable return value.
Loading history...
506
     * @throws Exception\TransportException
507
     */
508
    public function info_upload()
0 ignored issues
show
Coding Style introduced by
Method name "Statement::info_upload" is not in camel caps format
Loading history...
introduced by
Method \ClickHouseDB\Statement::info_upload() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "array".
Loading history...
509 1
    {
510
        $this->check();
511 1
        return [
0 ignored issues
show
introduced by
Expected 1 lines before "return", found 0.
Loading history...
512
            'size_upload'    => $this->response()->size_upload(),
513 1
            'upload_content' => $this->response()->upload_content_length(),
514 1
            'speed_upload'   => $this->response()->speed_upload(),
515 1
            'time_request'   => $this->response()->total_time()
0 ignored issues
show
introduced by
Multi-line arrays must have a trailing comma after the last element.
Loading history...
516 1
        ];
517 1
    }
518 1
519 1
    /**
520
     * Return size_upload,upload_content,speed_upload,time_request,starttransfer_time,size_download,speed_download
521
     *
522
     * @return array
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
introduced by
@return annotation of method \ClickHouseDB\Statement::info() does not specify type hint for items of its traversable return value.
Loading history...
523
     * @throws Exception\TransportException
524
     */
525
    public function info()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Statement::info() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "array".
Loading history...
526
    {
527 1
        $this->check();
528
        return [
0 ignored issues
show
introduced by
Expected 1 lines before "return", found 0.
Loading history...
529 1
            'starttransfer_time'    => $this->response()->starttransfer_time(),
530
            'size_download'    => $this->response()->size_download(),
531
            'speed_download'    => $this->response()->speed_download(),
532
            'size_upload'    => $this->response()->size_upload(),
533
            'upload_content' => $this->response()->upload_content_length(),
534
            'speed_upload'   => $this->response()->speed_upload(),
535
            'time_request'   => $this->response()->total_time()
0 ignored issues
show
introduced by
Multi-line arrays must have a trailing comma after the last element.
Loading history...
536 9
        ];
537
    }
538 9
539 8
    /**
540
     * get format in sql
541
     * @return mixed
0 ignored issues
show
introduced by
Expected 1 lines between description and annotations, found 0.
Loading history...
542
     */
543
    public function getFormat()
544
    {
545
        return $this->format;
546
    }
547 4
548
    /**
549 4
     * @return array
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
introduced by
@return annotation of method \ClickHouseDB\Statement::rows() does not specify type hint for items of its traversable return value.
Loading history...
550
     * @throws Exception\TransportException
551
     */
552 4
    public function rows()
0 ignored issues
show
introduced by
Method \ClickHouseDB\Statement::rows() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "array".
Loading history...
553 4
    {
554
        $this->init();
555 4
        return $this->array_data;
0 ignored issues
show
introduced by
Expected 1 lines before "return", found 0.
Loading history...
Coding Style introduced by
Member variable "array_data" is not in valid camel caps format
Loading history...
556 4
    }
557 4
558
    /**
559
     * @return false|string
560
    */
0 ignored issues
show
Coding Style introduced by
Expected 5 spaces before asterisk; 4 found
Loading history...
561
    public function jsonRows()
562
    {
563
        return json_encode($this->rows(), JSON_PRETTY_PRINT);
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...
introduced by
Constant JSON_PRETTY_PRINT should not be referenced via a fallback global name, but via a use statement.
Loading history...
564 4
    }
565 4
566 4
    /**
567
     * @param array|string $arr
0 ignored issues
show
introduced by
Incorrect annotations group.
Loading history...
introduced by
@param annotation of method \ClickHouseDB\Statement::array_to_tree() does not specify type hint for items of its traversable parameter $arr.
Loading history...
Coding Style introduced by
Expected 6 spaces after parameter type; 1 found
Loading history...
568 4
     * @param null|string|array $path
0 ignored issues
show
introduced by
Null type hint should be on last position in "null|string|array".
Loading history...
introduced by
@param annotation of method \ClickHouseDB\Statement::array_to_tree() does not specify type hint for items of its traversable parameter $path.
Loading history...
569 4
     * @return array
0 ignored issues
show
introduced by
@return annotation of method \ClickHouseDB\Statement::array_to_tree() does not specify type hint for items of its traversable return value.
Loading history...
570
     */
571
    private function array_to_tree($arr, $path = null)
0 ignored issues
show
Coding Style introduced by
Method name "Statement::array_to_tree" is not in camel caps format
Loading history...
introduced by
Method \ClickHouseDB\Statement::array_to_tree() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "array".
Loading history...
572
    {
573
        if (is_array($path)) {
0 ignored issues
show
introduced by
Function is_array() should not be referenced via a fallback global name, but via a use statement.
Loading history...
574 4
            $keys = $path;
575
        } else {
576 4
            $args = func_get_args();
0 ignored issues
show
introduced by
Function func_get_args() should not be referenced via a fallback global name, but via a use statement.
Loading history...
577
            array_shift($args);
0 ignored issues
show
introduced by
Function array_shift() should not be referenced via a fallback global name, but via a use statement.
Loading history...
578
579 4
            if (sizeof($args) < 2) {
0 ignored issues
show
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...
580
                $separator = '.';
581
                $keys = explode($separator, $path);
0 ignored issues
show
Coding Style introduced by
Equals sign not aligned with surrounding assignments; expected 6 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...
introduced by
Function explode() should not be referenced via a fallback global name, but via a use statement.
Loading history...
582
            } else {
583
                $keys = $args;
584
            }
585
        }
586
587
        //
0 ignored issues
show
introduced by
Empty comment
Loading history...
588
        $tree = $arr;
589
        while (count($keys)) {
0 ignored issues
show
introduced by
Expected 1 lines after "while", found 0.
Loading history...
introduced by
Function count() should not be referenced via a fallback global name, but via a use statement.
Loading history...
590
            $key = array_pop($keys);
0 ignored issues
show
introduced by
Function array_pop() should not be referenced via a fallback global name, but via a use statement.
Loading history...
591
592
            if (isset($arr[$key])) {
593
                $val = $arr[$key];
594
            } else {
595
                $val = $key;
596
            }
597
598
            $tree = array($val => $tree);
0 ignored issues
show
Coding Style introduced by
Short array syntax must be used to define arrays
Loading history...
599
        }
600
        if (!is_array($tree)) {
0 ignored issues
show
introduced by
Expected 1 lines after "if", found 0.
Loading history...
Coding Style introduced by
Expected 1 space 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...
601
            return [];
602
        }
603
        return $tree;
0 ignored issues
show
introduced by
Expected 1 lines before "return", found 0.
Loading history...
604
    }
0 ignored issues
show
Coding Style introduced by
Expected 1 blank line after function; 2 found
Loading history...
605
606
607
    public function rewind(): void {
608
        $this->iterator = 0;
609
    }
610
611
    /**
612
     * @return mixed
613
     */
614
    #[\ReturnTypeWillChange]
0 ignored issues
show
introduced by
Constant \ReturnTypeWillChange should not be referenced via a fully qualified name, but via a use statement.
Loading history...
615
    public function current() {
616
        if (!isset($this->array_data[$this->iterator])) {
0 ignored issues
show
introduced by
Expected 1 lines after "if", found 0.
Loading history...
Coding Style introduced by
Expected 1 space after NOT operator; 0 found
Loading history...
Coding Style introduced by
Member variable "array_data" is not in valid camel caps format
Loading history...
617
            return null;
618
        }
619
        return $this->array_data[$this->iterator];
0 ignored issues
show
introduced by
Expected 1 lines before "return", found 0.
Loading history...
Coding Style introduced by
Member variable "array_data" is not in valid camel caps format
Loading history...
620
    }
621
622
    public function key(): int {
623
        return $this->iterator;
624
    }
625
626
    public function next(): void {
627
        ++$this->iterator;
628
    }
629
630
    public function valid(): bool {
631
        $this->init();
632
        return isset($this->array_data[$this->iterator]);
0 ignored issues
show
introduced by
Expected 1 lines before "return", found 0.
Loading history...
Coding Style introduced by
Member variable "array_data" is not in valid camel caps format
Loading history...
633
    }
634
}
635