smi2 / phpClickHouse

src/Statement.php

<?php

Missing declare(strict_types=1).

namespace ClickHouseDB;

use ClickHouseDB\Exception\ClickHouseUnavailableException;
use ClickHouseDB\Exception\DatabaseException;
use ClickHouseDB\Exception\QueryException;
use ClickHouseDB\Query\Query;
use ClickHouseDB\Transport\CurlerRequest;
use ClickHouseDB\Transport\CurlerResponse;

class Statement implements \Iterator

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

{
    private const CLICKHOUSE_ERROR_REGEX = "%Code:\s(\d+)\.\s*DB::Exception\s*:\s*(.*)(?:,\s*e\.what|\(version).*%ius";

The string literal %Code:\s(\d+)\.\s*DB::Exception\s*:\s*(.*)(?:,\s*e\.what|\(version).*%ius does not require double quotes, as per coding-style, please use single quotes.

    /**

Found multi-line comment for property \ClickHouseDB\Statement::$_rawData with single line content, use one-line comment instead.

     * @var string|mixed
     */
    private $_rawData;

    /**

Found multi-line comment for property \ClickHouseDB\Statement::$_http_code with single line content, use one-line comment instead.

     * @var int
     */
    private $_http_code = -1;

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

    /**

Found multi-line comment for property \ClickHouseDB\Statement::$_request with single line content, use one-line comment instead.

     * @var CurlerRequest
     */
    private $_request = null;

    /**

Found multi-line comment for property \ClickHouseDB\Statement::$_init with single line content, use one-line comment instead.

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

    /**

Found multi-line comment for property \ClickHouseDB\Statement::$query with single line content, use one-line comment instead.

     * @var Query
     */
    private $query;

    /**

Found multi-line comment for property \ClickHouseDB\Statement::$format with single line content, use one-line comment instead.

     * @var mixed
     */
    private $format;

    /**

Found multi-line comment for property \ClickHouseDB\Statement::$sql with single line content, use one-line comment instead.

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

    /**

Found multi-line comment for property \ClickHouseDB\Statement::$meta with single line content, use one-line comment instead.

     * @var array

@var annotation of property \ClickHouseDB\Statement::$meta does not specify type hint for its items.

     */
    private $meta;

    /**

Found multi-line comment for property \ClickHouseDB\Statement::$totals with single line content, use one-line comment instead.

     * @var array

@var annotation of property \ClickHouseDB\Statement::$totals does not specify type hint for its items.

     */
    private $totals;

    /**

Found multi-line comment for property \ClickHouseDB\Statement::$extremes with single line content, use one-line comment instead.

     * @var array

@var annotation of property \ClickHouseDB\Statement::$extremes does not specify type hint for its items.

     */
    private $extremes;

    /**

Found multi-line comment for property \ClickHouseDB\Statement::$rows with single line content, use one-line comment instead.

     * @var int
     */
    private $rows;

    /**

Found multi-line comment for property \ClickHouseDB\Statement::$rows_before_limit_at_least with single line content, use one-line comment instead.

     * @var bool|integer

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

     */
    private $rows_before_limit_at_least = false;

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

    /**

Found multi-line comment for property \ClickHouseDB\Statement::$array_data with single line content, use one-line comment instead.

     * @var array

@var annotation of property \ClickHouseDB\Statement::$array_data does not specify type hint for its items.

     */
    private $array_data = [];

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

    /**

Found multi-line comment for property \ClickHouseDB\Statement::$statistics with single line content, use one-line comment instead.

     * @var array|null

@var annotation of property \ClickHouseDB\Statement::$statistics does not specify type hint for its items.

     */
    private $statistics = null;

    /**

Found multi-line comment for property \ClickHouseDB\Statement::$iterator with single line content, use one-line comment instead.

     * @var int
     */
    public $iterator = 0;


    public function __construct(CurlerRequest $request)

Expected 1 blank line before function; 2 found

    {
        $this->_request = $request;
        $this->format = $this->_request->getRequestExtendedInfo('format');

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

        $this->query = $this->_request->getRequestExtendedInfo('query');

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

        $this->sql = $this->_request->getRequestExtendedInfo('sql');

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

    }

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

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

    {
        return $this->_request;
    }

    /**
     * @return CurlerResponse

Incorrect annotations group.

     * @throws Exception\TransportException
     */
    private function response()

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

    {
        return $this->_request->response();
    }

    /**
     * @return mixed

Incorrect annotations group.

     * @throws Exception\TransportException
     */
    public function responseInfo()
    {
        return $this->response()->info();
    }

    /**
     * @return mixed|string
     */
    public function sql()
    {
        return $this->sql;
    }

    /**
     * @param string $body

Incorrect annotations group.

Method \ClickHouseDB\Statement::parseErrorClickHouse() has useless @param annotation for parameter $body.

     * @return array|bool

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

     */
    private function parseErrorClickHouse(string $body)
    {
        $body = trim($body);

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

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

        $matches = [];

        // Code: 115. DB::Exception: Unknown setting readonly[0], e.what() = DB::Exception
        // Code: 192. DB::Exception: Unknown user x, e.what() = DB::Exception
        // Code: 60. DB::Exception: Table default.ZZZZZ doesn't exist., e.what() = DB::Exception
        // 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))

        if (preg_match(self::CLICKHOUSE_ERROR_REGEX, $body, $matches)) {

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

            return ['code' => $matches[1], 'message' => $matches[2]];
        }

        return false;
    }

    private function hasErrorClickhouse(string $body): bool {

Expected 0 blank lines after opening function brace; 1 found

        return preg_match(self::CLICKHOUSE_ERROR_REGEX, $body) === 1;

Expected 0 lines before "return", found 1.

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

    }

    /**
     * @return bool

Incorrect annotations group.

     * @throws Exception\TransportException
     */
    public function error()

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

    {
        if (!$this->isError()) {

Expected 1 space after NOT operator; 0 found

            return false;
        }

        $body = $this->response()->body();

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

        $error_no = $this->response()->error_no();

The variable $error_no should be in camel caps format.

        $error = $this->response()->error();

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

        $dumpStatement = false;
        if (!$error_no && !$error) {

Expected 1 space after NOT operator; 0 found

The variable $error_no should be in camel caps format.

            $parse = $this->parseErrorClickHouse($body);

            if ($parse) {

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.

$parse is a non-empty array, thus is always true.

                throw new DatabaseException($parse['message'] . "\nIN:" . $this->sql(), $parse['code']);
            } else {

Remove useless "else" to reduce code nesting.

                $code = $this->response()->http_code();

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

                $message = "HttpCode:" . $this->response()->http_code() . " ; " . $this->response()->error() . " ;" . $body;

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

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

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

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

                $dumpStatement = true;
            }
        } else {
            $code = $error_no;

The variable $error_no should be in camel caps format.

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

            $message = $this->response()->error();
        }

        $exception = new QueryException($message, $code);
        if ($code === CURLE_COULDNT_CONNECT) {

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

            $exception = new ClickHouseUnavailableException($message, $code);
        }

        if ($dumpStatement) {

The condition $dumpStatement is always false.

            $exception->setRequestDetails($this->_request->getDetails());
            $exception->setResponseDetails($this->response()->getDetails());
        }

        throw $exception;
    }

    /**
     * @return bool

Incorrect annotations group.

Method \ClickHouseDB\Statement::isError() has useless @return annotation.

     * @throws Exception\TransportException
     */
    public function isError(): bool
    {
        if ($this->response()->http_code() !== 200) {
            return true;
        }

        if ($this->response()->error_no()) {
            return true;
        }

        if ($this->hasErrorClickhouse($this->response()->body())) {

Useless condition.

            return true;
        }

        return false;
    }

    private function check(): bool
    {
        if (!$this->_request->isResponseExists()) {

Expected 1 space after NOT operator; 0 found

            throw QueryException::noResponse();
        }

        if ($this->isError()) {
            $this->error();
        }

        return true;
    }

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

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

    {
        return $this->_init;
    }

    /**
     * @return bool

Incorrect annotations group.

     * @throws Exception\TransportException
     */
    private function init()

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

    {
        if ($this->_init) {
            return false;
        }

        $this->check();

        $this->_rawData = $this->response()->rawDataOrJson($this->format);

        if (!$this->_rawData) {

Expected 1 space after NOT operator; 0 found

            $this->_init = true;
            return false;

Expected 1 lines before "return", found 0.

        }

        $data = [];
        foreach (['meta', 'data', 'totals', 'extremes', 'rows', 'rows_before_limit_at_least', 'statistics'] as $key) {

Blank line found at start of control structure

            if (isset($this->_rawData[$key])) {

Use early exit to reduce code nesting.

                if ($key=='data') {

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

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

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

                    $data=$this->_rawData[$key];

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

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

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

                } else {
                    $this->{$key} = $this->_rawData[$key];
                }
            }
        }

        if (empty($this->meta)) {
            throw new QueryException('Can`t find meta');
        }

        $isJSONCompact=(stripos($this->format,'JSONCompact')!==false?true:false);

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

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

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

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

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

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

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

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

Useless ternary operator.

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

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

        $this->array_data = [];

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

        foreach ($data as $rows) {
            $r = [];

            if ($isJSONCompact) {
                $r[] = $rows;
            } else {
                foreach ($this->meta as $meta) {
                    $r[$meta['name']] = $rows[$meta['name']];
                }
            }

            $this->array_data[] = $r;

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

        }

        $this->_init = true;

        return true;
    }

    /**
     * @return array

Incorrect annotations group.

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

     * @throws \Exception

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

     */
    public function extremes()

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

    {
        $this->init();
        return $this->extremes;

Expected 1 lines before "return", found 0.

    }

    /**
     * @return mixed

Incorrect annotations group.

     * @throws Exception\TransportException
     */
    public function totalTimeRequest()
    {
        $this->check();
        return $this->response()->total_time();

Expected 1 lines before "return", found 0.

Expected 0 lines after "return", found 1.

    }

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

    /**
     * @return array

Incorrect annotations group.

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

     * @throws \Exception

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

     */
    public function extremesMin()

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

    {
        $this->init();

        if (empty($this->extremes['min'])) {
            return [];
        }

        return $this->extremes['min'];
    }

    /**
     * @return array

Incorrect annotations group.

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

     * @throws \Exception

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

     */
    public function extremesMax()

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

    {
        $this->init();

        if (empty($this->extremes['max'])) {
            return [];
        }

        return $this->extremes['max'];
    }

    /**
     * @return array

Incorrect annotations group.

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

     * @throws Exception\TransportException
     */
    public function totals()

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

    {
        $this->init();
        return $this->totals;

Expected 1 lines before "return", found 0.

    }

    /**

Empty comment

     *
     */
    public function dump()

Method \ClickHouseDB\Statement::dump() does not have void return type hint.

Method \ClickHouseDB\Statement::dump() does not need documentation comment.

    {
        $this->_request->dump();
        $this->response()->dump();
    }

    /**
     * @return bool|int

Incorrect annotations group.

     * @throws Exception\TransportException
     */
    public function countAll()
    {
        $this->init();
        return $this->rows_before_limit_at_least;

Expected 1 lines before "return", found 0.

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

    }

    /**
     * @param bool $key

Incorrect annotations group.

     * @return array|mixed|null

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

     * @throws Exception\TransportException
     */
    public function statistics($key = false)

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

    {
        $this->init();

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

Expected 1 space after NOT operator; 0 found

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

            return null;
        }

        if (!$key) return $this->statistics;

It is generally a best practice to always use braces with control structures.

Expected 1 space after NOT operator; 0 found

Expected 1 lines before "return", found -1.

        if (!isset($this->statistics[$key])) {

Expected 1 space after NOT operator; 0 found

            return null;
        }

        return $this->statistics[$key];

Expected 0 lines after "return", found 1.

    }

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

    /**
     * @return int

Incorrect annotations group.

     * @throws Exception\TransportException
     */
    public function count()

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

    {
        $this->init();
        return $this->rows;

Expected 1 lines before "return", found 0.

    }

    /**
     * @return mixed|string

Incorrect annotations group.

     * @throws Exception\TransportException
     */
    public function rawData()
    {
        if ($this->_init) {
            return $this->_rawData;
        }

        $this->check();

        return $this->response()->rawDataOrJson($this->format);
    }

    /**

Empty comment

     *
     */
    public function resetIterator()

Method \ClickHouseDB\Statement::resetIterator() does not have void return type hint.

Method \ClickHouseDB\Statement::resetIterator() does not need documentation comment.

    {
        $this->iterator=0;

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

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

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

    }

    public function fetchRow($key = null)

Method \ClickHouseDB\Statement::fetchRow() does not have parameter type hint nor @param annotation for its parameter $key.

Method \ClickHouseDB\Statement::fetchRow() does not have return type hint nor @return annotation for its return value.

    {
        $this->init();

        $position=$this->iterator;

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

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

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

        if (!isset($this->array_data[$position])) {

Expected 1 space after NOT operator; 0 found

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

            return null;
        }

        $this->iterator++;

        if (!$key) {

Expected 1 lines after "if", found 0.

Expected 1 space after NOT operator; 0 found

            return $this->array_data[$position];

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

        }
        if (!isset($this->array_data[$position][$key])) {

Expected 1 space after NOT operator; 0 found

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

            return null;
        }

        return $this->array_data[$position][$key];

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

Expected 0 lines after "return", found 1.

    }

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

Expected 1 blank line after function; 0 found

    /**
     * @param string $key

Incorrect annotations group.

     * @return mixed|null
     * @throws Exception\TransportException
     */
    public function fetchOne($key = null)

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

    {
        $this->init();
        if (!isset($this->array_data[0])) {

Expected 1 space after NOT operator; 0 found

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

            return null;
        }

        if (!$key) {

Expected 1 space after NOT operator; 0 found

            return $this->array_data[0];

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

        }

        if (!isset($this->array_data[0][$key])) {

Expected 1 space after NOT operator; 0 found

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

            return null;
        }

        return $this->array_data[0][$key];

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

    }

    /**
     * @param string|null $path

Incorrect annotations group.

     * @return array

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

     * @throws Exception\TransportException
     */
    public function rowsAsTree($path)

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

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

    {
        $this->init();

        $out = [];
        foreach ($this->array_data as $row) {

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

            $d = $this->array_to_tree($row, $path);

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

            $out = array_replace_recursive($d, $out);

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

        }

        return $out;
    }

    /**
     * Return size_upload,upload_content,speed_upload,time_request
     *
     * @return array

Incorrect annotations group.

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

     * @throws Exception\TransportException
     */
    public function info_upload()

Method name "Statement::info_upload" is not in camel caps format

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

    {
        $this->check();
        return [

Expected 1 lines before "return", found 0.

            'size_upload'    => $this->response()->size_upload(),
            'upload_content' => $this->response()->upload_content_length(),
            'speed_upload'   => $this->response()->speed_upload(),
            'time_request'   => $this->response()->total_time()

Multi-line arrays must have a trailing comma after the last element.

        ];
    }

    /**
     * Return size_upload,upload_content,speed_upload,time_request,starttransfer_time,size_download,speed_download
     *
     * @return array

Incorrect annotations group.

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

     * @throws Exception\TransportException
     */
    public function info()

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

    {
        $this->check();
        return [

Expected 1 lines before "return", found 0.

            'starttransfer_time'    => $this->response()->starttransfer_time(),
            'size_download'    => $this->response()->size_download(),
            'speed_download'    => $this->response()->speed_download(),
            'size_upload'    => $this->response()->size_upload(),
            'upload_content' => $this->response()->upload_content_length(),
            'speed_upload'   => $this->response()->speed_upload(),
            'time_request'   => $this->response()->total_time()

Multi-line arrays must have a trailing comma after the last element.

        ];
    }

    /**
     * get format in sql
     * @return mixed

Expected 1 lines between description and annotations, found 0.

     */
    public function getFormat()
    {
        return $this->format;
    }

    /**
     * @return array

Incorrect annotations group.

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

     * @throws Exception\TransportException
     */
    public function rows()

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

    {
        $this->init();
        return $this->array_data;

Expected 1 lines before "return", found 0.

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

    }

    /**
     * @return false|string
    */

Expected 5 spaces before asterisk; 4 found

    public function jsonRows()
    {
        return json_encode($this->rows(), JSON_PRETTY_PRINT);

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

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

    }

    /**
     * @param array|string $arr

Incorrect annotations group.

@param annotation of method \ClickHouseDB\Statement::array_to_tree() does not specify type hint for items of its traversable parameter $arr.

Expected 6 spaces after parameter type; 1 found

     * @param null|string|array $path

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

@param annotation of method \ClickHouseDB\Statement::array_to_tree() does not specify type hint for items of its traversable parameter $path.

     * @return array

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

     */
    private function array_to_tree($arr, $path = null)

Method name "Statement::array_to_tree" is not in camel caps format

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

    {
        if (is_array($path)) {

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

            $keys = $path;
        } else {
            $args = func_get_args();

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

            array_shift($args);

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

            if (sizeof($args) < 2) {

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

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

                $separator = '.';
                $keys = explode($separator, $path);

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

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

            } else {
                $keys = $args;
            }
        }

        //

Empty comment

        $tree = $arr;
        while (count($keys)) {

Expected 1 lines after "while", found 0.

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

            $key = array_pop($keys);

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

            if (isset($arr[$key])) {
                $val = $arr[$key];
            } else {
                $val = $key;
            }

            $tree = array($val => $tree);

Short array syntax must be used to define arrays

        }
        if (!is_array($tree)) {

Expected 1 lines after "if", found 0.

Expected 1 space after NOT operator; 0 found

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

            return [];
        }
        return $tree;

Expected 1 lines before "return", found 0.

    }

Expected 1 blank line after function; 2 found

    public function rewind(): void {
        $this->iterator = 0;
    }

    /**
     * @return mixed
     */
    #[\ReturnTypeWillChange]

Constant \ReturnTypeWillChange should not be referenced via a fully qualified name, but via a use statement.

    public function current() {
        if (!isset($this->array_data[$this->iterator])) {

Expected 1 lines after "if", found 0.

Expected 1 space after NOT operator; 0 found

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

            return null;
        }
        return $this->array_data[$this->iterator];

Expected 1 lines before "return", found 0.

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

    }

    public function key(): int {
        return $this->iterator;
    }

    public function next(): void {
        ++$this->iterator;
    }

    public function valid(): bool {
        $this->init();
        return isset($this->array_data[$this->iterator]);

Expected 1 lines before "return", found 0.

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

    }
}