ImpressCMS/impresscms

File comment_post.php has 357 lines of code (exceeds 250 allowed). Consider refactoring.

Open

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

Consider simplifying this complex logical expression.

Open

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

Reference to static property module from undeclared class \icms

Open

    $comment_config = icms::$module->getInfo('comments');

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property module from undeclared class \icms

Open

                switch (icms::$module->config['com_rule']) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property module from undeclared class \icms

Open

    $moddir = icms::$module->getVar('dirname');

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

                if (!icms::$user->isAdmin($com_modid)) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property module from undeclared class \icms

Open

if ('system' == icms::$module->getVar('dirname')) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

    $module_handler = icms::handler('icms_module');

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property module from undeclared class \icms

Open

        if (icms::$module->getVar('dirname') != 'system') {

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

                $sysperm_handler = icms::handler('icms_member_groupperm');

• Exclude checks
  • Disable engine
  • Disable check

Call to method icms_trim from undeclared class \icms_core_DataFilter

Open

        $com_title = icms_core_DataFilter::icms_trim($_POST['com_title']);

• Exclude checks
  • Disable engine
  • Disable check

Call to method checkVar from undeclared class \icms_core_DataFilter

Open

        $comment->setVar('com_text', icms_core_DataFilter::checkVar($_POST['com_text'], '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('com_itemid', $com_itemid));

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property module from undeclared class \icms

Open

                if ('system' == icms::$module->getVar('dirname')) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

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

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

                if (icms::$user->isAdmin($com_modid)

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

                || $sysperm_handler->checkRight('system_admin', XOOPS_SYSTEM_COMMENT, icms::$user->getGroups())) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property module from undeclared class \icms

Open

                    switch (icms::$module->config['com_rule']) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_NOTITLE

Open

        $com_title = ($com_title == '') ? _NOTITLE : $com_title;

• Exclude checks
  • Disable engine
  • Disable check

Call to method stripSlashesGPC from undeclared class \icms_core_DataFilter

Open

        $com_title = icms_core_DataFilter::htmlSpecialChars(icms_core_DataFilter::stripSlashesGPC($_POST['com_title']));

• 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

Reference to static property user from undeclared class \icms

Open

                if (icms::$user->isAdmin($com_modid)

• Exclude checks
  • Disable engine
  • Disable check

Call to method add from undeclared class \icms_db_criteria_Compo

Open

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

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property module from undeclared class \icms

Open

                    $not_module =& icms::$module;

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_CM_THANKSPOST

Open

                    2, _CM_THANKSPOST);

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_CM_THANKSPOST

Open

                    . '#comment' . $newcid, 2, _CM_THANKSPOST);

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property module from undeclared class \icms

Open

    $com_modid = icms::$module->getVar('mid');

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

Open

    $com_mode = isset($_POST['com_mode']) ? icms_core_DataFilter::htmlSpecialChars(trim($_POST['com_mode']), ENT_QUOTES) : 'flat';

• Exclude checks
  • Disable engine
  • Disable check

Call to method htmlSpecialChars from undeclared class \icms_core_DataFilter

Open

                            ? $extra_param . '=' . icms_core_DataFilter::htmlSpecialChars(trim($_POST[$extra_param])) . '&'

• Exclude checks
  • Disable engine
  • Disable check

Call to method checkVar from undeclared class \icms_core_DataFilter

Open

        $com_text = icms_core_DataFilter::checkVar($_POST['com_text'], 'html', 'output');

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property module from undeclared class \icms

Open

                if (!empty(icms::$module->config['com_anonpost']) && !empty($noname)) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

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

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

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

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

                    if ($comment->getVar('com_uid') != icms::$user->getVar('uid')) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property module from undeclared class \icms

Open

if (!defined('ICMS_ROOT_PATH') || !is_object(icms::$module)) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method instance from undeclared class \icms_form_elements_captcha_Object

Open

            $icmsCaptcha = icms_form_elements_captcha_Object::instance();

• 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

Reference to static property module from undeclared class \icms

Open

    if (XOOPS_COMMENT_APPROVENONE == icms::$module->config['com_rule']) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

                || $sysperm_handler->checkRight('system_admin', XOOPS_SYSTEM_COMMENT, icms::$user->getGroups())) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

                    if (!$sysperm_handler->checkRight('system_admin', XOOPS_SYSTEM_COMMENT, icms::$user->getGroups())) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method error from undeclared class \icms_core_Message

Open

                    icms_core_Message::error();

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

Open

        redirect_header(ICMS_URL.'/',3, implode('<br />', icms::$security->getErrors()));

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property module from undeclared class \icms

Open

                if (icms::$module->config['com_anonpost'] != 1) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property security from undeclared class \icms

Open

        if (!icms::$security->check()) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

                    $sysperm_handler = icms::handler('icms_member_groupperm');

• Exclude checks
  • Disable engine
  • Disable check

Call to method checkVar from undeclared class \icms_core_DataFilter

Open

        $p_comment =& icms_core_DataFilter::checkVar($_POST['com_text'], 'html', 'input');

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Compo

Open

                    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('com_modid', $com_modid));

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

                    $criteria->add(new icms_db_criteria_Item('com_status', XOOPS_COMMENT_ACTIVE));

