Function b_system_admin_modules_show has a Cognitive Complexity of 30 (exceeds 5 allowed). Consider refactoring.

Open

function b_system_admin_modules_show() {
    $block['mods'] = array();
    $module_handler = icms::handler('icms_module');
    $moduleperm_handler = icms::handler('icms_member_groupperm');
    $criteria = new icms_db_criteria_Compo();

• Read up
  Read up

Function b_system_admin_warnings_show has a Cognitive Complexity of 26 (exceeds 5 allowed). Consider refactoring.

Open

function b_system_admin_warnings_show() {
    $block = array();
    $block['msg'] = array();

    // Check if PDO is being used - if not, issue a warning, because it is being removed from PHP

• Read up
  Read up

Method b_system_admin_warnings_show has 69 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function b_system_admin_warnings_show() {
    $block = array();
    $block['msg'] = array();

    // Check if PDO is being used - if not, issue a warning, because it is being removed from PHP

Method b_system_admin_modules_show has 57 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function b_system_admin_modules_show() {
    $block['mods'] = array();
    $module_handler = icms::handler('icms_module');
    $moduleperm_handler = icms::handler('icms_member_groupperm');
    $criteria = new icms_db_criteria_Compo();

Function b_system_admin_cp_new_show has a Cognitive Complexity of 14 (exceeds 5 allowed). Consider refactoring.

Open

function b_system_admin_cp_new_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;

• Read up
  Read up

Function b_system_admin_cp_show has a Cognitive Complexity of 14 (exceeds 5 allowed). Consider refactoring.

Open

function b_system_admin_cp_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;
    $block['lang_insmodules'] = _AD_INSTALLEDMODULES;

• Read up
  Read up

Method b_system_admin_cp_new_show has 38 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function b_system_admin_cp_new_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;

Method b_system_admin_cp_show has 37 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function b_system_admin_cp_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;
    $block['lang_insmodules'] = _AD_INSTALLEDMODULES;

The function b_system_admin_warnings_show() has an NPath complexity of 331776. The configured NPath complexity threshold is 200.

Open

function b_system_admin_warnings_show() {
    $block = array();
    $block['msg'] = array();

    // Check if PDO is being used - if not, issue a warning, because it is being removed from PHP

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    function bar() {
        // lots of complicated code
    }
}

The function b_system_admin_modules_show() has an NPath complexity of 10370. The configured NPath complexity threshold is 200.

Open

function b_system_admin_modules_show() {
    $block['mods'] = array();
    $module_handler = icms::handler('icms_module');
    $moduleperm_handler = icms::handler('icms_member_groupperm');
    $criteria = new icms_db_criteria_Compo();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    function bar() {
        // lots of complicated code
    }
}

The function b_system_admin_cp_show() has a Cyclomatic Complexity of 10. The configured cyclomatic complexity threshold is 10.

Open

function b_system_admin_cp_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;
    $block['lang_insmodules'] = _AD_INSTALLEDMODULES;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

The function b_system_admin_modules_show() has a Cyclomatic Complexity of 20. The configured cyclomatic complexity threshold is 10.

Open

function b_system_admin_modules_show() {
    $block['mods'] = array();
    $module_handler = icms::handler('icms_module');
    $moduleperm_handler = icms::handler('icms_member_groupperm');
    $criteria = new icms_db_criteria_Compo();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

The function b_system_admin_warnings_show() has a Cyclomatic Complexity of 22. The configured cyclomatic complexity threshold is 10.

Open

function b_system_admin_warnings_show() {
    $block = array();
    $block['msg'] = array();

    // Check if PDO is being used - if not, issue a warning, because it is being removed from PHP

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

The function b_system_admin_cp_new_show() has a Cyclomatic Complexity of 10. The configured cyclomatic complexity threshold is 10.

Open

function b_system_admin_cp_new_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

Avoid using undefined variables such as '$modversion' which will lead to PHP notices.

Open

        unset($modversion);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$modversion' which will lead to PHP notices.

Open

            if (FALSE != $all_ok || in_array($modversion['category'], $ok_syscats)) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$block' which will lead to PHP notices.

Open

    if (count($block['mods']) > 0) return $block;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$modversion' which will lead to PHP notices.

Open

                $sysmod = array('title' => $modversion['name'], 'link' => ICMS_MODULES_URL . '/system/admin.php?fct=' . $file, 'image' => ICMS_MODULES_URL . '/system/admin/' . $file . '/images/' . $file . '_big.png');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$modversion' which will lead to PHP notices.

Open

        if ($modversion['hasAdmin']) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$block' which will lead to PHP notices.

Open

    $block['mods'] = array();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$modversion' which will lead to PHP notices.

Open

            $category = isset($modversion['category']) ? (int) ($modversion['category']) : 0;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$modversion' which will lead to PHP notices.

Open

                $block[$modversion['group']][] = array('title' => $modversion['name'], 'link' => ICMS_MODULES_URL . '/system/admin.php?fct=' . $file, 'image' => ICMS_MODULES_URL . '/system/admin/' . $file . '/images/' . $file . '_big.png');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$modversion' which will lead to PHP notices.

Open

                $block[$modversion['group']][] = array('title' => $modversion['name'], 'link' => ICMS_MODULES_URL . '/system/admin.php?fct=' . $file, 'image' => ICMS_MODULES_URL . '/system/admin/' . $file . '/images/' . $file . '_big.png');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$block' which will lead to PHP notices.

Open

    $block['lang_insmodules'] = _AD_INSTALLEDMODULES;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$modversion' which will lead to PHP notices.

Open

        if ($modversion['hasAdmin']) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$block' which will lead to PHP notices.

Open

    $block['lang_cp'] = _CPHOME;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$block' which will lead to PHP notices.

Open

    if (count($block['sysmod']) > 0) return $block;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$modversion' which will lead to PHP notices.

Open

        unset($modversion);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$block' which will lead to PHP notices.

Open

    $block['sysmod'] = array();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$modversion' which will lead to PHP notices.

Open

            if (FALSE != $all_ok || in_array($modversion['category'], $ok_syscats)) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$block' which will lead to PHP notices.

Open

                array_push($block['sysmod'], $sysmod);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$block' which will lead to PHP notices.

Open

    if (count($block['sysmod']) > 0) return $block;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$block' which will lead to PHP notices.

Open

                $block['mods'][] = $rtn;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$modversion' which will lead to PHP notices.

Open

            $category = isset($modversion['category']) ? (int) ($modversion['category']) : 0;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$modversion' which will lead to PHP notices.

Open

            $category = isset($modversion['category']) ? (int) ($modversion['category']) : 0;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$block' which will lead to PHP notices.

Open

    if (count($block['mods']) > 0) return $block;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$modversion' which will lead to PHP notices.

Open

            $category = isset($modversion['category']) ? (int) ($modversion['category']) : 0;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Missing class import via use statement (line '177', column '21').

Open

    $criteria->add(new icms_db_criteria_Item('isactive', 1));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '81', column '18').

Open

    $criteria = new icms_db_criteria_Compo();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '96', column '21').

Open

    $criteria->add(new icms_db_criteria_Item('conf_value', 'tinymce'));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '95', column '18').

Open

    $criteria = new icms_db_criteria_Compo();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '176', column '21').

Open

    $criteria->add(new icms_db_criteria_Item('hasadmin', 1));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '82', column '21').

Open

    $criteria->add(new icms_db_criteria_Item('conf_value', 'FCKeditor'));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '78', column '23').

