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

Open

    function check_sql_union($sanitize = true) {
        if ($this->_done_union)
            return $this->_safe_union;
        else
            $this->_done_union = true;

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                    if (!$lang_updated && $config->getVar('conf_catid') == ICMS_CONF && $config->getVar('conf_name') == 'language') {
                        $icmsConfig['language'] = ${$config->getVar('conf_name')};
                        $lang_updated = TRUE;
                    }

Avoid deeply nested control flow statements.

Open

                    if (!$theme_updated && $config->getVar('conf_catid') == ICMS_CONF && $config->getVar('conf_name') == 'theme_set') {
                        $member_handler = icms::handler('icms_member');
                        $member_handler->updateUsersByField('theme', ${$config->getVar('conf_name')});
                        $theme_updated = TRUE;
                    }

Avoid deeply nested control flow statements.

Open

                    if ($counter == 1 && $currentpage > 5) {
                        $hiddenform .= "... ";
                    }

Method getForm has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function getForm($form_caption, $form_name, $form_action = FALSE, $submit_button_caption = _CO_ICMS_SUBMIT, $cancel_js_action = FALSE, $captcha = FALSE) {

Avoid deeply nested control flow statements.

Open

                            if ($template != '') {
                                $tplfile = &$tplfile_handler->find('default', 'block', $fblock['bid']);
                                if (count($tplfile) == 0) {
                                    $tplfile_new = &$tplfile_handler->create();
                                    $tplfile_new->setVar('tpl_module', $dirname);

Avoid deeply nested control flow statements.

Open

                                for ($k = 0; $k < $btcount; $k++ ) {
                                    if (!$tplfile_handler->delete($tplfiles[$k])) {
                                        $msgs[] = sprintf('&nbsp;&nbsp;<span style="color:#ff0000;">' . _MD_AM_BLOCK_TMPLT_DELETE_FAILED . '</span>', '<strong>' . $tplfiles[$k]->getVar('tpl_file') . '</strong>', '<strong>' . $tplfiles[$k]->getVar('tpl_id') . '</strong>');
                                    } else {
                                        $msgs[] = sprintf('&nbsp;&nbsp;' . _MD_AM_BLOCK_TMPLT_DELETED, '<strong>' . $tplfiles[$k]->getVar('tpl_file') . '</strong>', '<strong>' . $tplfiles[$k]->getVar('tpl_id') . '</strong>');

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

Open

    function get_bad_ips($with_jailed_time = false) {
        list($bad_ips_serialized) = @file(self::get_filepath4badips());
        $bad_ips = empty($bad_ips_serialized) ? array() : @unserialize($bad_ips_serialized);
        if (!is_array($bad_ips) || isset($bad_ips[0])) $bad_ips = array();

• Read up
  Read up

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

Open

function displayGroups() {
    icms_cp_header();
    echo '<div class="CPbigTitle" style="background-image: url(' . ICMS_MODULES_URL . '/system/admin/groups/images/groups_big.png)">' . _AM_EDITADG . '</div><br />';
    $member_handler = icms::handler('icms_member');
    $groups = $member_handler->getGroups();

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                    if (!$encryption_updated && $config->getVar('conf_catid') == ICMS_CONF_USER && $config->getVar('conf_name') == 'enc_type') {
                        if ($icmsConfig['closesite'] !== 1) {
                            $member_handler = icms::handler('icms_member');
                            $member_handler->updateUsersByField('pass_expired', 1);
                            $encryption_updated = TRUE;

Avoid deeply nested control flow statements.

Open

                    if (!$purifier_style_updated
                        && $config->getVar('conf_catid') == ICMS_CONF_PURIFIER
                        && $config->getVar('conf_name') == 'purifier_Filter_ExtractStyleBlocks'
                        ) {
                        if ($config->getVar('purifier_Filter_ExtractStyleBlocks') == 1) {

Avoid deeply nested control flow statements.

Open

                            for ($j = 0; $j < $btcount; $j++ ) {
                                if (!$tplfile_handler->delete($templates[$j])) {
                                    $msgs[] = sprintf('&nbsp;&nbsp;<span style="color:#ff0000;">' . _MD_AM_BLOCK_TMPLT_DELETE_FAILED . '</span>', $templates[$j]->getVar('tpl_file'), '<strong>' . icms_conv_nr2local($templates[$j]->getVar('tpl_id')) . '</strong>');
                                } else {
                                    $msgs[] = sprintf('&nbsp;&nbsp;' . _MD_AM_BLOCK_TMPLT_DELETED, '<strong>' . $templates[$j]->getVar('tpl_file') . '</strong>', '<strong>' . icms_conv_nr2local($templates[$j]->getVar('tpl_id')) . '</strong>');

Avoid deeply nested control flow statements.

Open

                            if ($template != '') {
                                $tplfile = &$tplfile_handler->create();
                                $tplfile->setVar('tpl_module', $dirname);
                                $tplfile->setVar('tpl_refid', (int) $newbid);
                                $tplfile->setVar('tpl_source', $content, TRUE);

Avoid deeply nested control flow statements.

Open

                    if (! $startmod_updated && $new_value != '--' && $config->getVar('conf_catid') == ICMS_CONF && $config->getVar('conf_name') == 'startpage') {
                        $moduleperm_handler = icms::handler('icms_member_groupperm');
                        $module_handler = icms::handler('icms_module');

                        foreach ($new_value as $k => $v) {

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

Open

    function checkData() {
        $ret = '';
        $error = array();

        if (empty($this->dbhost)) {

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                    if (0 === strpos($oldavatar_path, ICMS_UPLOAD_PATH) && is_file($oldavatar_path)) {
                        unlink($oldavatar_path);
                    }

Avoid deeply nested control flow statements.

Open

                        if (!empty($oldavatar) && preg_match("/^cavt/", strtolower($oldavatar))) {
                            $avatars =& $avt_handler->getObjects(new icms_db_criteria_Item('avatar_file', $oldavatar));
                            if (!empty($avatars) && count($avatars) == 1 && is_object($avatars[0])) {
                                $avt_handler->delete($avatars[0]);
                                $oldavatar_path = str_replace("\\", "/", realpath(ICMS_UPLOAD_PATH . '/' . $oldavatar));

Avoid deeply nested control flow statements.

Open

                    if ($counter == $totalpages && $currentpage < $totalpages - 4) {
                        $hiddenform .= "... ";
                    }

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

Open

        function __construct() {
            global $icmsConfig;

            // language file
            if (defined('ICMS_ROOT_PATH') && !empty($icmsConfig['language']) && !strstr($icmsConfig['language'], '/')) {

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                            if (empty($newbid)) {
                                $newbid = icms::$xoopsDB->getInsertId();
                            }