b_system_login_show accesses the super-global variable $_COOKIE.

Open

function b_system_login_show() {
    global $icmsConfig, $icmsConfigAuth, $icmsConfigUser;
    if (!icms::$user) {
        $block = array();
        $block['lang_username'] = _USERNAME;

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

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

b_system_online_show accesses the super-global variable $_SERVER.

Open

function b_system_online_show() {
    $online_handler = icms::handler('icms_core_Online');
    mt_srand((double) microtime() * 1000000);
    // set gc probabillity to 10% for now..
    if (mt_rand(1, 100) < 11) {

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

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

b_system_online_show accesses the super-global variable $_SERVER.

Open

function b_system_online_show() {
    $online_handler = icms::handler('icms_core_Online');
    mt_srand((double) microtime() * 1000000);
    // set gc probabillity to 10% for now..
    if (mt_rand(1, 100) < 11) {

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

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

b_system_comments_show accesses the super-global variable $GLOBALS.

Open

function b_system_comments_show($options) {
    $block = array();
    include_once ICMS_ROOT_PATH . '/include/comment_constants.php';
    $comment_handler = icms::handler('icms_data_comment');
    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('com_status', XOOPS_COMMENT_ACTIVE));

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

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

b_system_comments_show accesses the super-global variable $GLOBALS.

Open

function b_system_comments_show($options) {
    $block = array();
    include_once ICMS_ROOT_PATH . '/include/comment_constants.php';
    $comment_handler = icms::handler('icms_data_comment');
    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('com_status', XOOPS_COMMENT_ACTIVE));

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

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

b_system_notification_show accesses the super-global variable $_SERVER.

Open

function b_system_notification_show() {
    icms_loadLanguageFile('core', 'notification');
    // Notification must be enabled, and user must be logged in
    if (empty(icms::$user) || !icms_data_notification_Handler::isEnabled('block')) {
        return false; // do not display block

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

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

b_system_login_show accesses the super-global variable $_COOKIE.

Open

function b_system_login_show() {
    global $icmsConfig, $icmsConfigAuth, $icmsConfigUser;
    if (!icms::$user) {
        $block = array();
        $block['lang_username'] = _USERNAME;

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

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

b_system_info_show accesses the super-global variable $GLOBALS.

Open

function b_system_info_show($options) {
    global $icmsConfig;
    $block = array();
    if (!empty($options[3])) {
        $block['showgroups'] = TRUE;

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

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

b_system_info_show accesses the super-global variable $GLOBALS.

Open

function b_system_info_show($options) {
    global $icmsConfig;
    $block = array();
    if (!empty($options[3])) {
        $block['showgroups'] = TRUE;

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

class Foo {
    public function bar() {
        $name = $_POST['foo'];
    }
}

File system_blocks.php has 553 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php

// $Id: system_blocks.php 12313 2013-09-15 21:14:35Z skenow $
// ------------------------------------------------------------------------ //
// XOOPS - PHP Content Management System //

Function b_system_bookmarks_show has a Cognitive Complexity of 25 (exceeds 5 allowed). Consider refactoring.

Open

function b_system_bookmarks_show() {
    global $icmsConfig;
    $block = array();
    icms_loadLanguageFile('core', 'notification');
    // User must be logged in

• Read up
  Read up

Function b_system_info_show has a Cognitive Complexity of 23 (exceeds 5 allowed). Consider refactoring.

Open

function b_system_info_show($options) {
    global $icmsConfig;
    $block = array();
    if (!empty($options[3])) {
        $block['showgroups'] = TRUE;

• Read up
  Read up

Method b_system_comments_show has 50 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function b_system_comments_show($options) {
    $block = array();
    include_once ICMS_ROOT_PATH . '/include/comment_constants.php';
    $comment_handler = icms::handler('icms_data_comment');
    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('com_status', XOOPS_COMMENT_ACTIVE));

Function b_system_online_show has a Cognitive Complexity of 15 (exceeds 5 allowed). Consider refactoring.

Open

function b_system_online_show() {
    $online_handler = icms::handler('icms_core_Online');
    mt_srand((double) microtime() * 1000000);
    // set gc probabillity to 10% for now..
    if (mt_rand(1, 100) < 11) {

• Read up
  Read up

Method b_system_bookmarks_show has 45 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function b_system_bookmarks_show() {
    global $icmsConfig;
    $block = array();
    icms_loadLanguageFile('core', 'notification');
    // User must be logged in

Method b_system_online_show has 45 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function b_system_online_show() {
    $online_handler = icms::handler('icms_core_Online');
    mt_srand((double) microtime() * 1000000);
    // set gc probabillity to 10% for now..
    if (mt_rand(1, 100) < 11) {

Method b_system_social_show has 40 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function b_system_social_show($options) {
    $block = array();
    $block['provider'] = array();

    $i = 0;

Function b_system_main_show has a Cognitive Complexity of 13 (exceeds 5 allowed). Consider refactoring.

Open

function b_system_main_show() {
    global $icmsConfigUser;

    $block = array();
    $block['lang_home'] = _MB_SYSTEM_HOME;

• Read up
  Read up

Method b_system_info_show has 39 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function b_system_info_show($options) {
    global $icmsConfig;
    $block = array();
    if (!empty($options[3])) {
        $block['showgroups'] = TRUE;

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

Open

function b_system_newmembers_show($options) {
    global $icmsConfigUser;

    $block = array();
    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('level', 0, '>'));

Function b_system_comments_show has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

function b_system_comments_show($options) {
    $block = array();
    include_once ICMS_ROOT_PATH . '/include/comment_constants.php';
    $comment_handler = icms::handler('icms_data_comment');
    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('com_status', XOOPS_COMMENT_ACTIVE));

• Read up
  Read up

Method b_system_notification_show has 33 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function b_system_notification_show() {
    icms_loadLanguageFile('core', 'notification');
    // Notification must be enabled, and user must be logged in
    if (empty(icms::$user) || !icms_data_notification_Handler::isEnabled('block')) {
        return false; // do not display block

Function b_system_topposters_show has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

function b_system_topposters_show($options) {
    global $icmsConfigUser;

    $block = array();
    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('level', 0, '>'));

• Read up
  Read up

Function b_system_newmembers_show has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

function b_system_newmembers_show($options) {
    global $icmsConfigUser;

    $block = array();
    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('level', 0, '>'));

• Read up
  Read up

Function b_system_notification_show has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

function b_system_notification_show() {
    icms_loadLanguageFile('core', 'notification');
    // Notification must be enabled, and user must be logged in
    if (empty(icms::$user) || !icms_data_notification_Handler::isEnabled('block')) {
        return false; // do not display block

• Read up
  Read up

Method b_system_main_show has 31 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function b_system_main_show() {
    global $icmsConfigUser;

    $block = array();
    $block['lang_home'] = _MB_SYSTEM_HOME;

Method b_system_topposters_show has 31 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function b_system_topposters_show($options) {
    global $icmsConfigUser;

    $block = array();
    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('level', 0, '>'));

Function b_system_login_show has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

function b_system_login_show() {
    global $icmsConfig, $icmsConfigAuth, $icmsConfigUser;
    if (!icms::$user) {
        $block = array();
        $block['lang_username'] = _USERNAME;

• Read up
  Read up

Function b_system_topposters_edit has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

function b_system_topposters_edit($options) {
    $inputtag = "<input type='text' name='options[]' value='" . (int) $options[0] . "' />";
    $form = sprintf(_MB_SYSTEM_DISPLAY, $inputtag);
    $form .= "<br />" . _MB_SYSTEM_DISPLAYA . "&nbsp;<input type='radio' id='options[]' name='options[]' value='1'";
    if ($options[1] == 1) {

• Read up
  Read up

Function b_system_social_show has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

function b_system_social_show($options) {
    $block = array();
    $block['provider'] = array();

    $i = 0;

• Read up
  Read up

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

Open

function b_system_newmembers_show($options) {
    global $icmsConfigUser;

    $block = array();
    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('level', 0, '>'));

• 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_bookmarks_show() has a Cyclomatic Complexity of 11. The configured cyclomatic complexity threshold is 10.

Open

function b_system_bookmarks_show() {
    global $icmsConfig;
    $block = array();
    icms_loadLanguageFile('core', 'notification');
    // User must be logged in

• 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 '$com' which will lead to PHP notices.

Open

        $com['id'] = $i;

• 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 '$com' which will lead to PHP notices.

Open

        $com['module'] = '<a href="' . ICMS_MODULES_URL . '/' . $modules[$mid]->getVar('dirname') . '/">' . $modules[$mid]->getVar('name') . '</a>';

• 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 '$com' which will lead to PHP notices.

Open

        $com['icon'] = ($com['icon'] != '') ? $com['icon'] : 'icon1.gif';

• 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 '$com' which will lead to PHP notices.

Open

        $com['icon'] = ($com['icon'] != '') ? $com['icon'] : 'icon1.gif';

• 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 '$section' which will lead to PHP notices.

Open

        $section['name'] = $category['name'];

• 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 '$section' which will lead to PHP notices.

Open

        $section['description'] = $category['description'];

• 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 '$com' which will lead to PHP notices.

Open

        unset($com);

• 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 '$com' which will lead to PHP notices.

Open

                $com['poster'] = '<a href="' . ICMS_URL . '/userinfo.php?uid=' . $comments[$i]->getVar('com_uid') . '">' . $poster->getVar('uname') . '</a>';

• 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 '$com' which will lead to PHP notices.

Open

                $com['poster'] = $GLOBALS['icmsConfig']['anonymous'];

• 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 '$com' which will lead to PHP notices.

Open

        $block['comments'][] = &$com;

• 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 '$com' which will lead to PHP notices.

Open

        $com['icon'] = htmlspecialchars($comments[$i]->getVar('com_icon'), ENT_QUOTES);

• 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 '$com' which will lead to PHP notices.

Open

            $com['poster'] = $GLOBALS['icmsConfig']['anonymous'];

• 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 '$section' which will lead to PHP notices.

Open

        $section['itemid'] = $category['item_id'];

• 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 '$section' which will lead to PHP notices.

Open

        $section['title'] = $category['title'];

• 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 '$section' which will lead to PHP notices.

Open

        $block['categories'][$category['name']] = $section;

• 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 '$section' which will lead to PHP notices.

Open

            $section['events'][$event['name']] = array('name' => $event['name'], 'title' => $event['title'], 'caption' => $event['caption'], 'description' => $event['description'], 'subscribed' => $subscribed);

• 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 '$com' which will lead to PHP notices.

Open

        $com['icon'] = ($com['icon'] != '') ? $com['icon'] : 'icon1.gif';

• 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 '$com' which will lead to PHP notices.

Open

        $com['title'] = '<a href="' . ICMS_MODULES_URL . '/' . $modules[$mid]->getVar('dirname') . '/' . $comment_config[$mid]['pageName'] . '?' . $comment_config[$mid]['itemName'] . '=' . $comments[$i]->getVar('com_itemid') . '&amp;com_id=' . $i . '&amp;com_rootid=' . $comments[$i]->getVar('com_rootid') . '&amp;' . htmlspecialchars($comments[$i]->getVar('com_exparams')) . '#comment' . $i . '">' . $comments[$i]->getVar('com_title') . '</a>';

• 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 '$com' which will lead to PHP notices.

Open

        $com['time'] = formatTimestamp($comments[$i]->getVar('com_created'), 'm');

• 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 '$section' which will lead to PHP notices.

Open

        $section['events'] = array();

• 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 '80', column '45').

Open

            $mytotal = $online_handler->getCount(new icms_db_criteria_Item('online_module', icms::$module->getVar('mid')));

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

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

Missing class import via use statement (line '197', column '22').

Open

        $criteria->add(new icms_db_criteria_Item('to_userid', icms::$user->getVar('uid')));

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

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

Missing class import via use statement (line '293', column '75').

Open

        $block['active'] = icms_conv_nr2local($member_handler->getUserCount(new icms_db_criteria_Item('level', 0, '>')));

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

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

Missing class import via use statement (line '352', column '45').

Open

    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('com_status', XOOPS_COMMENT_ACTIVE));

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

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

Missing class import via use statement (line '147', 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 '369', column '22').

Open

        $criteria->add(new icms_db_criteria_Item('com_modid', '(' . implode(',', $modIds) . ')', 'IN'));

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

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

Missing class import via use statement (line '292', column '77').

Open

        $block['inactive'] = icms_conv_nr2local($member_handler->getUserCount(new icms_db_criteria_Item('level', 0)));

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

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

Missing class import via use statement (line '376', column '45').

Open

    $modules = $module_handler->getObjects(new icms_db_criteria_Item('hascomments', 1), TRUE);

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

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

Missing class import via use statement (line '757', column '16').

Open

        $yesno = new icms_form_elements_Radioyn('', 'options[' . $i . ']', $option);

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

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

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

Open

    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('level', 0, '>'));

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

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

Missing class import via use statement (line '196', column '19').

Open

        $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('read_msg', 0));

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

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

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

Open

    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('hasmain', 1));

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

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

Missing class import via use statement (line '146', column '45').

Open

    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('hasmain', 1));

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

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

Missing class import via use statement (line '196', column '46').

Open

        $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('read_msg', 0));

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

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

Missing class import via use statement (line '361', column '22').

Open

    $criteria1->add(new icms_db_criteria_Item('gperm_groupid', '(' . implode(',', $gperm_groupid) . ')', 'IN'));

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

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

Missing class import via use statement (line '315', column '22').

Open

        $criteria->add(new icms_db_criteria_Item('rank', $options[$i], '<>'));

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

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

Missing class import via use statement (line '360', column '46').

Open

    $criteria1 = new icms_db_criteria_Compo(new icms_db_criteria_Item('gperm_name', 'module_read', '='));

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

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

Missing class import via use statement (line '291', column '79').

Open

        $block['registered'] = icms_conv_nr2local($member_handler->getUserCount(new icms_db_criteria_Item('level')));

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

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

Missing class import via use statement (line '360', column '19').

Open

    $criteria1 = new icms_db_criteria_Compo(new icms_db_criteria_Item('gperm_name', 'module_read', '='));

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

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

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

Open

    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('level', 0, '>'));

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

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

Missing class import via use statement (line '311', column '45').

Open

    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('level', 0, '>'));

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

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

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

Open

    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('com_status', XOOPS_COMMENT_ACTIVE));

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

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

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

Open

    $criteria->add(new icms_db_criteria_Item('weight', 0, '>'));

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

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

Missing class import via use statement (line '264', column '45').

Open

    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('level', 0, '>'));

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

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

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

Open

        } else {
            $block['users'][$i]['avatar'] = '';
        }

• 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_online_show uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

    } else {
        return 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_online_show uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

    } else {
        $uid = 0;
        $uname = '';
    }

• 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_comments_show'.

Open

    $comment_handler = icms::handler('icms_data_comment');

• 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_comments_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();
    }
}

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

Open

        } else {
            $block['users'][$i]['avatar'] = '';
        }

• 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_DataFilter' in method 'b_system_info_show'.

Open

                        $block['groups'][$i]['users'][] = array('id' => $userinfo['uid'], 'name' => icms_core_DataFilter::htmlSpecialChars($userinfo['uname']), 'msglink' => ' ', 'avatar' => ICMS_UPLOAD_URL . '/' . $userinfo['user_avatar']);

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

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

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

Open

        } else {
            $com['poster'] = $GLOBALS['icmsConfig']['anonymous'];
        }

• 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_topposters_show'.

Open

    $member_handler = icms::handler('icms_member');

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

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

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

Open

    } else {
        $online_handler->write($uid, $uname, time(), 0, $_SERVER['REMOTE_ADDR']);
    }

• 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_user_show'.

Open

        $pm_handler = icms::handler('icms_data_privmessage');

• 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_notification_show'.

Open

    $notification_handler = icms::handler('icms_data_notification');

• 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_bookmarks_show'.

Open

    $notification_handler = icms::handler('icms_data_notification');

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

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

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

Open

                } else {
                    if ($userinfo['user_viewemail']) {
                        $block['groups'][$i]['users'][] = array(
                            'id' => $userinfo['uid'],
                            'name' => icms_core_DataFilter::htmlSpecialChars($userinfo['uname']),

• 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_data_notification_Handler' in method 'b_system_notification_show'.

Open

    if (empty(icms::$user) || !icms_data_notification_Handler::isEnabled('block')) {

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

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

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

Open

            } else {
                $block['users'][$i]['avatar'] = '';
            }

• 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_comments_show uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                $com['poster'] = $GLOBALS['icmsConfig']['anonymous'];
            }

• 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_online_show uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                $guests++ ;
            }

• 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_main_show uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                $block['modules'][$i]['sublinks'] = 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' in method 'b_system_newmembers_show'.

Open

    $member_handler = icms::handler('icms_member');

• 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_comments_show'.

Open

    $member_handler = icms::handler('icms_member');

• 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_bookmarks_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_DataFilter' in method 'b_system_info_show'.

Open

                        'name' => icms_core_DataFilter::htmlSpecialChars($userinfo['uname']),

• 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_DataFilter' in method 'b_system_info_show'.

Open

                            'name' => icms_core_DataFilter::htmlSpecialChars($userinfo['uname']),

• 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_comments_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' in method 'b_system_online_show'.

Open

    $online_handler = icms::handler('icms_core_Online');

• 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_main_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();
    }
}

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

Open

    } else {
        $block['showgroups'] = 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_bookmarks_show uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                $item_info = array('name' => '[' . _NOT_NAMENOTAVAILABLE . ']', 'url' => '');
            }

• 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_main_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_DataFilter' in method 'b_system_info_show'.

Open

                    $block['groups'][$i]['name'] = icms_core_DataFilter::htmlSpecialChars($userinfo['groupname']);

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

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

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

Open

            } else {
                $block['users'][$i]['avatar'] = '';
            }

• 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_info_show uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                    } else {
                        $block['groups'][$i]['users'][] = array('id' => $userinfo['uid'], 'name' => icms_core_DataFilter::htmlSpecialChars($userinfo['uname']), 'msglink' => ' ', 'avatar' => ICMS_UPLOAD_URL . '/' . $userinfo['user_avatar']);
                    }

• 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 '$icmsConfig'.

Open

    global $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 $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 '$icmsConfigAuth'.

Open

    global $icmsConfig, $icmsConfigAuth, $icmsConfigUser;

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

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

FIXME found

Open

    // FIXME: better or more standardized way to do this?

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

        $uname = icms::$user->getVar('uname');

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_MEMBERS (Did you mean $members)

Open

        $block['lang_members'] = _MEMBERS;

• Exclude checks
  • Disable engine
  • Disable check

When fetching an array index from a value of type array, found an array index of type null, but expected the index to be of the non-nullable type int|string

Open

            $block['unamevalue'] = $_COOKIE[$icmsConfig['usercookie']];

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_ONLINEPHRASEX

Open

            $block['online_total'] .= ' (' . sprintf(_ONLINEPHRASEX, $mytotal, icms::$module->getVar('name')) . ')';

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_USERNAME

Open

        $block['lang_username'] = _USERNAME;

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('hasmain', 1));

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_LOGIN

Open

        $block['lang_login'] = _LOGIN;

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property module from undeclared class \icms

Open

            $block['online_total'] .= ' (' . sprintf(_ONLINEPHRASEX, $mytotal, icms::$module->getVar('name')) . ')';

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

    $member_handler = icms::handler('icms_member');

• 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 __construct from undeclared class \icms_db_criteria_Compo

Open

    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('hasmain', 1));

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property module from undeclared class \icms

Open

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

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_GUESTS (Did you mean $guests)

Open

        $block['lang_guests'] = _GUESTS;

• Exclude checks
  • Disable engine
  • Disable check

Call to method htmlSpecialChars from undeclared class \icms_core_DataFilter

Open

                    $block['groups'][$i]['name'] = icms_core_DataFilter::htmlSpecialChars($userinfo['groupname']);

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

    if (!icms::$user) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_NO

Open

    $form .= '&nbsp;<input type="radio" name="options[0]" value="0"' . $chk . ' />' . _NO;

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

        $uid = icms::$user->getVar('uid');

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property module from undeclared class \icms

Open

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

• 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('to_userid', icms::$user->getVar('uid')));

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

    $online_handler = icms::handler('icms_core_Online');

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (seed) is float but \mt_srand() takes int

Open

    mt_srand((double) microtime() * 1000000);

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

            $mytotal = $online_handler->getCount(new icms_db_criteria_Item('online_module', icms::$module->getVar('mid')));

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_PASSWORD

Open

        $block['lang_password'] = _PASSWORD;

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Compo

Open

        $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('read_msg', 0));

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_MORE

Open

        $block['lang_more'] = _MORE;

• 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

Reference to static property module from undeclared class \icms

Open

        $online_handler->write($uid, $uname, time(), icms::$module->getVar('mid'), $_SERVER['REMOTE_ADDR']);

• Exclude checks
  • Disable engine
  • Disable check

Suspicious type null of a variable or expression used to build a string. (Expected type to be able to cast to a string)

Open

            $block['sslloginlink'] = "<a href=\"javascript:openWithSelfMain('" . $icmsConfig['sslloginlink'] . "', 'ssllogin', 300, 200);\">" . _MB_SYSTEM_SECURE . "</a>";

• 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

Reference to static property user from undeclared class \icms

Open

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

• Exclude checks
  • Disable engine
  • Disable check

Call to method setLimit from undeclared class \icms_db_criteria_Compo

Open

    $criteria->setLimit((int) $options[0]);

• Exclude checks
  • Disable engine
  • Disable check

Saw an @param annotation for options, but the param list of function b_system_notification_show() : array; is empty

Open

 * @param array $options The block options

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_YES

Open

    $form .= " />&nbsp;" . _YES . "<input type='radio' id='options[1]' name='options[1]' value='0'";

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_NO

Open

    $form .= " /> " . _NO . "";

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_YES

Open

    $form .= "<input type='radio' name='options[3]' value='1'" . $chk . " />&nbsp;" . _YES . "";

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_PROFILE

Open

                $members .= ' <a href="' . ICMS_URL . '/userinfo.php?uid=' . $onlines[$i]['online_uid'] . '" title="' . $onlines[$i]['online_uname'] . '\'s ' . _PROFILE . '">' . $onlines[$i]['online_uname'] . '</a>, ';

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property module from undeclared class \icms

Open

            $mytotal = $online_handler->getCount(new icms_db_criteria_Item('online_module', icms::$module->getVar('mid')));

• 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('to_userid', icms::$user->getVar('uid')));

• Exclude checks
  • Disable engine
  • Disable check

Invalid offset "usercookie" of array type array{template_set:mixed}

Open

        if (isset($_COOKIE[$icmsConfig['usercookie']])) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_CLOSE

Open

    $block['lang_close'] = _CLOSE;

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('com_status', XOOPS_COMMENT_ACTIVE));

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property module from undeclared class \icms

Open

            if ((count($sublinks) > 0) && (!empty(icms::$module)) && ($i == icms::$module->getVar('mid'))) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_ONLINEPHRASE

Open

        $block['online_total'] = sprintf(_ONLINEPHRASE, $total);

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

        $block['uid'] = icms::$user->getVar('uid');

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

        $block['inactive'] = icms_conv_nr2local($member_handler->getUserCount(new icms_db_criteria_Item('level', 0)));

• 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 setSort from undeclared class \icms_db_criteria_Compo

Open

    $criteria->setSort('user_regdate');

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Compo

Open

    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('level', 0, '>'));

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

    $notification_handler = icms::handler('icms_data_notification');

• Exclude checks
  • Disable engine
  • Disable check

Call to method setOrder from undeclared class \icms_db_criteria_Compo

Open

    $criteria->setOrder('DESC');

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $section['name'] = $category['name'];

• 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('com_modid', '(' . implode(',', $modIds) . ')', 'IN'));

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_NOT_UPDATENOW

Open

    $block['submit_button'] = _NOT_UPDATENOW;

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_YES

Open

    $form .= " />&nbsp;" . _YES . "<input type='radio' id='options[2]' name='options[2]' value='0'";

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

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

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

    $notifications = &$notification_handler->getByUser(icms::$user->getVar('uid'));

• Exclude checks
  • Disable engine
  • Disable check

When appending to a value of type array{}, found an array access index of type null, but expected the index to be of type int|string

Open

            $block[$module_name][] = array('name' => $item_info['name'], 'url' => $item_info['url']);

• Exclude checks
  • Disable engine
  • Disable check

Invalid offset "usercookie" of array type array{template_set:mixed}

Open

            $block['unamevalue'] = $_COOKIE[$icmsConfig['usercookie']];

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

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

• 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 handler from undeclared class \icms

Open

        $pm_handler = icms::handler('icms_data_privmessage');

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

        $criteria->add(new icms_db_criteria_Item('to_userid', icms::$user->getVar('uid')));

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property xoopsDB from undeclared class \icms

Open

        if (icms::$xoopsDB->getRowsNum($result) > 0) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method setSort from undeclared class \icms_db_criteria_Compo

Open

    $criteria->setSort('posts');

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

    $comment_handler = icms::handler('icms_data_comment');

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

    $criteria1 = new icms_db_criteria_Compo(new icms_db_criteria_Item('gperm_name', 'module_read', '='));

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_NOT_NAMENOTAVAILABLE

Open

                $item_info = array('name' => '[' . _NOT_NAMENOTAVAILABLE . ']', 'url' => '');

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

    $member_handler = icms::handler('icms_member');

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

    $member_handler = icms::handler('icms_member');

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

            if (!empty($event['admin_only']) && !icms::$user->isAdmin(icms::$module->getVar('mid'))) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

    $criteria1->add(new icms_db_criteria_Item('gperm_groupid', '(' . implode(',', $gperm_groupid) . ')', 'IN'));

• Exclude checks
  • Disable engine
  • Disable check

Invalid offset "language" of array type array{template_set:mixed}

Open

    $block['current_language'] = $icmsConfig['language'];

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_YES

Open

    $form .= " />&nbsp;" . _YES . "<input type='radio' id='options[]' name='options[]' value='0'";

• 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('weight', 0, '>'));

• Exclude checks
  • Disable engine
  • Disable check

null passed to foreach instead of array

Open

    foreach ($icmsConfig['theme_set_allowed'] as $theme) {

• Exclude checks
  • Disable engine
  • Disable check

Invalid offset "theme_set" of array type array{template_set:mixed}

Open

        $block['theme_select'][$theme]['current'] = $theme == $icmsConfig['theme_set'];

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_form_elements_Radioyn

Open

        $yesno = new icms_form_elements_Radioyn('', 'options[' . $i . ']', $option);

• Exclude checks
  • Disable engine
  • Disable check

Invalid offset "use_ssl" of array type array{template_set:mixed}

Open

        if ($icmsConfig['use_ssl'] == 1 && $icmsConfig['sslloginlink'] != '') {

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('level', 0, '>'));

• Exclude checks
  • Disable engine
  • Disable check

Invalid offset "sslloginlink" of array type array{template_set:mixed}

Open

            $block['sslloginlink'] = "<a href=\"javascript:openWithSelfMain('" . $icmsConfig['sslloginlink'] . "', 'ssllogin', 300, 200);\">" . _MB_SYSTEM_SECURE . "</a>";

• 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('weight', 0, '>'));

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

        $block['registered'] = icms_conv_nr2local($member_handler->getUserCount(new icms_db_criteria_Item('level')));

• Exclude checks
  • Disable engine
  • Disable check

Returning type false but b_system_notification_show() is declared to return array

Open

        return false;

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property module from undeclared class \icms

Open

        $subscribed_events = $notification_handler->getSubscribedEvents($category['name'], $category['item_id'], icms::$module->getVar('mid'), icms::$user->getVar('uid'));

• 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('rank', $options[$i], '<>'));

• 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

Invalid offset "theme_set_allowed" of array type array{template_set:mixed}

Open

    foreach ($icmsConfig['theme_set_allowed'] as $theme) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property xoopsDB from undeclared class \icms

Open

        $result = icms::$xoopsDB->query("SELECT u.uid, u.uname, u.email, u.user_viewemail, u.user_avatar, g.name AS groupname FROM " . icms::$xoopsDB->prefix("groups_users_link") . " l LEFT JOIN " . icms::$xoopsDB->prefix("users") . " u ON l.uid=u.uid LEFT JOIN " . icms::$xoopsDB->prefix("groups") . " g ON l.groupid=g.groupid WHERE g.group_type='Admin' ORDER BY l.groupid, u.uid");

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Compo

Open

    $criteria1 = new icms_db_criteria_Compo(new icms_db_criteria_Item('gperm_name', 'module_read', '='));

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

        $block['active'] = icms_conv_nr2local($member_handler->getUserCount(new icms_db_criteria_Item('level', 0, '>')));

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

    if (empty(icms::$user) || !icms_data_notification_Handler::isEnabled('block')) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method setLimit from undeclared class \icms_db_criteria_Compo

Open

    $criteria->setLimit($limit);

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property module from undeclared class \icms

Open

            if (!empty($event['admin_only']) && !icms::$user->isAdmin(icms::$module->getVar('mid'))) {

• Exclude checks
  • Disable engine
  • Disable check

Returning type false but b_system_notification_show() is declared to return array

Open

        return false; // do not display block

• Exclude checks
  • Disable engine
  • Disable check

Returning type false but b_system_bookmarks_show() is declared to return array

Open

        return FALSE; // do not display block

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

    $notification_handler = icms::handler('icms_data_notification');

• Exclude checks
  • Disable engine
  • Disable check

Call to method setOrder from undeclared class \icms_db_criteria_Compo

Open

    $criteria->setOrder('DESC');

• 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 setLimit from undeclared class \icms_db_criteria_Compo

Open

    $criteria->setLimit($limit);

• 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('com_modid', '(' . implode(',', $modIds) . ')', 'IN'));

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        $com['module'] = '<a href="' . ICMS_MODULES_URL . '/' . $modules[$mid]->getVar('dirname') . '/">' . $modules[$mid]->getVar('name') . '</a>';

• 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 __construct from undeclared class \icms_db_criteria_Item

Open

        $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('read_msg', 0));

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property xoopsDB from undeclared class \icms

Open

            while ($userinfo = icms::$xoopsDB->fetchArray($result)) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method htmlSpecialChars from undeclared class \icms_core_DataFilter

Open

                        'name' => icms_core_DataFilter::htmlSpecialChars($userinfo['uname']),

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('level', 0, '>'));

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Compo

Open

    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('com_status', XOOPS_COMMENT_ACTIVE));

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

    $gperm_groupid = is_object(icms::$user) ? icms::$user->getGroups() : array(XOOPS_GROUP_ANONYMOUS);

• Exclude checks
  • Disable engine
  • Disable check

Call to method add from undeclared class \icms_db_criteria_Compo

Open

    $criteria1->add(new icms_db_criteria_Item('gperm_groupid', '(' . implode(',', $gperm_groupid) . ')', 'IN'));

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

    $modules = $module_handler->getObjects(new icms_db_criteria_Item('hascomments', 1), TRUE);

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_NO

Open

    $form .= " /> " . _NO . "";

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_NO

Open

    $form .= " /> " . _NO . "";

• Exclude checks
  • Disable engine
  • Disable check

Invalid offset "sslloginlink" of array type array{template_set:mixed}

Open

        if ($icmsConfig['use_ssl'] == 1 && $icmsConfig['sslloginlink'] != '') {

• Exclude checks
  • Disable engine
  • Disable check

Call to method htmlSpecialChars from undeclared class \icms_core_DataFilter

Open

                        $block['groups'][$i]['users'][] = array('id' => $userinfo['uid'], 'name' => icms_core_DataFilter::htmlSpecialChars($userinfo['uname']), 'msglink' => ' ', 'avatar' => ICMS_UPLOAD_URL . '/' . $userinfo['user_avatar']);

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Compo

Open

    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('level', 0, '>'));

• 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('rank', $options[$i], '<>'));

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property security from undeclared class \icms