Open

    $editorhandler = new icms_plugins_EditorHandler();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '175', column '18').

Open

    $criteria = new icms_db_criteria_Compo();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Avoid using static access to class 'icms_core_Message' in method 'b_system_admin_warnings_show'.

Open

            array_push($block['msg'], icms_core_Message::error(_FCK_PRESENT_WARNING), '', false);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'icms' in method 'b_system_admin_cp_show'.

Open

        $sysperm_handler = icms::handler('icms_member_groupperm');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid assigning values to variables in if clauses and the like (line '53', column '6').

Open

function b_system_admin_warnings_show() {
    $block = array();
    $block['msg'] = array();

    // Check if PDO is being used - if not, issue a warning, because it is being removed from PHP

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

Avoid using static access to class 'icms_core_Message' in method 'b_system_admin_warnings_show'.

Open

        array_push($block['msg'], icms_core_Message::error('<a href="' . ICMS_MODULES_URL . '/system/admin.php?fct=modulesadmin&amp;op=update&amp;module=system">' . _WARNINGUPDATESYSTEM . '</a>'));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

The method b_system_admin_cp_show uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

    } else {
        $groups = array();
    }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class 'icms_core_Message' in method 'b_system_admin_warnings_show'.

Open

        array_push($block['msg'], icms_core_Message::error(_FCK_DEFAULT_WARNING), '', false);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'icms_core_Filesystem' in method 'b_system_admin_cp_show'.

Open

    $dirlist = icms_core_Filesystem::getDirList($admin_dir);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'icms' in method 'b_system_admin_modules_show'.

Open

    $module_handler = icms::handler('icms_module');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'icms_core_Message' in method 'b_system_admin_warnings_show'.

Open

        array_push($block['msg'], icms_core_Message::error(_CORE_MYSQL_DEPRECATED, '', FALSE));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'icms_core_Message' in method 'b_system_admin_warnings_show'.

Open

    if (!is_writable(ICMS_COMPILE_PATH)) array_push($block['msg'], icms_core_Message::warning(sprintf(_WARNINNOTWRITEABLE, ICMS_COMPILE_PATH)), '', FALSE);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'icms' in method 'b_system_admin_warnings_show'.

Open

    $config_handler = icms::handler('icms_config');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'icms_core_Message' in method 'b_system_admin_warnings_show'.