• Exclude checks
  • Disable engine
  • Disable check

Call to method htmlSpecialChars from undeclared class \icms_core_DataFilter

Open

            ? $extra_param . '=' . icms_core_DataFilter::htmlSpecialChars(trim($_POST[$extra_param])) . '&'

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

                $sysperm_handler = icms::handler('icms_member_groupperm');

• Exclude checks
  • Disable engine
  • Disable check

Call to method __construct from undeclared class \icms_db_criteria_Item

Open

                    $criteria = new icms_db_criteria_Compo(new icms_db_criteria_Item('com_modid', $com_modid));

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_NOPERM

Open

                2, _NOPERM);

• Exclude checks
  • Disable engine
  • Disable check

Call to method add from undeclared class \icms_db_criteria_Compo

Open

                    $criteria->add(new icms_db_criteria_Item('com_status', XOOPS_COMMENT_ACTIVE));

• Exclude checks
  • Disable engine
  • Disable check

Call to method handler from undeclared class \icms

Open

                    $module_handler = icms::handler('icms_module');

• Exclude checks
  • Disable engine
  • Disable check

Call to method error from undeclared class \icms_core_Message

Open

                icms_core_Message::error($comment->getHtmlErrors());

• Exclude checks
  • Disable engine
  • Disable check

Call to method htmlSpecialChars from undeclared class \icms_core_DataFilter

Open

        $com_title = icms_core_DataFilter::htmlSpecialChars(icms_core_DataFilter::stripSlashesGPC($_POST['com_title']));

• Exclude checks
  • Disable engine
  • Disable check

Reference to static property user from undeclared class \icms

Open

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

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \_NOPERM

Open

                    1, _NOPERM);

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be indented to the same level as the CASE body

Open

                    break;

• Exclude checks
  • Disable engine
  • Disable check

Only one argument is allowed per line in a multi-line function call

Open

                    1, _NOPERM);

• Exclude checks
  • Disable engine
  • Disable check

Only one argument is allowed per line in a multi-line function call

Open

                2, _NOPERM);

• Exclude checks
  • Disable engine
  • Disable check

Only one argument is allowed per line in a multi-line function call

Open

                    2, _CM_THANKSPOST);

• Exclude checks
  • Disable engine
  • Disable check

Only one argument is allowed per line in a multi-line function call

Open

                2, $icmsCaptcha->getMessage());

• Exclude checks
  • Disable engine
  • Disable check

Closing parenthesis of a multi-line function call must be on a line by itself

Open

                2, $icmsCaptcha->getMessage());

• Exclude checks
  • Disable engine
  • Disable check

Opening parenthesis of a multi-line function call must be the last content on the line

Open

                    redirect_header($redirect_page . '=' . $com_itemid . '&com_id=' . $com_id . '&com_mode=' . $com_mode . '&com_order=' . $com_order,

• Exclude checks
  • Disable engine
  • Disable check

No space found after comma in function call

Open

        redirect_header(ICMS_URL.'/',3, implode('<br />', icms::$security->getErrors()));

• Exclude checks
  • Disable engine
  • Disable check

Multi-line function call not indented correctly; expected 24 spaces but found 20

Open

                    . $com_rootid . '&com_mode=' . $com_mode . '&com_order=' . $com_order . '#comment' . $newcid,

• Exclude checks
  • Disable engine
  • Disable check

Closing parenthesis of a multi-line function call must be on a line by itself

Open

                    2, _CM_THANKSPOST);

• Exclude checks
  • Disable engine
  • Disable check

Opening parenthesis of a multi-line function call must be the last content on the line

Open

                redirect_header($redirect_page . '=' . $com_itemid . '&com_id=' . $com_id . '&com_mode=' . $com_mode . '&com_order=' . $com_order,

• Exclude checks
  • Disable engine
  • Disable check

Opening parenthesis of a multi-line function call must be the last content on the line

Open

                redirect_header($redirect_page . '=' . $com_itemid . '&com_id=' . $com_id . '&com_mode=' . $com_mode . '&com_order=' . $com_order,

• Exclude checks
  • Disable engine
  • Disable check

Closing parenthesis of a multi-line function call must be on a line by itself

Open

                    1, _NOPERM);

• Exclude checks
  • Disable engine
  • Disable check

Opening parenthesis of a multi-line function call must be the last content on the line

Open

                    redirect_header($redirect_page . '=' . $com_itemid . '&com_id=' . $newcid . '&com_rootid='

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 newline at end of file; 0 found

Open

}

• Exclude checks
  • Disable engine
  • Disable check

Closing parenthesis of a multi-line function call must be on a line by itself

Open

                2, _NOPERM);

• Exclude checks
  • Disable engine
  • Disable check