Open

    $block['notification_token'] = icms::$security->createToken();

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_NO

Open

    $form .= "&nbsp;<input type='radio' name='options[3]' value='0'" . $chk . " />" . _NO . "";

• 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

Call to method htmlSpecialChars from undeclared class \icms_core_DataFilter

Open

                            'name' => icms_core_DataFilter::htmlSpecialChars($userinfo['uname']),

• Exclude checks
  • Disable engine
  • Disable check

Call to method setOrder from undeclared class \icms_db_criteria_Compo

Open

    $criteria->setOrder('DESC');

• Exclude checks
  • Disable engine
  • Disable check

Call to method setSort from undeclared class \icms_db_criteria_Compo

Open

    $criteria->setSort('com_created');

• Exclude checks
  • Disable engine
  • Disable check

Call to method isEnabled from undeclared class \icms_data_notification_Handler

Open

    if (empty(icms::$user) || !icms_data_notification_Handler::isEnabled('block')) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

        $subscribed_events = $notification_handler->getSubscribedEvents($category['name'], $category['item_id'], icms::$module->getVar('mid'), icms::$user->getVar('uid'));

• Exclude checks
  • Disable engine
  • Disable check

Invalid offset "theme_set" of array type array{template_set:mixed}

Open

    $block['current_theme'] = $icmsConfig['theme_set'];

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_YES