Open

            array_push($block['msg'], icms_core_Message::error(_TinyMCE_PRESENT_WARNING), '', false);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'icms_core_Message' in method 'b_system_admin_warnings_show'.

Open

        array_push($block['msg'], icms_core_Message::error(sprintf(_WARNINSTALL2, ICMS_ROOT_PATH . '/upgrade/'), '', FALSE));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

The method b_system_admin_warnings_show uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

    } else {
        if (in_array('FCKeditor', array_flip($editorhandler->getList()))) {
            array_push($block['msg'], icms_core_Message::error(_FCK_PRESENT_WARNING), '', false);
        }
    }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method b_system_admin_warnings_show uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

    } else {
        if (in_array('tinymce', array_flip($editorhandler->getList()))) {
            array_push($block['msg'], icms_core_Message::error(_TinyMCE_PRESENT_WARNING), '', false);
        }
    }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class 'icms' in method 'b_system_admin_cp_new_show'.

Open

        $sysperm_handler = icms::handler('icms_member_groupperm');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

The method b_system_admin_modules_show uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            $rtn['hassubs'] = 0;
            unset($rtn['subs']);
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method b_system_admin_cp_show uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

    } else {
        $all_ok = TRUE;
    }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class 'icms_core_Message' in method 'b_system_admin_warnings_show'.

Open

        array_push($block['msg'], icms_core_Message::error(_TRUST_PATH_HELP));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'icms_core_Message' in method 'b_system_admin_warnings_show'.

Open

        array_push($block['msg'], icms_core_Message::error(_PROTECTOR_NOT_FOUND, '', FALSE));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'icms' in method 'b_system_admin_modules_show'.

Open

    $moduleperm_handler = icms::handler('icms_member_groupperm');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'icms_core_Filesystem' in method 'b_system_admin_cp_new_show'.

Open

    $dirlist = icms_core_Filesystem::getDirList($admin_dir);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'icms_core_Message' in method 'b_system_admin_warnings_show'.

Open

        array_push($block['msg'], icms_core_Message::error(sprintf(_WARNINSTALL2, ICMS_ROOT_PATH . '/install/'), '', FALSE));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'icms_core_Message' in method 'b_system_admin_warnings_show'.

Open

        array_push($block['msg'], icms_core_Message::error(sprintf(_WARNINWRITEABLE, ICMS_ROOT_PATH . '/mainfile.php'), '', FALSE));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'icms_core_Message' in method 'b_system_admin_warnings_show'.

Open

    if (!is_writable(ICMS_CACHE_PATH)) array_push($block['msg'], icms_core_Message::warning(sprintf(_WARNINNOTWRITEABLE, ICMS_CACHE_PATH)), '', FALSE);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

The method b_system_admin_cp_new_show uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

    } else {
        $all_ok = TRUE;
    }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class 'icms_core_Message' in method 'b_system_admin_warnings_show'.

Open

        array_push($block['msg'], icms_core_Message::error(sprintf(_CSSTIDY_VULN, 'plugins/csstidy/css_optimiser.php'), '', false));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'icms_core_Message' in method 'b_system_admin_warnings_show'.

Open

        array_push($block['msg'], icms_core_Message::error(_TinyMCE_DEFAULT_WARNING), '', false);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

The method b_system_admin_modules_show uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            $rtn['hassubs'] = 0;
            unset($rtn['subs']);
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class 'icms_core_Message' in method 'b_system_admin_warnings_show'.

Open

    if (!is_writable(ICMS_UPLOAD_PATH)) array_push($block['msg'], icms_core_Message::warning(sprintf(_WARNINNOTWRITEABLE, ICMS_UPLOAD_PATH)), '', FALSE);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'icms_core_Message' in method 'b_system_admin_warnings_show'.

Open

    if (version_compare(phpversion(), '7.2', '<')) array_push($block['msg'], icms_core_Message::warning(sprintf(_OUTDATED_PHP, phpversion())), '', FALSE);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

The method b_system_admin_cp_new_show uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

    } else {
        $groups = array();
    }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid unused local variables such as '$icmsTpl'.

Open

    global $icmsTpl, $icmsConfig;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}

Avoid unused local variables such as '$systemadm'.

Open

            $systemadm = TRUE;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}

Avoid unused local variables such as '$icmsTpl'.

Open

    global $icmsTpl, $icmsConfig;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}

Avoid unused local variables such as '$icmsConfig'.

Open

    global $icmsTpl, $icmsConfig;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}

Avoid unused local variables such as '$category'.

Open

            $category = isset($modversion['category']) ? (int) ($modversion['category']) : 0;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}

Avoid unused local variables such as '$icmsConfig'.

Open

    global $icmsTpl, $icmsConfig;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}

Avoid unused local variables such as '$category'.

Open

            $category = isset($modversion['category']) ? (int) ($modversion['category']) : 0;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}

Reference to static property xoopsDB from undeclared class \icms

Open

    $sql1 = "SELECT conf_modid FROM `" . icms::$xoopsDB->prefix('config') . "` WHERE conf_name = 'dos_skipmodules'";

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_CSSTIDY_VULN

