ImpressCMS/impresscms

Consider simplifying this complex logical expression.

Open

                if ( matcher[ expando ] ) {
                    j = ++i;
                    for ( ; j < len; j++ ) {
                        if ( Expr.relative[ tokens[j].type ] ) {
                            break;

Consider simplifying this complex logical expression.

Open

            if (child.className && (cn = ' ' + child.className + ' ') && (cn.include(className) ||
                (classNames && classNames.all(function(name) {
                    return !name.toString().blank() && cn.include(' ' + name + ' ');
                }))))
                elements.push(Element.extend(child));

Consider simplifying this complex logical expression.

Open

    if(defined('SID') && SID && (!isset($_COOKIE[session_name()]) || ($icmsConfig['use_mysession'] && $icmsConfig['session_name'] != '' && !isset($_COOKIE[$icmsConfig['session_name']]))))
    {
        if(!strstr($url, '?')) {$url .= '?' . SID;}
        else {$url .= '&'.SID;}
    }

Consider simplifying this complex logical expression.

Open

if (!empty($_POST)) {
    if (isset($_POST['com_dopost'])) {
        $op = 'post';
    } elseif (isset($_POST['com_dopreview'])) {
        $op = 'preview';

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();

Consider simplifying this complex logical expression.

Open

                    if ($password == $username || $password == icms_core_DataFilter::utf8_strrev($username, TRUE)
                        || strripos($password, $username) === TRUE || $password == $login_name
                        || $password == icms_core_Datafilter::utf8_strrev($login_name, TRUE)
                        || strripos($password, $login_name) === TRUE
                    ) {

Each class must be in a file by itself

Open

class ProtectorFilterHandler {

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a file by itself

Open

class SystemBlocksadminHandler extends icms_view_block_Handler {

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a file by itself

Open

class SystemBlockspadminHandler extends icms_view_block_position_Handler {

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a file by itself

Open

class SystemCustomtagHandler extends icms_ipf_Handler {

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a file by itself

Open

class SystemAdsenseHandler extends icms_ipf_Handler {

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a file by itself

Open

class SystemAutotasksHandler extends icms_ipf_Handler {

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a file by itself

Open

class SystemPagesHandler extends icms_data_page_Handler {

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a file by itself

Open

class SystemMimetypeHandler extends icms_ipf_Handler {

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a file by itself

Open

class SystemUserrankHandler extends icms_ipf_Handler {

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a file by itself

Open

class SystemRatingHandler extends icms_ipf_Handler {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    var IS_DONTENUM_BUGGY = (function(){
        for (var p in { toString: 1 }) {
            if (p === 'toString') return false;
        }
        return true;

• Read up
  Read up

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

Open

    var IS_DONTENUM_BUGGY = (function(){
        for (var p in { toString: 1 }) {
            if (p === 'toString') return false;
        }
        return true;

• Read up
  Read up

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

Open

        foreach ($_POST as $k => $v) {
            if ($k == 'XOOPS_TOKEN_REQUEST') {
                // regenerate token value
                $hiddenform .= icms::$security->getTokenHTML() . "\n";
            } else {

• Read up
  Read up

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

Open

        foreach ($_POST as $k => $v) {
            if ($k == 'XOOPS_TOKEN_REQUEST') {
                // regenerate token value
                $hiddenform .= icms::$security->getTokenHTML() . "\n";
            } else {

• Read up
  Read up