ImpressCMS/impresscms

Method __construct has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function __construct() {
        if (isset( $_SESSION['settings']['ROOT_PATH'] )) {
            $this->xoopsRootPath = $_SESSION['settings']['ROOT_PATH'];
        } else {
            $path = str_replace( "\\", "/", @realpath( '../' ) );

Method showRSS has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function showRSS() {
    global $icmsAdminTpl, $icmsConfigPersona;

    $rssurl = $icmsConfigPersona['rss_local'];
    $rssfile = ICMS_CACHE_PATH . '/adminnews_' . _LANGCODE . '.xml';

Method adminNav has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function adminNav($id = NULL, $separador = "/", $list = FALSE, $style = "style='font-weight:bold'") {
    global $target, $type;

    $admin_url = 'browser.php?target=' . $target . '&type=' . $type;
    if ($id === FALSE) {

Consider simplifying this complex logical expression.

Open

        if (element && isPercentage) {
            context = context || element.parentNode;
            var decimal = toDecimal(value), whole = null;

            var isHorizontal = property.include('left') || property.include('right') ||

Consider simplifying this complex logical expression.

Open

            if ( tokens.length > 2 && (token = tokens[0]).type === "ID" &&
                support.getById && context.nodeType === 9 && documentIsHTML &&
                Expr.relative[ tokens[1].type ] ) {

                context = ( Expr.find["ID"]( token.matches[0].replace(runescape, funescape), context ) || [] )[0];

Consider simplifying this complex logical expression.

Open

            if (element === document.body) {
                var bodyScrollNode = document.documentElement || document.body.parentNode || document.body;
                valueT += !Object.isUndefined(window.pageYOffset) ? window.pageYOffset : bodyScrollNode.scrollTop || 0;
                valueL += !Object.isUndefined(window.pageXOffset) ? window.pageXOffset : bodyScrollNode.scrollLeft || 0;
                break;

Method make_data has 8 arguments (exceeds 4 allowed). Consider refactoring.

Open

function make_data(&$dbm, &$cm, $adminname, $adminlogin_name, $adminpass, $adminmail, $language, $gruops) {

Consider simplifying this complex logical expression.

Open

                    if (!isset($yname) || trim($yname) == '' || $ymail == ''
                        || !isset($fname) || trim($fname) == ''
                        || !isset($fmail) || trim($fmail) == '') {
                        redirect_header(ICMS_URL.'/misc.php?action=showpopups&type=friend&op=sendform',2,_MSC_NEEDINFO);
                    }

Consider simplifying this complex logical expression.

Open

                                if (isset($infoarr[3]) && trim($infoarr[3]) == 'blocks') {
                                    $default =& $tpltpl_handler->find('default', 'block', NULL, trim($infoarr[2]), trim($infoarr[4]));
                                } elseif ((!isset($infoarr[4]) || trim($infoarr[4]) == '') && $infoarr[1] == 'templates') {
                                    $default =& $tpltpl_handler->find('default', 'module', NULL, trim($infoarr[2]), trim($infoarr[3]));
                                } elseif ($infoarr[1] == "templates" && $infoarr[2] == "system" && $infoarr[3] == "admin") {

Each class must be in a namespace of at least one level (a top-level vendor name)

Open

class IcmsInstallWizard {

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a namespace of at least one level (a top-level vendor name)

Open

class db_manager {

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a namespace of at least one level (a top-level vendor name)

Open

class setting_manager {

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a namespace of at least one level (a top-level vendor name)

Open

class mainfile_manager {

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a namespace of at least one level (a top-level vendor name)

Open

class TextSanitizer {

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a namespace of at least one level (a top-level vendor name)

Open

class ProtectorMySQLDatabase extends icms_db_legacy_mysql_Proxy {

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a namespace of at least one level (a top-level vendor name)

Open

class ProtectorFilterAbstract {

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a namespace of at least one level (a top-level vendor name)

Open

class ProtectorFilterHandler {

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a namespace of at least one level (a top-level vendor name)

Open

    class XoopsGTicket {

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a namespace of at least one level (a top-level vendor name)

Open

class protector_postcommon_post_need_multibyte extends ProtectorFilterAbstract {

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a namespace of at least one level (a top-level vendor name)

Open

class Protector {

• Exclude checks
  • Disable engine
  • Disable check