Open

        array_push($block['msg'], icms_core_Message::error(sprintf(_CSSTIDY_VULN, 'plugins/csstidy/css_optimiser.php'), '', false));

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_WARNINNOTWRITEABLE

Open

    if (!is_writable(ICMS_COMPILE_PATH)) array_push($block['msg'], icms_core_Message::warning(sprintf(_WARNINNOTWRITEABLE, ICMS_COMPILE_PATH)), '', FALSE);

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property xoopsDB from undeclared class \icms

Open

    if (getDbValue(icms::$xoopsDB, 'modules', 'version', 'version="120" AND mid="1"') !== FALSE) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method error from undeclared class \icms_core_Message

Open

        array_push($block['msg'], icms_core_Message::error(sprintf(_WARNINWRITEABLE, ICMS_ROOT_PATH . '/mainfile.php'), '', FALSE));

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property xoopsDB from undeclared class \icms

Open

    if ($result1 = icms::$xoopsDB->query($sql1)) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_PROTECTOR_NOT_FOUND

Open

        array_push($block['msg'], icms_core_Message::error(_PROTECTOR_NOT_FOUND, '', FALSE));

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Compo

Open

    $criteria = new icms_db_criteria_Compo();

• Exclude checks
  • Disable engine
  • Disable check

Call to method getList from undeclared class \icms_plugins_EditorHandler

Open

        if (in_array('FCKeditor', array_flip($editorhandler->getList()))) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method setSort from undeclared class \icms_db_criteria_Compo

Open

    $criteria->setSort('mid');

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property xoopsDB from undeclared class \icms

Open

        list($modid) = icms::$xoopsDB->FetchRow($result1);

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_plugins_EditorHandler

Open

    $editorhandler = new icms_plugins_EditorHandler();

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Compo

Open

    $criteria = new icms_db_criteria_Compo();

• Exclude checks
  • Disable engine
  • Disable check

Call to method error from undeclared class \icms_core_Message

Open

        array_push($block['msg'], icms_core_Message::error(_CORE_MYSQL_DEPRECATED, '', FALSE));

• Exclude checks
  • Disable engine
  • Disable check

Call to method error from undeclared class \icms_core_Message

Open

            array_push($block['msg'], icms_core_Message::error(_TinyMCE_PRESENT_WARNING), '', false);

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

        $sysperm_handler = icms::handler('icms_member_groupperm');

• Exclude checks
  • Disable engine
  • Disable check

Call to method error from undeclared class \icms_core_Message

Open

        array_push($block['msg'], icms_core_Message::error(_TRUST_PATH_HELP));

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property xoopsDB from undeclared class \icms

Open

            $sql2 = "SELECT isactive FROM `" . icms::$xoopsDB->prefix('modules') . "` WHERE mid =" . $modid;

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property xoopsDB from undeclared class \icms

Open

            list($protector_is_active) = icms::$xoopsDB->FetchRow($result2);

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_TinyMCE_DEFAULT_WARNING

Open

        array_push($block['msg'], icms_core_Message::error(_TinyMCE_DEFAULT_WARNING), '', false);

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_PREFERENCES

Open

            $subs = array('title' => _PREFERENCES, 'link' => ICMS_MODULES_URL . '/system/admin.php?fct=preferences&op=showmod&mod=' . $module->getVar('mid'));

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_TRUST_PATH_HELP

Open

        array_push($block['msg'], icms_core_Message::error(_TRUST_PATH_HELP));

• Exclude checks
  • Disable engine
  • Disable check

Call to method warning from undeclared class \icms_core_Message

Open

    if (!is_writable(ICMS_COMPILE_PATH)) array_push($block['msg'], icms_core_Message::warning(sprintf(_WARNINNOTWRITEABLE, ICMS_COMPILE_PATH)), '', FALSE);

• Exclude checks
  • Disable engine
  • Disable check

Variable $block was undeclared, but array fields are being added to it.

Open

    $block['lang_cp'] = _CPHOME;

• Exclude checks
  • Disable engine
  • Disable check

Call to method error from undeclared class \icms_core_Message

Open

        array_push($block['msg'], icms_core_Message::error('<a href="' . ICMS_MODULES_URL . '/system/admin.php?fct=modulesadmin&amp;op=update&amp;module=system">' . _WARNINGUPDATESYSTEM . '</a>'));

• Exclude checks
  • Disable engine
  • Disable check

Call to method error from undeclared class \icms_core_Message

Open

        array_push($block['msg'], icms_core_Message::error(sprintf(_CSSTIDY_VULN, 'plugins/csstidy/css_optimiser.php'), '', false));

• Exclude checks
  • Disable engine
  • Disable check

Call to method error from undeclared class \icms_core_Message

Open

            array_push($block['msg'], icms_core_Message::error(_FCK_PRESENT_WARNING), '', false);

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_FCK_PRESENT_WARNING