Open

    $form .= "<input type='radio' name='options[0]' value='1'" . $chk . " />&nbsp;" . _YES;

• Exclude checks
  • Disable engine
  • Disable check

Call to method render from undeclared class \icms_form_elements_Radioyn

Open

        $form .= '<tr><td width="25%">' . constant('_MB_SYSTEM_SOCIAL_PROVIDER_' . $i) . '</td><td>' . $yesno->render() . '</td></tr>';

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        if ($options[1] == 1) {
            if ($topposters[$i]->getVar('user_avatar') && $topposters[$i]->getVar('user_avatar') != 'blank.gif' && $topposters[$i]->getVar('user_avatar') != '') {
                $block['users'][$i]['avatar'] = ICMS_UPLOAD_URL . '/' . $topposters[$i]->getVar('user_avatar');
            } elseif ($icmsConfigUser['avatar_allow_gravatar'] == 1) {
                $block['users'][$i]['avatar'] = $topposters[$i]->gravatar('G', $icmsConfigUser['avatar_width']);

• Read up
  Read up

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

Open

        if ($options[1] == 1) {
            if ($newmembers[$i]->getVar('user_avatar') && $newmembers[$i]->getVar('user_avatar') != 'blank.gif' && $newmembers[$i]->getVar('user_avatar') != '') {
                $block['users'][$i]['avatar'] = ICMS_UPLOAD_URL . '/' . $newmembers[$i]->getVar('user_avatar');
            } elseif ($icmsConfigUser['avatar_allow_gravatar'] == 1) {
                $block['users'][$i]['avatar'] = $newmembers[$i]->gravatar('G', $icmsConfigUser['avatar_width']);

• Read up
  Read up

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

Open

            if (!empty($not_config['lookup_file'])) {
                $lookup_file = ICMS_MODULES_PATH . '/' . $module->getVar('dirname') . '/' . $not_config['lookup_file'];
                if (file_exists($lookup_file)) {
                    include_once $lookup_file;
                    if (!empty($not_config['lookup_func']) && function_exists($not_config['lookup_func'])) {

• Read up
  Read up

Avoid variables with short names like $i. Configured minimum length is 3.

Open

            $i = 0;

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

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

Avoid variables with short names like $i. Configured minimum length is 3.

Open

    $i = 0;

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

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

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

Open

    if (FALSE !== $onlines) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    $perms = $moduleperm_handler->getObjects($criteria1, TRUE);

• Exclude checks
  • Disable engine
  • Disable check

Opening brace should be on the same line as the declaration

Open

{

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after "=>"; 0 found

Open

    foreach ($language_names as $namekey=>$language_name){

• Exclude checks
  • Disable engine
  • Disable check

Space found before closing bracket of FOR loop

Open

        for ($i = 0; $i < $total; $i++ ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space before "=>"; 0 found

Open

    foreach ($language_tags as $tagkey=>$language_tag) {

• Exclude checks
  • Disable engine
  • Disable check

Space found before closing bracket of FOR loop

Open

    for ($i = 0; $i < $count; $i++ ) {

• Exclude checks
  • Disable engine
  • Disable check

Space found before closing bracket of FOR loop

Open

    for ($i = 0; $i < $count; $i++ ) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

    for ($i = 0; $i < $count; $i++ ) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    $modules = $module_handler->getObjects(new icms_db_criteria_Item('hascomments', 1), TRUE);

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    $modules = $module_handler->getObjects($criteria, TRUE);

• Exclude checks
  • Disable engine
  • Disable check

Blank line found at end of control structure

Open

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space before "=>"; 0 found

Open

    foreach ($language_names as $namekey=>$language_name){

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open

        for ($i = 2; $i < $size; $i++ ) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    $comments = $comment_handler->getObjects($criteria, TRUE);

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        return FALSE; // do not display block

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces before closing bracket; 1 found

Open