smi2 / phpClickHouse

src/Query/Degeneration/Bindings.php

<?php

declare(strict_types=1);

namespace ClickHouseDB\Query\Degeneration;

use ClickHouseDB\Query\Degeneration;
use ClickHouseDB\Quote\ValueFormatter;
use function array_map;

Expected 1 lines between different types of use statement, found 0.

use function implode;
use function is_array;

class Bindings implements Degeneration
{
    /**

Found multi-line comment for property \ClickHouseDB\Query\Degeneration\Bindings::$bindings with single line content, use one-line comment instead.

     * @var array

@var annotation of property \ClickHouseDB\Query\Degeneration\Bindings::$bindings does not specify type hint for its items.

     */
    protected $bindings = [];

    /**
     * @param array $bindings

@param annotation of method \ClickHouseDB\Query\Degeneration\Bindings::bindParams() does not specify type hint for items of its traversable parameter $bindings.

     */
    public function bindParams(array $bindings)

Method \ClickHouseDB\Query\Degeneration\Bindings::bindParams() does not have void return type hint.

    {
        $this->bindings = [];
        foreach ($bindings as $column => $value) {
            $this->bindParam($column, $value);
        }
    }

    public function getBind(): array

Method \ClickHouseDB\Query\Degeneration\Bindings::getBind() does not have @return annotation for its traversable return value.

    {
        return $this->bindings;
    }

    /**
     * @param string $column
     * @param mixed  $value
     */
    public function bindParam($column, $value)

Method \ClickHouseDB\Query\Degeneration\Bindings::bindParam() does not have native type hint for its parameter $column but it should be possible to add it based on @param annotation "string".

Method \ClickHouseDB\Query\Degeneration\Bindings::bindParam() does not have void return type hint.

    {
        $this->bindings[$column] = $value;
    }

    /**
     * Binds a list of values to the corresponding parameters.
     * This is similar to [[bindValue()]] except that it binds multiple values at a time.
     *
     * @param string $sql

Incorrect annotations group.

     * @param array $binds

@param annotation of method \ClickHouseDB\Query\Degeneration\Bindings::compile_binds() does not specify type hint for items of its traversable parameter $binds.

Expected 2 spaces after parameter type; 1 found

     * @param string $pattern
     * @return string
     */
    public function compile_binds($sql, $binds,$pattern)

Method name "Bindings::compile_binds" is not in camel caps format

Method \ClickHouseDB\Query\Degeneration\Bindings::compile_binds() does not have native type hint for its parameter $sql but it should be possible to add it based on @param annotation "string".

Method \ClickHouseDB\Query\Degeneration\Bindings::compile_binds() does not have native type hint for its parameter $binds but it should be possible to add it based on @param annotation "array".

Method \ClickHouseDB\Query\Degeneration\Bindings::compile_binds() does not have native type hint for its parameter $pattern but it should be possible to add it based on @param annotation "string".

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

Expected 1 space between comma and argument "$pattern"; 0 found

    {
        return preg_replace_callback($pattern, function($m) use ($binds){

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

Expected 1 space after FUNCTION keyword; 0 found

Closure not using "$this" should be declared static.

Expected 1 space before opening brace; found 0

            if(isset($binds[$m[1]])){ // If it exists in our array

Expected 1 space after IF keyword; 0 found

                return $binds[$m[1]]; // Then replace it from our array
            }

            return $m[0]; // Otherwise return the whole match (basically we won't change it)
        }, $sql);
    }

    /**
     * Compile Bindings
     *
     * @param string $sql

Incorrect annotations group.

     * @return mixed
     */
    public function process($sql)

Method \ClickHouseDB\Query\Degeneration\Bindings::process() does not have native type hint for its parameter $sql but it should be possible to add it based on @param annotation "string".

    {
        $bindFormatted=[];

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

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

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

        $bindRaw=[];

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

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

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

        foreach ($this->bindings as $key => $value) {
            if (is_array($value)) {
                $valueSet = implode(', ', $value);

                $values = array_map(
                    function ($value) {

Closure not using "$this" should be declared static.

                        return ValueFormatter::formatValue($value);
                    },
                    $value
                );

                $formattedParameter = implode(',', $values);
            } else {
                $valueSet           = $value;
                $formattedParameter = ValueFormatter::formatValue($value);
            }

            if ($formattedParameter !== null) {
                $bindFormatted[$key]=$formattedParameter;

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 ($valueSet !== null) {

Use early exit to reduce code nesting.

                $bindRaw[$key]=$valueSet;

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

            }
        }

        for ($loop=0;$loop<2;$loop++)

Expected 1 lines after "for", found 0.

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

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

Expected 1 space after first semicolon of FOR loop; 0 found

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

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

Expected 1 space after second semicolon of FOR loop; 0 found

        {
            // dipping in binds
            // example ['A' => '{B}' , 'B'=>':C','C'=>123]
            $sql=$this->compile_binds($sql,$bindRaw,'#{([\w+]+)}#');

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

        }
        $sql=$this->compile_binds($sql,$bindFormatted,'#:([\w+]+)#');

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

        return $sql;
    }
}