Open

            array_push($block['msg'], icms_core_Message::error(_FCK_PRESENT_WARNING), '', false);

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

        $groups = icms::$user->getGroups();

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

        $sysperm_handler = icms::handler('icms_member_groupperm');

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_WARNINSTALL2

Open

        array_push($block['msg'], icms_core_Message::error(sprintf(_WARNINSTALL2, ICMS_ROOT_PATH . '/install/'), '', FALSE));

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

    $config_handler = icms::handler('icms_config');

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

    $criteria->add(new icms_db_criteria_Item('conf_value', 'tinymce'));

• Exclude checks
  • Disable engine
  • Disable check

Call to method add from undeclared class \icms_db_criteria_Compo

Open

    $criteria->add(new icms_db_criteria_Item('conf_value', 'tinymce'));

• Exclude checks
  • Disable engine
  • Disable check

Variable $block was undeclared, but array fields are being added to it.

Open

    $block['mods'] = array();

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_CPHOME

Open

    $block['lang_cp'] = _CPHOME;

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_WARNINNOTWRITEABLE

Open

    if (!is_writable(ICMS_UPLOAD_PATH)) array_push($block['msg'], icms_core_Message::warning(sprintf(_WARNINNOTWRITEABLE, ICMS_UPLOAD_PATH)), '', FALSE);

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

    $module_handler = icms::handler('icms_module');

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

    $moduleperm_handler = icms::handler('icms_member_groupperm');

• Exclude checks
  • Disable engine
  • Disable check

Call to method add from undeclared class \icms_db_criteria_Compo

Open

    $criteria->add(new icms_db_criteria_Item('hasadmin', 1));

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

        $groups = icms::$user->getGroups();

• Exclude checks
  • Disable engine
  • Disable check

Call to method getDirList from undeclared class \icms_core_Filesystem

Open

    $dirlist = icms_core_Filesystem::getDirList($admin_dir);

• Exclude checks
  • Disable engine
  • Disable check

Call to method warning from undeclared class \icms_core_Message

Open

    if (version_compare(phpversion(), '7.2', '<')) array_push($block['msg'], icms_core_Message::warning(sprintf(_OUTDATED_PHP, phpversion())), '', FALSE);

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_TinyMCE_PRESENT_WARNING

Open

            array_push($block['msg'], icms_core_Message::error(_TinyMCE_PRESENT_WARNING), '', false);

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_WARNINGUPDATESYSTEM

Open

        array_push($block['msg'], icms_core_Message::error('<a href="' . ICMS_MODULES_URL . '/system/admin.php?fct=modulesadmin&amp;op=update&amp;module=system">' . _WARNINGUPDATESYSTEM . '</a>'));

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property xoopsDB from undeclared class \icms

Open

            $result2 = icms::$xoopsDB->query($sql2);

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_WARNINNOTWRITEABLE

Open

    if (!is_writable(ICMS_CACHE_PATH)) array_push($block['msg'], icms_core_Message::warning(sprintf(_WARNINNOTWRITEABLE, ICMS_CACHE_PATH)), '', FALSE);

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_OUTDATED_PHP

Open

    if (version_compare(phpversion(), '7.2', '<')) array_push($block['msg'], icms_core_Message::warning(sprintf(_OUTDATED_PHP, phpversion())), '', FALSE);

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

    $criteria->add(new icms_db_criteria_Item('conf_value', 'FCKeditor'));

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_FCK_DEFAULT_WARNING

Open

        array_push($block['msg'], icms_core_Message::error(_FCK_DEFAULT_WARNING), '', false);

• Exclude checks
  • Disable engine
  • Disable check

Call to method error from undeclared class \icms_core_Message

Open

        array_push($block['msg'], icms_core_Message::error(_TinyMCE_DEFAULT_WARNING), '', false);

• Exclude checks
  • Disable engine
  • Disable check

Call to method getList from undeclared class \icms_plugins_EditorHandler

Open

        if (in_array('tinymce', array_flip($editorhandler->getList()))) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

    $criteria->add(new icms_db_criteria_Item('isactive', 1));

• Exclude checks
  • Disable engine
  • Disable check

Call to method error from undeclared class \icms_core_Message

Open

        array_push($block['msg'], icms_core_Message::error(sprintf(_WARNINSTALL2, ICMS_ROOT_PATH . '/install/'), '', FALSE));

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_CPHOME

Open

    $block['lang_cp'] = _CPHOME;

• Exclude checks
  • Disable engine
  • Disable check

Variable $modversion is undeclared

Open

        if ($modversion['hasAdmin']) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_WARNINWRITEABLE

Open

        array_push($block['msg'], icms_core_Message::error(sprintf(_WARNINWRITEABLE, ICMS_ROOT_PATH . '/mainfile.php'), '', FALSE));

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_WARNINSTALL2

Open

        array_push($block['msg'], icms_core_Message::error(sprintf(_WARNINSTALL2, ICMS_ROOT_PATH . '/upgrade/'), '', FALSE));

