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

Complexity

Conditions	16
Paths	42

Size

Total Lines	37
Code Lines	21

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	272

Importance

Changes

0

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

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 parameter type hint nor @param annotation for its parameter $else.

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

    {
        $content_false = '';

        if ($else)
        {
            list($condition, $preset, $variable, $content_true, $content_false) = $matches;
        } else

Expected 1 space(s) after ELSE keyword; newline found

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

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

        if ($preset == 'set')

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

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

There must be a single space after a NOT operator; 0 found

        }
        if ($preset == 'bool')

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

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

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

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

                ? $content_true
                : $content_false;
        }
        if ($preset == 'string')

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

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

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
                : $content_false;
        }
        if ($preset == 'int')

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

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

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

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

                ? $content_true
                : $content_false;
        }

        return '';
    }