displayUsers accesses the super-global variable $_GET.

Open

function displayUsers() {
    global $icmsConfig, $icmsModule, $icmsConfigUser;
    $userstart = isset($_GET['userstart']) ? (int) $_GET['userstart'] : 0;

    icms_cp_header();

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

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

displayUsers accesses the super-global variable $_GET.

Open

function displayUsers() {
    global $icmsConfig, $icmsModule, $icmsConfigUser;
    $userstart = isset($_GET['userstart']) ? (int) $_GET['userstart'] : 0;

    icms_cp_header();

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

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

Function updateUser has a Cognitive Complexity of 31 (exceeds 5 allowed). Consider refactoring.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

• Read up
  Read up

Method updateUser has 31 arguments (exceeds 4 allowed). Consider refactoring.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

Method displayUsers has 88 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function displayUsers() {
    global $icmsConfig, $icmsModule, $icmsConfigUser;
    $userstart = isset($_GET['userstart']) ? (int) $_GET['userstart'] : 0;

    icms_cp_header();

Method updateUser has 76 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

File users.php has 276 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php
// $Id: users.php 12313 2013-09-15 21:14:35Z skenow $
//  ------------------------------------------------------------------------ //
//                XOOPS - PHP Content Management System                      //
//                    Copyright (c) 2000 XOOPS.org                           //

Method modifyUser has 60 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function modifyUser($user) {
    global $icmsConfig, $icmsModule;
    icms_cp_header();
    echo '<div class="CPbigTitle" style="background-image: url(' . ICMS_MODULES_URL . '/system/admin/users/images/users_big.png)">' . _MD_AM_USER . '</div><br />';
    $member_handler = icms::handler('icms_member');

Function synchronize has a Cognitive Complexity of 16 (exceeds 5 allowed). Consider refactoring.

Open

function synchronize($id, $type) {
    switch($type) {
        case 'user':
            // Array of tables from which to count 'posts'
            $tables = array();

• Read up
  Read up

Method synchronize has 32 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function synchronize($id, $type) {
    switch($type) {
        case 'user':
            // Array of tables from which to count 'posts'
            $tables = array();

Consider simplifying this complex logical expression.

Open

    if ($edituser->getVar('uname') != $uname && $member_handler->getUserCount(new icms_db_criteria_Item('uname', $uname)) > 0 || $edituser->getVar('login_name') != $login_name && $member_handler->getUserCount(new icms_db_criteria_Item('login_name', $login_name)) > 0) {
        icms_cp_header();
        echo '<div class="CPbigTitle" style="background-image: url(' . ICMS_MODULES_URL . '/system/admin/users/images/users_big.png)">' . _MD_AM_USER . '</div><br />';
        echo _AM_UNAME . ' ' . $uname . ' ' . _AM_ALREADY_EXISTS;
        icms_cp_footer();

The function updateUser has 31 parameters. Consider reducing the number of parameters to less than 10.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

• Exclude checks
  • Disable engine
  • Disable check

The function updateUser() has an NPath complexity of 1948. The configured NPath complexity threshold is 200.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

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

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

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

Open

function synchronize($id, $type) {
    switch($type) {
        case 'user':
            // Array of tables from which to count 'posts'
            $tables = array();

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

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

• 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;
            }
        }
    }
}

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

Open

    $submit_button = new icms_form_elements_Button('', 'submit', _AM_GO, 'submit');

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

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

Missing class import via use statement (line '58', column '49').

Open

    $usercount = $member_handler->getUserCount(new icms_db_criteria_Item('level', '-1', '!='));

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

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

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

Open

    $criteria->add(new icms_db_criteria_Item('level', '-1', '!='));

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

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

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

Open

    $submit_button = new icms_form_elements_Button('', 'submit', _AM_GO, 'submit');

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

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

Missing class import via use statement (line '90', 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 '106', column '20').

Open

    $fct_hidden = new icms_form_elements_Hidden('fct', 'users');

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

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

Missing class import via use statement (line '354', column '101').

Open

            $tables[] = array ('table_name' => 'xoopscomments', 'uid_column' => 'com_uid', 'criteria' => 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 '69', column '26').

Open

    $user_select_tray = new icms_form_elements_Tray(_AM_NICKNAME, '<br />');

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

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

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

Open

    $user_select = new icms_form_elements_Select('', 'uid');

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

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

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

Open

    $editform = new icms_form_Theme(_AM_EDEUSER, 'edituser', 'admin.php');

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

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

Missing class import via use statement (line '87', column '13').

Open

    $nav = new icms_view_PageNav($usercount, 200, $userstart, 'userstart', 'fct=users');

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

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

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

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

Open

    if ($edituser->getVar('uname') != $uname && $member_handler->getUserCount(new icms_db_criteria_Item('uname', $uname)) > 0 || $edituser->getVar('login_name') != $login_name && $member_handler->getUserCount(new icms_db_criteria_Item('login_name', $login_name)) > 0) {

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

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

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

Open

    $op_select = new icms_form_elements_Select('', 'op');

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

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

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

Open

    $editform = new icms_form_Theme(_AM_REMOVED_USERS, 'edituser', 'admin.php');

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

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

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

Open

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

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

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

Missing class import via use statement (line '97', column '26').

Open

    $user_select_tray = new icms_form_elements_Tray(_AM_NICKNAME, '<br />');

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

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

Missing class import via use statement (line '62', 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 '71', column '25').

Open

    $user_select_nav = new icms_form_elements_Label('', $nav->renderNav(4));

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

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

Missing class import via use statement (line '86', column '49').

Open

    $usercount = $member_handler->getUserCount(new icms_db_criteria_Item('level', '-1'));

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

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

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

Open

    $user_select = new icms_form_elements_Select('', 'uid');

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

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

Missing class import via use statement (line '99', column '25').

Open

    $user_select_nav = new icms_form_elements_Label('', $nav->renderNav(4));

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

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

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

Open

                $criteria->add (new icms_db_criteria_Item($table['uid_column'], $id));

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

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

Missing class import via use statement (line '263', column '211').

Open

    if ($edituser->getVar('uname') != $uname && $member_handler->getUserCount(new icms_db_criteria_Item('uname', $uname)) > 0 || $edituser->getVar('login_name') != $login_name && $member_handler->getUserCount(new icms_db_criteria_Item('login_name', $login_name)) > 0) {

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

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

Missing class import via use statement (line '310', column '20').

Open

            $icmspass = new icms_core_Password();

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

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

Missing class import via use statement (line '59', column '13').

Open

    $nav = new icms_view_PageNav($usercount, 200, $userstart, 'userstart', 'fct=users');

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

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

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

Open

    $fct_hidden = new icms_form_elements_Hidden('fct', 'users');

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

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

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

Open

    $op_select = new icms_form_elements_Select('', 'op');

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

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

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

Open

    } else {
        $edituser->setVar('name', $name);
        $edituser->setVar('uname', $uname);
        $edituser->setVar('login_name', $login_name);
        $edituser->setVar('email', $email);

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

Open

    } else {
        echo "<h4 style='text-align:" . _GLOBAL_LEFT . ";'>" . _AM_USERDONEXIT . "</h4>";
    }

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

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

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

Open

function synchronize($id, $type) {
    switch($type) {
        case 'user':
            // Array of tables from which to count 'posts'
            $tables = array();

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

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

Open

        } else {
            $signature = icms_core_DataFilter::checkVar($user_sig, 'html', 'input');
            $edituser->setVar('user_sig', $signature);
        }

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

Open

            $signature = icms_core_DataFilter::checkVar($user_sig, 'html', 'input');

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

Open

            $edituser->setVar('user_sig', icms_core_DataFilter::icms_substr($signature, 0, (int) $icmsConfigUser['sig_max_length']));

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

Open

            icms_core_Message::confirm(array('fct' => 'users', 'op' => 'reactivate', 'uid' => $user->getVar('uid')), 'admin.php', _AM_NOTACTIVE);

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

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 assigning values to variables in if clauses and the like (line '368', column '9').

Open

function synchronize($id, $type) {
    switch($type) {
        case 'user':
            // Array of tables from which to count 'posts'
            $tables = array();

• 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 assigning values to variables in if clauses and the like (line '363', column '9').

Open

function synchronize($id, $type) {
    switch($type) {
        case 'user':
            // Array of tables from which to count 'posts'
            $tables = array();

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

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

Open

        } else {
            if ($groups != array()) {
                $oldgroups = $edituser->getGroups();
                //If the edited user is the current user and the current user WAS in the webmaster's group and is NOT in the new groups array
                if ($edituser->getVar('uid') == icms::$user->getVar('uid') && (in_array(XOOPS_GROUP_ADMIN, $oldgroups)) && !(in_array(XOOPS_GROUP_ADMIN, $groups))) {

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

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 assigning values to variables in if clauses and the like (line '364', column '10').

Open

function synchronize($id, $type) {
    switch($type) {
        case 'user':
            // Array of tables from which to count 'posts'
            $tables = array();

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

Open

            $signature = strip_tags(icms_core_DataFilter::codeDecode($user_sig, 1));

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

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

Avoid unused local variables such as '$groups'.

Open

    $groups = array(XOOPS_GROUP_USERS);

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

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

Avoid unused local variables such as '$pass_expired_value'.

Open

        $pass_expired_value = $user->getVar('pass_expired') ? 1 : 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 '$uname_value'.

Open

    $uname_value = '';

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

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

Avoid unused local variables such as '$notify_mode_value'.

Open

    $notify_mode_value = XOOPS_NOTIFICATION_MODE_SENDALWAYS;

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

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

Avoid unused local variables such as '$form_title'.

Open

    $form_title = _AM_ADDUSER;

• 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, $icmsModule;

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

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

Avoid unused local variables such as '$icmsModule'.

Open

    global $icmsConfig, $icmsModule;

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

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

Avoid unused local variables such as '$temp'.

Open

        $temp = $user->getVar('theme');

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

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

Avoid unused local variables such as '$name_value'.

Open

    $name_value = '';

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

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

Avoid unused local variables such as '$uorder_value'.

Open

    $uorder_value = $icmsConfig['com_order'];

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

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

Avoid unused local variables such as '$language_value'.

Open

    $language_value = $icmsConfig['language'];

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

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

Avoid unused local variables such as '$timezone_value'.

Open

        $timezone_value = $user->getVar('timezone_offset');

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

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

Avoid unused local variables such as '$email_cbox_value'.

Open

    $email_cbox_value = 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 '$location_value'.

Open

    $location_value = '';

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

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

Avoid unused local variables such as '$url_value'.

Open

        $url_value = $user->getVar('url', 'E');

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

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

Avoid unused local variables such as '$op_value'.

Open

        $op_value = 'updateUser';

• 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, $icmsModule, $icmsConfigUser;

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

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

Avoid unused local variables such as '$icq_value'.

Open

    $icq_value = '';

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

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

Avoid unused local variables such as '$msnm_value'.

Open

    $msnm_value = '';

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

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

Avoid unused local variables such as '$login_name_value'.

Open

        $login_name_value = $user->getVar('login_name', 'E');

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

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

Avoid unused local variables such as '$yim_value'.

Open

        $yim_value = $user->getVar('user_yim', "E");

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

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

Avoid unused local variables such as '$form_title'.

Open

        $form_title = _AM_UPDATEUSER . ': ' . $user->getVar('uname');

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

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

Avoid unused local variables such as '$sig_cbox_value'.

Open

    $sig_cbox_value = 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 '$umode_value'.

Open

    $umode_value = $icmsConfig['com_mode'];

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

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

Avoid unused local variables such as '$bio_value'.

Open

    $bio_value = '';

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

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

Avoid unused local variables such as '$mailok_value'.

Open

    $mailok_value = 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 '$op_value'.

Open

    $op_value = 'addUser';

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

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

The function modifyUser() contains an exit expression.

Open

            exit();

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

class Foo {
    public function bar($param)  {
        if ($param === 42) {
            exit(23);
        }
    }
}

Avoid unused local variables such as '$email_value'.

Open

    $email_value = '';

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

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

Avoid unused local variables such as '$interest_value'.

Open

        $interest_value = $user->getVar('user_intrest', 'E');

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

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

Avoid unused local variables such as '$sig_cbox_value'.

Open

        $sig_cbox_value = ($user->getVar('attachsig') == 1) ? 1 : 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 '$uorder_value'.

Open

        $uorder_value = $user->getVar('uorder');

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

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

Avoid unused local variables such as '$notify_mode_value'.

Open

        $notify_mode_value = $user->getVar('notify_mode');

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

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

Avoid unused local variables such as '$rank_value'.

Open

        $rank_value = $user->rank(FALSE);

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

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

Avoid unused local variables such as '$language_value'.

Open

        $language_value = $user->getVar('language');

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

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

Avoid unused local variables such as '$icmsModule'.

Open

    global $icmsConfig, $icmsModule, $icmsConfigUser;

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

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

Avoid unused parameters such as '$theme'.

Open

                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,

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

class Foo
{
    private function bar($howdy)
    {
        // $howdy is not used
    }
}

Avoid unused local variables such as '$name_value'.

Open

        $name_value = $user->getVar('name', 'E');

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

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

Avoid unused local variables such as '$aim_value'.

Open

        $aim_value = $user->getVar('user_aim', "E");

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

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

Avoid unused local variables such as '$uid_value'.

Open

    $uid_value = '';

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

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

Avoid unused local variables such as '$email_value'.

Open

        $email_value = $user->getVar('email', 'E');

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

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

Avoid unused local variables such as '$occ_value'.

Open

        $occ_value = $user->getVar('user_occ', 'E');

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

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

Avoid unused local variables such as '$icq_value'.

Open

        $icq_value = $user->getVar('user_icq', 'E');

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

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

Avoid unused local variables such as '$umode_value'.

Open

        $umode_value = $user->getVar('umode');

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

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

Avoid unused local variables such as '$aim_value'.

Open

    $aim_value = '';

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

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

Avoid unused local variables such as '$yim_value'.

Open

    $yim_value = '';

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

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

Avoid unused local variables such as '$rank_value'.

Open

    $rank_value = 0;

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

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

Avoid unused parameters such as '$user_avatar'.

Open

                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,

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

class Foo
{
    private function bar($howdy)
    {
        // $howdy is not used
    }
}

Avoid unused local variables such as '$icmsModule'.

Open

    global $icmsConfig, $icmsModule, $icmsConfigUser;

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

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

Avoid unused local variables such as '$form_isedit'.

Open

    $form_isedit = FALSE;

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

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

Avoid unused local variables such as '$form_isedit'.

Open

        $form_isedit = 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 '$notify_method_value'.

Open

    $notify_method_value = XOOPS_NOTIFICATION_METHOD_PM;

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

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

Avoid unused local variables such as '$uname_value'.

Open

        $uname_value = $user->getVar('uname', 'E');

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

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

Avoid unused local variables such as '$location_value'.

Open

        $location_value = $user->getVar('user_from', 'E');

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

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

Avoid unused local variables such as '$sig_value'.

Open

        $sig_value = $user->getVar('user_sig', 'E');

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

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

Avoid unused local variables such as '$mailok_value'.

Open

        $mailok_value = $user->getVar('user_mailok', 'E');

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

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

The function updateUser() contains an exit expression.

Open

    exit();

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

class Foo {
    public function bar($param)  {
        if ($param === 42) {
            exit(23);
        }
    }
}

The function synchronize() contains an exit expression.

Open

            if (!$result = icms::$xoopsDB->query($sql)) {exit(_AM_CNGUSERID);}

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

class Foo {
    public function bar($param)  {
        if ($param === 42) {
            exit(23);
        }
    }
}

The function synchronize() contains an exit expression.

Open

            if (!$result = icms::$xoopsDB->query($sql)) {exit(sprintf(_AM_CNUUSER %s , $id));}

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

class Foo {
    public function bar($param)  {
        if ($param === 42) {
            exit(23);
        }
    }
}

Avoid unused local variables such as '$login_name_value'.

Open

    $login_name_value = '';

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

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

Avoid unused local variables such as '$interest_value'.

Open

    $interest_value = '';

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

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

Avoid unused local variables such as '$sig_value'.

Open

    $sig_value = '';

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

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

Avoid unused local variables such as '$msnm_value'.

Open

        $msnm_value = $user->getVar('user_msnm', 'E');

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

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

Avoid unused local variables such as '$notify_method_value'.

Open

        $notify_method_value = $user->getVar('notify_method');

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

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

The function updateUser() contains an exit expression.

Open

                exit();

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

class Foo {
    public function bar($param)  {
        if ($param === 42) {
            exit(23);
        }
    }
}

Avoid unused local variables such as '$url_value'.

Open

    $url_value = '';

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

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

Avoid unused local variables such as '$timezone_value'.

Open

    $timezone_value = $icmsConfig['default_TZ'];

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

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

Avoid unused local variables such as '$pass_expired_value'.

Open

    $pass_expired_value = 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 '$icmsConfigUser'.

Open

    global $icmsConfig, $icmsModule, $icmsConfigUser;

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

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

Avoid unused local variables such as '$occ_value'.

Open

    $occ_value = '';

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

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

Avoid unused local variables such as '$uid_value'.

Open

        $uid_value = $user->getVar('uid');

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

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

Avoid unused local variables such as '$groups'.

Open

        $groups = array_values($user->getGroups());

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

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

Avoid unused local variables such as '$email_cbox_value'.

Open

        $email_cbox_value = $user->getVar('user_viewemail') ? 1 : 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 '$bio_value'.

Open

        $bio_value = $user->getVar('bio', 'E');

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

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

The function synchronize() contains an exit expression.

Open

    exit();

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

class Foo {
    public function bar($param)  {
        if ($param === 42) {
            exit(23);
        }
    }
}

Call to method setOrder from undeclared class \icms_db_criteria_Compo

Open

    $criteria->setOrder('ASC');

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

Open

    if ($edituser->getVar('uname') != $uname && $member_handler->getUserCount(new icms_db_criteria_Item('uname', $uname)) > 0 || $edituser->getVar('login_name') != $login_name && $member_handler->getUserCount(new icms_db_criteria_Item('login_name', $login_name)) > 0) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_core_Password

Open

            $icmspass = new icms_core_Password();

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_form_elements_Tray

Open

    $user_select_tray = new icms_form_elements_Tray(_AM_NICKNAME, '<br />');

• Exclude checks
  • Disable engine
  • Disable check

Call to method addElement from undeclared class \icms_form_Theme

Open

    $editform->addElement($submit_button);

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property xoopsDB from undeclared class \icms

Open

                    if ($row = icms::$xoopsDB->fetchArray($result)) {$total_posts = $total_posts + $row['total'];}

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_view_PageNav

Open

    $nav = new icms_view_PageNav($usercount, 200, $userstart, 'userstart', 'fct=users');

• Exclude checks
  • Disable engine
  • Disable check

Call to method setLimit from undeclared class \icms_db_criteria_Compo

Open

    $criteria->setLimit(200);

• Exclude checks
  • Disable engine
  • Disable check

Call to method addElement from undeclared class \icms_form_elements_Tray

Open

    $user_select_tray->addElement($user_select);

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_form_elements_Button

Open

    $submit_button = new icms_form_elements_Button('', 'submit', _AM_GO, 'submit');

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    $umode_value = $icmsConfig['com_mode'];

• Exclude checks
  • Disable engine
  • Disable check

Parameter $type has undeclared type \str

Open

function synchronize($id, $type) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method add from undeclared class \icms_db_criteria_Compo

Open

                if (!empty($table['criteria'])) {$criteria->add ($table['criteria']);}

• 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('level', '-1', '!='));

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

    $usercount = $member_handler->getUserCount(new icms_db_criteria_Item('level', '-1'));

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_form_Theme

Open

    $editform = new icms_form_Theme(_AM_REMOVED_USERS, 'edituser', 'admin.php');

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

Open

            . icms::$security->getTokenHTML() . "\n"

• Exclude checks
  • Disable engine
  • Disable check

Call to method setSort from undeclared class \icms_db_criteria_Compo

Open

    $criteria->setSort('uname');

• Exclude checks
  • Disable engine
  • Disable check

Call to method addElement from undeclared class \icms_form_elements_Tray

Open

    $user_select_tray->addElement($user_select_nav);

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    $timezone_value = $icmsConfig['default_TZ'];

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property xoopsDB from undeclared class \icms

Open

            if (!$result = icms::$xoopsDB->query($sql)) {exit(sprintf(_AM_CNUUSER %s , $id));}

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property xoopsDB from undeclared class \icms

Open

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

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

                if ($edituser->getVar('uid') == icms::$user->getVar('uid') && (in_array(XOOPS_GROUP_ADMIN, $oldgroups)) && !(in_array(XOOPS_GROUP_ADMIN, $groups))) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property xoopsDB from undeclared class \icms

Open

            $sql = "SELECT uid FROM " . icms::$xoopsDB->prefix('users') . "";

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

            $tables[] = array ('table_name' => 'xoopscomments', 'uid_column' => 'com_uid', 'criteria' => new icms_db_criteria_Item('com_status', XOOPS_COMMENT_ACTIVE));

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property xoopsDB from undeclared class \icms

Open

                if ($result = icms::$xoopsDB->query($sql)) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_view_PageNav

Open

    $nav = new icms_view_PageNav($usercount, 200, $userstart, 'userstart', 'fct=users');

• 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('level', '-1', '!='));

• Exclude checks
  • Disable engine
  • Disable check

Call to method addElement from undeclared class \icms_form_Theme

Open

    $editform->addElement($fct_hidden);

• Exclude checks
  • Disable engine
  • Disable check

Call to method addOptionArray from undeclared class \icms_form_elements_Select

Open

    $user_select->addOptionArray($member_handler->getUserList($criteria));

• Exclude checks
  • Disable engine
  • Disable check

Call to method addElement from undeclared class \icms_form_Theme

Open

    $editform->addElement($op_select);

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    $language_value = $icmsConfig['language'];

• 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

    || !icms::$user->isAdmin($icmsModule->getVar('mid'))

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_form_elements_Select

Open

    $user_select = new icms_form_elements_Select('', 'uid');

• Exclude checks
  • Disable engine
  • Disable check

Call to method setSort from undeclared class \icms_db_criteria_Compo

Open

    $criteria->setSort('uname');

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_form_elements_Button

Open

    $submit_button = new icms_form_elements_Button('', 'submit', _AM_GO, 'submit');

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_form_elements_Hidden

Open

    $fct_hidden = new icms_form_elements_Hidden('fct', 'users');

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_form_elements_Select

Open

    $user_select = new icms_form_elements_Select('', '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('level', '-1'));

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_form_elements_Select

Open

    $op_select = new icms_form_elements_Select('', 'op');

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_form_elements_Hidden

Open

    $fct_hidden = new icms_form_elements_Hidden('fct', 'users');

• Exclude checks
  • Disable engine
  • Disable check

Call to method codeDecode from undeclared class \icms_core_DataFilter

Open

            $signature = strip_tags(icms_core_DataFilter::codeDecode($user_sig, 1));

• Exclude checks
  • Disable engine
  • Disable check

Call to method encryptPass from undeclared class \icms_core_Password

Open

            $pass = $icmspass->encryptPass($pass);

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

Open

    $op_select = new icms_form_elements_Select('', 'op');

• Exclude checks
  • Disable engine
  • Disable check

Call to method display from undeclared class \icms_form_Theme

Open

    $editform->display();

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_form_elements_Label

Open

    $user_select_nav = new icms_form_elements_Label('', $nav->renderNav(4));

• Exclude checks
  • Disable engine
  • Disable check

Call to method renderNav from undeclared class \icms_view_PageNav

Open

    $user_select_nav = new icms_form_elements_Label('', $nav->renderNav(4));

• Exclude checks
  • Disable engine
  • Disable check

Call to method icms_substr from undeclared class \icms_core_DataFilter

Open

            $edituser->setVar('user_sig', icms_core_DataFilter::icms_substr($signature, 0, (int) $icmsConfigUser['sig_max_length']));

• Exclude checks
  • Disable engine
  • Disable check

Call to method renderWhere from undeclared class \icms_db_criteria_Compo

Open

                $sql = "SELECT COUNT(*) AS total FROM " . icms::$xoopsDB->prefix($table['table_name']) . ' ' . $criteria->renderWhere();

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property xoopsDB from undeclared class \icms

Open

            $sql = "UPDATE " . icms::$xoopsDB->prefix("users") . " SET posts = '". (int) $total_posts . "' WHERE uid = '". (int) $id . "'";

• Exclude checks
  • Disable engine
  • Disable check

Variable $icmsModule is undeclared

Open

    || !is_object($icmsModule)

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

    $usercount = $member_handler->getUserCount(new icms_db_criteria_Item('level', '-1', '!='));

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_form_Theme

Open

    $editform = new icms_form_Theme(_AM_EDEUSER, 'edituser', 'admin.php');

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_form_elements_Label

Open

    $user_select_nav = new icms_form_elements_Label('', $nav->renderNav(4));

• 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 checkVar from undeclared class \icms_core_DataFilter

Open

            $signature = icms_core_DataFilter::checkVar($user_sig, 'html', 'input');

• 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($table['uid_column'], $id));

• 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($table['uid_column'], $id));

• Exclude checks
  • Disable engine
  • Disable check

Call to method setLimit from undeclared class \icms_db_criteria_Compo

Open

    $criteria->setLimit(200);

• Exclude checks
  • Disable engine
  • Disable check

Call to method addElement from undeclared class \icms_form_Theme

Open

    $editform->addElement($op_select);

• Exclude checks
  • Disable engine
  • Disable check

Call to method setStart from undeclared class \icms_db_criteria_Compo

Open

    $criteria->setStart($userstart);

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_form_elements_Tray

Open

    $user_select_tray = new icms_form_elements_Tray(_AM_NICKNAME, '<br />');

• Exclude checks
  • Disable engine
  • Disable check

Call to method addOptionArray from undeclared class \icms_form_elements_Select

Open

    $op_select->addOptionArray(array('modifyUser'=>_AM_MODIFYUSER));

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \s

Open

            if (!$result = icms::$xoopsDB->query($sql)) {exit(sprintf(_AM_CNUUSER %s , $id));}

• 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 addOptionArray from undeclared class \icms_form_elements_Select

Open

    $user_select->addOptionArray($member_handler->getUserList($criteria));

• Exclude checks
  • Disable engine
  • Disable check

Call to method addElement from undeclared class \icms_form_Theme

Open

    $editform->addElement($user_select_tray);

• Exclude checks
  • Disable engine
  • Disable check

Call to method addElement from undeclared class \icms_form_Theme

Open

    $editform->addElement($fct_hidden);

• Exclude checks
  • Disable engine
  • Disable check

Call to method renderNav from undeclared class \icms_view_PageNav

Open

    $user_select_nav = new icms_form_elements_Label('', $nav->renderNav(4));

• Exclude checks
  • Disable engine
  • Disable check

Variable $icmsModule is undeclared

Open

    || !icms::$user->isAdmin($icmsModule->getVar('mid'))

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    $uorder_value = $icmsConfig['com_order'];

• Exclude checks
  • Disable engine
  • Disable check

Call to method addElement from undeclared class \icms_form_elements_Tray

Open

    $user_select_tray->addElement($user_select);

• Exclude checks
  • Disable engine
  • Disable check

Call to method addElement from undeclared class \icms_form_Theme

Open

    $editform->addElement($submit_button);

• Exclude checks
  • Disable engine
  • Disable check

Call to method confirm from undeclared class \icms_core_Message

Open

            icms_core_Message::confirm(array('fct' => 'users', 'op' => 'reactivate', 'uid' => $user->getVar('uid')), 'admin.php', _AM_NOTACTIVE);

• Exclude checks
  • Disable engine
  • Disable check

Call to method setOrder from undeclared class \icms_db_criteria_Compo

Open

    $criteria->setOrder('ASC');

• Exclude checks
  • Disable engine
  • Disable check

Call to method addElement from undeclared class \icms_form_elements_Tray

Open

    $user_select_tray->addElement($user_select_nav);

• Exclude checks
  • Disable engine
  • Disable check

Call to method addElement from undeclared class \icms_form_Theme

Open

    $editform->addElement($user_select_tray);

• Exclude checks
  • Disable engine
  • Disable check

Call to method addOptionArray from undeclared class \icms_form_elements_Select

Open

    $op_select->addOptionArray(array('modifyUser'=>_AM_MODIFYUSER, 'delUser'=>_AM_DELUSER));

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

Open

                $sql = "SELECT COUNT(*) AS total FROM " . icms::$xoopsDB->prefix($table['table_name']) . ' ' . $criteria->renderWhere();

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property xoopsDB from undeclared class \icms

Open

            if (!$result = icms::$xoopsDB->query($sql)) {exit(_AM_CNGUSERID);}

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (format) is int but \sprintf() takes string

Open

            if (!$result = icms::$xoopsDB->query($sql)) {exit(sprintf(_AM_CNUUSER %s , $id));}

• Exclude checks
  • Disable engine
  • Disable check

Argument 2 (type) is 'user' but \synchronize() takes \str defined at /code/htdocs/modules/system/admin/users/users.php:347

Open

                synchronize($id, "user");

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

Open

    $criteria->setStart($userstart);

• 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('level', '-1'));

• Exclude checks
  • Disable engine
  • Disable check

Call to method display from undeclared class \icms_form_Theme

Open

    $editform->display();

• Exclude checks
  • Disable engine
  • Disable check

The parameter $timezone_offset is not named in camelCase.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $user_mailok is not named in camelCase.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $user_aim is not named in camelCase.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $user_occ is not named in camelCase.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $user_viewemail is not named in camelCase.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $login_name is not named in camelCase.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $user_sig is not named in camelCase.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $user_avatar is not named in camelCase.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $user_yim is not named in camelCase.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

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

class ClassName {
    public function doSomething($user_name) {
    }
}

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

Open

function synchronize($id, $type) {

• 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;
        }
    }
}

The parameter $user_from is not named in camelCase.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $notify_mode is not named in camelCase.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $user_msnm is not named in camelCase.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $user_icq is not named in camelCase.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $user_intrest is not named in camelCase.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $notify_method is not named in camelCase.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $pass_expired is not named in camelCase.

Open

function updateUser($uid, $uname, $login_name, $name, $url, $email, $user_icq, $user_aim, $user_yim,
                    $user_msnm, $user_from, $user_occ, $user_intrest, $user_viewemail, $user_avatar,
                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,
                    $notify_mode, $timezone_offset, $user_mailok, $language,
                    $pass_expired, $groups = array()

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

class ClassName {
    public function doSomething($user_name) {
    }
}

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

Open

    $form_isedit = FALSE;

• Exclude checks
  • Disable engine
  • Disable check

Multi-line function declarations must define one parameter per line

Open

                    $notify_mode, $timezone_offset, $user_mailok, $language,

• Exclude checks
  • Disable engine
  • Disable check

Multi-line function declarations must define one parameter per line

Open

                    $pass_expired, $groups = array()

• Exclude checks
  • Disable engine
  • Disable check

Space found before comma in function call

Open

            if (!$result = icms::$xoopsDB->query($sql)) {exit(sprintf(_AM_CNUUSER %s , $id));}

• Exclude checks
  • Disable engine
  • Disable check

Multi-line function declaration not indented correctly; expected 4 spaces but found 20

Open

                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,

• Exclude checks
  • Disable engine
  • Disable check

Multi-line function declarations must define one parameter per line

Open

                    $user_sig, $attachsig, $theme, $pass, $pass2, $rank, $bio, $uorder, $umode, $notify_method,

• Exclude checks
  • Disable engine
  • Disable check

Multi-line function declarations must define one parameter per line

Open

                    $notify_mode, $timezone_offset, $user_mailok, $language,

• Exclude checks
  • Disable engine
  • Disable check

Multi-line function declaration not indented correctly; expected 0 spaces but found 20

Open

                    ) {

• Exclude checks
  • Disable engine
  • Disable check