• Exclude checks
  • Disable engine
  • Disable check

Call to method error from undeclared class \icms_core_Message

Open

        array_push($block['msg'], icms_core_Message::error(_PROTECTOR_NOT_FOUND, '', FALSE));

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property db from undeclared class \icms

Open

    if (icms::$db === NULL) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method warning from undeclared class \icms_core_Message

Open

    if (!is_writable(ICMS_CACHE_PATH)) array_push($block['msg'], icms_core_Message::warning(sprintf(_WARNINNOTWRITEABLE, ICMS_CACHE_PATH)), '', FALSE);

• Exclude checks
  • Disable engine
  • Disable check

Call to method error from undeclared class \icms_core_Message

Open

        array_push($block['msg'], icms_core_Message::error(sprintf(_WARNINSTALL2, ICMS_ROOT_PATH . '/upgrade/'), '', FALSE));

• Exclude checks
  • Disable engine
  • Disable check

Call to method add from undeclared class \icms_db_criteria_Compo

Open

    $criteria->add(new icms_db_criteria_Item('conf_value', 'FCKeditor'));

• Exclude checks
  • Disable engine
  • Disable check

Call to method warning from undeclared class \icms_core_Message

Open

    if (!is_writable(ICMS_UPLOAD_PATH)) array_push($block['msg'], icms_core_Message::warning(sprintf(_WARNINNOTWRITEABLE, ICMS_UPLOAD_PATH)), '', FALSE);

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Compo

Open

    $criteria = new icms_db_criteria_Compo();

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

    if (is_object(icms::$user)) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method error from undeclared class \icms_core_Message

Open

        array_push($block['msg'], icms_core_Message::error(_FCK_DEFAULT_WARNING), '', false);

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_AD_INSTALLEDMODULES

Open

    $block['lang_insmodules'] = _AD_INSTALLEDMODULES;

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

        if (is_object(icms::$user)) $admin_perm = $moduleperm_handler->checkRight('module_admin', $module->getVar('mid'), icms::$user->getGroups());

• Exclude checks
  • Disable engine
  • Disable check

Argument 2 (haystack) is null but \in_array() takes array

