ClickHouseDB\Query\Degeneration\Conditions::__ifsets()

Complexity

Conditions	21
Paths	55

Size

Total Lines	41
Code Lines	30

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	25
CRAP Score	21.1794

Importance

Changes

0

<?php
    static function __ifsets($matches, $markers)

It is generally recommended to explicitly declare the visibility for methods.

It is recommend to declare an explicit visibility for __ifsets.

Method \ClickHouseDB\Query\Degeneration\Conditions::__ifsets() does not have parameter type hint nor @param annotation for its parameter $matches.

Method \ClickHouseDB\Query\Degeneration\Conditions::__ifsets() does not have parameter type hint nor @param annotation for its parameter $markers.

Method \ClickHouseDB\Query\Degeneration\Conditions::__ifsets() does not have return type hint nor @return annotation for its return value.

    {
        $content_false = '';

The variable $content_false should be in camel caps format.

        $condition = '';

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

        $flag_else = '';

The assignment to $flag_else is dead and can be removed.

The variable $flag_else should be in camel caps format.

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

//print_r($matches);
        if (sizeof($matches) == 4) {

Expected 1 lines after "if", found 0.

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

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

            list($condition, $preset, $variable, $content_true) = $matches;

list(...) is forbidden, use [...] instead.

The variable $content_true should be in camel caps format.

        } elseif (sizeof($matches) == 6) {

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

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

            list($condition, $preset, $variable, $content_true, $flag_else, $content_false) = $matches;

list(...) is forbidden, use [...] instead.

The variable $content_true should be in camel caps format.

The variable $flag_else should be in camel caps format.

The variable $content_false should be in camel caps format.

        } else {
            throw new QueryException('Error in parse Conditions' . json_encode($matches));

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

        }
        $variable = trim($variable);

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

        $preset = strtolower(trim($preset));

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

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

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

        if ($preset == '') {

Expected 1 lines after "if", found 0.

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

            return (isset($markers[$variable]) && ($markers[$variable] || is_numeric($markers[$variable])))

Useless parentheses.

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

                ? $content_true

The variable $content_true should be in camel caps format.

                : $content_false;

The variable $content_false should be in camel caps format.

        }
        if ($preset == 'set') {

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

Expected 1 lines after "if", found 0.

            return (isset($markers[$variable]) && !empty($markers[$variable])) ? $content_true : $content_false;

The variable $content_true should be in camel caps format.

Useless parentheses.

The variable $content_false should be in camel caps format.

Expected 1 space after NOT operator; 0 found

        }
        if ($preset == 'bool') {

Expected 1 lines after "if", found 0.

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

            return (isset($markers[$variable]) && is_bool($markers[$variable]) && $markers[$variable] == true)

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

Useless parentheses.

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

                ? $content_true

The variable $content_true should be in camel caps format.

                : $content_false;

The variable $content_false should be in camel caps format.

        }
        if ($preset == 'string') {

Expected 1 lines after "if", found 0.

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

            return (isset($markers[$variable]) && is_string($markers[$variable]) && strlen($markers[$variable]))

Useless parentheses.

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

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

                ? $content_true

The variable $content_true should be in camel caps format.

                : $content_false;

The variable $content_false should be in camel caps format.

        }
        if ($preset == 'int') {

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

            return (isset($markers[$variable]) && intval($markers[$variable]) <> 0)

Useless parentheses.

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

Operator <> is disallowed, use !== instead.

                ? $content_true

The variable $content_true should be in camel caps format.

                : $content_false;

The variable $content_false should be in camel caps format.

        }

        return '';
    }