Open

            if (FALSE != $all_ok || in_array($modversion['category'], $ok_syscats)) {

• Exclude checks
  • Disable engine
  • Disable check

Variable $modversion is undeclared

Open

        if ($modversion['hasAdmin']) {

• Exclude checks
  • Disable engine
  • Disable check

Argument 2 (haystack) is null but \in_array() takes array

Open

            if (FALSE != $all_ok || in_array($modversion['category'], $ok_syscats)) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

    $criteria->add(new icms_db_criteria_Item('hasadmin', 1));

• Exclude checks
  • Disable engine
  • Disable check

Variable $block was undeclared, but array fields are being added to it.

Open

    $block['lang_cp'] = _CPHOME;

• Exclude checks
  • Disable engine
  • Disable check

Call to method getDirList from undeclared class \icms_core_Filesystem

Open

    $dirlist = icms_core_Filesystem::getDirList($admin_dir);

• Exclude checks
  • Disable engine
  • Disable check

Call to method add from undeclared class \icms_db_criteria_Compo

Open

    $criteria->add(new icms_db_criteria_Item('isactive', 1));

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

    if (is_object(icms::$user)) {

• Exclude checks
  • Disable engine
  • Disable check

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    if ($confcount > 0) {
        array_push($block['msg'], icms_core_Message::error(_TinyMCE_DEFAULT_WARNING), '', false);
    } else {
        if (in_array('tinymce', array_flip($editorhandler->getList()))) {
            array_push($block['msg'], icms_core_Message::error(_TinyMCE_PRESENT_WARNING), '', false);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    if ($confcount > 0) {
        array_push($block['msg'], icms_core_Message::error(_FCK_DEFAULT_WARNING), '', false);
    } else {
        if (in_array('FCKeditor', array_flip($editorhandler->getList()))) {
            array_push($block['msg'], icms_core_Message::error(_FCK_PRESENT_WARNING), '', false);

• Read up
  Read up

TRUE, FALSE and NULL must be lowercase; expected "false" but found "FALSE"

Open

    if (getDbValue(icms::$xoopsDB, 'modules', 'version', 'version="120" AND mid="1"') !== FALSE) {

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "true" but found "TRUE"

Open

        $all_ok = TRUE;

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "false" but found "FALSE"

Open

            if (FALSE != $all_ok || in_array($modversion['category'], $ok_syscats)) {

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "false" but found "FALSE"

Open

        array_push($block['msg'], icms_core_Message::error(_CORE_MYSQL_DEPRECATED, '', FALSE));

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "false" but found "FALSE"

Open

        array_push($block['msg'], icms_core_Message::error(sprintf(_WARNINSTALL2, ICMS_ROOT_PATH . '/upgrade/'), '', FALSE));

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "false" but found "FALSE"

Open

    if (!is_writable(ICMS_CACHE_PATH)) array_push($block['msg'], icms_core_Message::warning(sprintf(_WARNINNOTWRITEABLE, ICMS_CACHE_PATH)), '', FALSE);

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "true" but found "TRUE"

Open

        $all_ok = TRUE;

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "true" but found "TRUE"

Open

            $systemadm = TRUE;

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "false" but found "FALSE"

Open

            if (FALSE != $all_ok || in_array($modversion['category'], $ok_syscats)) {

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "false" but found "FALSE"

Open

    $all_ok = FALSE;

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "null" but found "NULL"

Open

        if (NULL !== $modid) {

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "false" but found "FALSE"

Open

    if (!is_writable(ICMS_COMPILE_PATH)) array_push($block['msg'], icms_core_Message::warning(sprintf(_WARNINNOTWRITEABLE, ICMS_COMPILE_PATH)), '', FALSE);

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "null" but found "NULL"

Open

    if (icms::$db === NULL) {

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "false" but found "FALSE"

Open

        array_push($block['msg'], icms_core_Message::error(sprintf(_WARNINSTALL2, ICMS_ROOT_PATH . '/install/'), '', FALSE));

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "false" but found "FALSE"

Open

    if (version_compare(phpversion(), '7.2', '<')) array_push($block['msg'], icms_core_Message::warning(sprintf(_OUTDATED_PHP, phpversion())), '', FALSE);

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "false" but found "FALSE"

Open

        array_push($block['msg'], icms_core_Message::error(sprintf(_WARNINWRITEABLE, ICMS_ROOT_PATH . '/mainfile.php'), '', FALSE));

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "false" but found "FALSE"

Open

    if (!is_writable(ICMS_UPLOAD_PATH)) array_push($block['msg'], icms_core_Message::warning(sprintf(_WARNINNOTWRITEABLE, ICMS_UPLOAD_PATH)), '', FALSE);

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "false" but found "FALSE"

Open

        array_push($block['msg'], icms_core_Message::error(_PROTECTOR_NOT_FOUND, '', FALSE));

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "false" but found "FALSE"

Open

    $all_ok = FALSE;

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

    if (count($block['sysmod']) > 0) return $block;

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

        if (is_object(icms::$user)) $admin_perm = $moduleperm_handler->checkRight('module_admin', $module->getVar('mid'), icms::$user->getGroups());

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

    if (count($block['mods']) > 0) return $block;

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

    if (!is_writable(ICMS_CACHE_PATH)) array_push($block['msg'], icms_core_Message::warning(sprintf(_WARNINNOTWRITEABLE, ICMS_CACHE_PATH)), '', FALSE);

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

    if (!is_writable(ICMS_UPLOAD_PATH)) array_push($block['msg'], icms_core_Message::warning(sprintf(_WARNINNOTWRITEABLE, ICMS_UPLOAD_PATH)), '', FALSE);

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

    if (!is_writable(ICMS_COMPILE_PATH)) array_push($block['msg'], icms_core_Message::warning(sprintf(_WARNINNOTWRITEABLE, ICMS_COMPILE_PATH)), '', FALSE);

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

    if (version_compare(phpversion(), '7.2', '<')) array_push($block['msg'], icms_core_Message::warning(sprintf(_OUTDATED_PHP, phpversion())), '', FALSE);

• Exclude checks
  • Disable engine
  • Disable check

The variable $config_handler is not named in camelCase.

Open

function b_system_admin_warnings_show() {
    $block = array();
    $block['msg'] = array();

    // Check if PDO is being used - if not, issue a warning, because it is being removed from PHP

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $config_handler is not named in camelCase.

Open

function b_system_admin_warnings_show() {
    $block = array();
    $block['msg'] = array();

    // Check if PDO is being used - if not, issue a warning, because it is being removed from PHP

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $ok_syscats is not named in camelCase.

Open

function b_system_admin_cp_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;
    $block['lang_insmodules'] = _AD_INSTALLEDMODULES;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $admin_dir is not named in camelCase.

Open

function b_system_admin_cp_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;
    $block['lang_insmodules'] = _AD_INSTALLEDMODULES;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $moduleperm_handler is not named in camelCase.

Open

function b_system_admin_modules_show() {
    $block['mods'] = array();
    $module_handler = icms::handler('icms_module');
    $moduleperm_handler = icms::handler('icms_member_groupperm');
    $criteria = new icms_db_criteria_Compo();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $moduleperm_handler is not named in camelCase.

Open

function b_system_admin_modules_show() {
    $block['mods'] = array();
    $module_handler = icms::handler('icms_module');
    $moduleperm_handler = icms::handler('icms_member_groupperm');
    $criteria = new icms_db_criteria_Compo();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $ok_syscats is not named in camelCase.

Open

function b_system_admin_cp_new_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $protector_is_active is not named in camelCase.

Open

function b_system_admin_warnings_show() {
    $block = array();
    $block['msg'] = array();

    // Check if PDO is being used - if not, issue a warning, because it is being removed from PHP

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $ok_syscats is not named in camelCase.

Open

function b_system_admin_cp_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;
    $block['lang_insmodules'] = _AD_INSTALLEDMODULES;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $all_ok is not named in camelCase.

Open

function b_system_admin_cp_new_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $all_ok is not named in camelCase.

Open

function b_system_admin_cp_new_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $module_handler is not named in camelCase.

Open

function b_system_admin_modules_show() {
    $block['mods'] = array();
    $module_handler = icms::handler('icms_module');
    $moduleperm_handler = icms::handler('icms_member_groupperm');
    $criteria = new icms_db_criteria_Compo();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $admin_dir is not named in camelCase.

Open

function b_system_admin_cp_new_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $ok_syscats is not named in camelCase.

Open

function b_system_admin_cp_new_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $admin_perm is not named in camelCase.

Open

function b_system_admin_modules_show() {
    $block['mods'] = array();
    $module_handler = icms::handler('icms_module');
    $moduleperm_handler = icms::handler('icms_member_groupperm');
    $criteria = new icms_db_criteria_Compo();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $sysperm_handler is not named in camelCase.

Open

function b_system_admin_cp_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;
    $block['lang_insmodules'] = _AD_INSTALLEDMODULES;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $admin_dir is not named in camelCase.

Open

function b_system_admin_cp_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;
    $block['lang_insmodules'] = _AD_INSTALLEDMODULES;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $mod_version_file is not named in camelCase.

Open

function b_system_admin_cp_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;
    $block['lang_insmodules'] = _AD_INSTALLEDMODULES;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $all_ok is not named in camelCase.

Open

function b_system_admin_cp_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;
    $block['lang_insmodules'] = _AD_INSTALLEDMODULES;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $admin_dir is not named in camelCase.

Open

function b_system_admin_cp_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;
    $block['lang_insmodules'] = _AD_INSTALLEDMODULES;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $mod_version_file is not named in camelCase.

Open

function b_system_admin_cp_new_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $sysperm_handler is not named in camelCase.

Open

function b_system_admin_cp_new_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $admin_dir is not named in camelCase.

Open

function b_system_admin_cp_new_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $admin_dir is not named in camelCase.

Open

function b_system_admin_cp_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;
    $block['lang_insmodules'] = _AD_INSTALLEDMODULES;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $protector_is_active is not named in camelCase.

Open

function b_system_admin_warnings_show() {
    $block = array();
    $block['msg'] = array();

    // Check if PDO is being used - if not, issue a warning, because it is being removed from PHP

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $module_handler is not named in camelCase.

Open

function b_system_admin_modules_show() {
    $block['mods'] = array();
    $module_handler = icms::handler('icms_module');
    $moduleperm_handler = icms::handler('icms_member_groupperm');
    $criteria = new icms_db_criteria_Compo();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $admin_perm is not named in camelCase.

Open

function b_system_admin_modules_show() {
    $block['mods'] = array();
    $module_handler = icms::handler('icms_module');
    $moduleperm_handler = icms::handler('icms_member_groupperm');
    $criteria = new icms_db_criteria_Compo();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $sysperm_handler is not named in camelCase.

Open

function b_system_admin_cp_new_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $mod_version_file is not named in camelCase.

Open

function b_system_admin_cp_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;
    $block['lang_insmodules'] = _AD_INSTALLEDMODULES;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $protector_is_active is not named in camelCase.

Open

function b_system_admin_warnings_show() {
    $block = array();
    $block['msg'] = array();

    // Check if PDO is being used - if not, issue a warning, because it is being removed from PHP

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $config_handler is not named in camelCase.

Open

function b_system_admin_warnings_show() {
    $block = array();
    $block['msg'] = array();

    // Check if PDO is being used - if not, issue a warning, because it is being removed from PHP

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $mod_version_file is not named in camelCase.

Open

function b_system_admin_cp_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;
    $block['lang_insmodules'] = _AD_INSTALLEDMODULES;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $admin_dir is not named in camelCase.

Open

function b_system_admin_cp_new_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $mod_version_file is not named in camelCase.

Open

function b_system_admin_cp_new_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $mod_version_file is not named in camelCase.

Open

function b_system_admin_cp_new_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $all_ok is not named in camelCase.

Open

function b_system_admin_cp_new_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $all_ok is not named in camelCase.

Open

function b_system_admin_cp_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;
    $block['lang_insmodules'] = _AD_INSTALLEDMODULES;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $sysperm_handler is not named in camelCase.

Open

function b_system_admin_cp_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;
    $block['lang_insmodules'] = _AD_INSTALLEDMODULES;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $all_ok is not named in camelCase.

Open

function b_system_admin_cp_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;
    $block['lang_insmodules'] = _AD_INSTALLEDMODULES;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $admin_dir is not named in camelCase.

Open

function b_system_admin_cp_new_show() {
    global $icmsTpl, $icmsConfig;

    $block['lang_cp'] = _CPHOME;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}