mambax7 / newbb5

viewpost.php

<?php
//
//  ------------------------------------------------------------------------ //
//                XOOPS - PHP Content Management System                      //
//                  Copyright (c) 2000-2016 XOOPS.org                        //
//                       <https://xoops.org/>                             //
//  ------------------------------------------------------------------------ //
//  This program is free software; you can redistribute it and/or modify     //
//  it under the terms of the GNU General Public License as published by     //
//  the Free Software Foundation; either version 2 of the License, or        //
//  (at your option) any later version.                                      //
//                                                                           //
//  You may not change or alter any portion of this comment or credits       //
//  of supporting developers from this source code or any supporting         //
//  source code which is considered copyrighted (c) material of the          //
//  original comment or credit authors.                                      //
//                                                                           //
//  This program is distributed in the hope that it will be useful,          //
//  but WITHOUT ANY WARRANTY; without even the implied warranty of           //
//  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the            //
//  GNU General Public License for more details.                             //
//                                                                           //
//  You should have received a copy of the GNU General Public License        //
//  along with this program; if not, write to the Free Software              //
//  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA //
//  ------------------------------------------------------------------------ //
//  Author: phppp (D.J., infomax@gmail.com)                                  //
//  URL: https://xoops.org                                                    //
//  Project: Article Project                                                 //
//  ------------------------------------------------------------------------ //

use Xmf\Request;
use XoopsModules\Newbb;

include_once __DIR__ . '/header.php';

$start    = Request::getInt('start', 0, 'GET');
$forum_id = Request::getInt('forum', 0, 'GET');

$forum_id does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$order    = Request::getString('order', 'DESC', 'GET');

$uid = Request::getInt('uid', 0, 'GET');

$status = (Request::getString('status', '', 'GET')
           && in_array(Request::getString('status', '', 'GET'), ['active', 'pending', 'deleted', 'new', 'all', 'digest'], true)) ? Request::getString('status', '', 'GET') : '';
$mode   = Request::getInt('mode', 0, 'GET');
$mode   = (!empty($status) && in_array($status, ['active', 'pending', 'deleted'], true)) ? 2 : $mode;

///** @var Newbb\ForumHandler $forumHandler */

55% of this comment could be valid code. Did you maybe forget this after debugging?

//$forumHandler = Newbb\Helper::getInstance()->getHandler('Forum');
///** @var Newbb\PostHandler $postHandler */
//$postHandler = Newbb\Helper::getInstance()->getHandler('Post');

if (empty($forum_id)) {

$forum_id does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $forums       = $forumHandler->getByPermission(0, 'view');
    $accessForums = array_keys($forums);
    $isAdmin      = $GLOBALS['xoopsUserIsAdmin'];
} else {
    $forumObject       = $forumHandler->get($forum_id);

$forum_id does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $forums[$forum_id] = $forumObject;

$forum_id does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $accessForums      = [$forum_id];

$forum_id does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $isAdmin           = newbbIsAdmin($forumObject);
}

/* Only admin has access to admin mode */
if (!$isAdmin && 2 === $mode) {

This code seems to be duplicated across your project.

    $status = in_array($status, ['active', 'pending', 'deleted'], true) ? '' : $status;
    $mode   = 0;
}
if ($mode) {
    $_GET['viewmode'] = 'flat';
}
//echo $mode.' - '.$status;

63% of this comment could be valid code. Did you maybe forget this after debugging?

$post_perpage = $GLOBALS['xoopsModuleConfig']['posts_per_page'];

$post_perpage does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$criteria_count = new \CriteriaCompo(new \Criteria('forum_id', '(' . implode(',', $accessForums) . ')', 'IN'));

$criteria_count does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$criteria_post  = new \CriteriaCompo(new \Criteria('p.forum_id', '(' . implode(',', $accessForums) . ')', 'IN'));

$criteria_post does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$criteria_post->setSort('p.post_id');

$criteria_post does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$criteria_post->setOrder($order);

$criteria_post does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

if (!empty($uid)) {
    $criteria_count->add(new \Criteria('uid', $uid));

$criteria_count does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $criteria_post->add(new \Criteria('p.uid', $uid));

$criteria_post does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

}

$join = null;
// START irmtfan solve the status issues and specially status = new issue
switch ($status) {
    case 'pending':
        $criteria_count->add(new \Criteria('approved', 0)); // irmtfan add new \Criteria

$criteria_count does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $criteria_post->add(new \Criteria('p.approved', 0)); // irmtfan add new \Criteria

$criteria_post does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        break;
    case 'deleted':
        $criteria_count->add(new \Criteria('approved', -1)); // irmtfan add new \Criteria

$criteria_count does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $criteria_post->add(new \Criteria('p.approved', -1)); // irmtfan add new \Criteria

$criteria_post does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        break;
    case 'new':
        //$criteria_status_count = new \CriteriaCompo(new \Criteria("post_time", (int)($last_visit), ">"));// irmtfan commented and removed

64% of this comment could be valid code. Did you maybe forget this after debugging?

        //$criteria_status_post = new \CriteriaCompo(new \Criteria("p.post_time", (int)($last_visit), ">"));// irmtfan commented and removed

64% of this comment could be valid code. Did you maybe forget this after debugging?

        $criteria_count->add(new \Criteria('approved', 1)); // irmtfan uncomment

$criteria_count does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $criteria_post->add(new \Criteria('p.approved', 1)); // irmtfan uncomment

$criteria_post does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        // following is for 'unread' -- not finished -- irmtfan Now it is finished!
        if (empty($GLOBALS['xoopsModuleConfig']['read_mode'])) {

This if statement is empty and can be removed.

            //$criteria_status_count->add(new \Criteria('approved', 1));// irmtfan commented and removed

74% of this comment could be valid code. Did you maybe forget this after debugging?

            //$criteria_status_post->add(new \Criteria('p.approved', 1));// irmtfan commented and removed

74% of this comment could be valid code. Did you maybe forget this after debugging?

        } elseif (2 == $GLOBALS['xoopsModuleConfig']['read_mode']) {
            // START irmtfan use read_uid to find the unread posts when the user is logged in
            $read_uid = is_object($GLOBALS['xoopsUser']) ? $GLOBALS['xoopsUser']->getVar('uid') : 0;

$read_uid does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            if (!empty($read_uid)) {

$read_uid does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                $join                 = ' LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('newbb_reads_topic') . ' AS r ON r.read_item = p.topic_id AND r.uid = ' . $read_uid . ' '; // irmtfan corrected add AS

$read_uid does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                $criteria_status_post = new \CriteriaCompo();// irmtfan new \Criteria

$criteria_status_post does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                $criteria_status_post->add(new \Criteria('p.post_id', 'r.`post_id`', '>')); // irmtfan corrected - should use $value='r.``' to render in XOOPS/class/criteria.php

$criteria_status_post does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                $criteria_status_post->add(new \Criteria('r.read_id', null, 'IS NULL'), 'OR');// irmtfan corrected - should use "IS NULL" to render in XOOPS/class/criteria.php

$criteria_status_post does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                $criteria_post->add($criteria_status_post); // irmtfan add the status criteria to post criteria - move here

$criteria_post does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                $criteria_count = $criteria_post;// irmtfan criteria count is equal to criteria post - move here

$criteria_count does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            } else {

This else statement is empty and can be removed.

            }
            // END irmtfan use read_uid to find the unread posts when the user is logged in
            //$criteria_status_post->add(new \Criteria("p.approved", 1)); // irmtfan commented and removed

69% of this comment could be valid code. Did you maybe forget this after debugging?

            //$criteria_status_count =& $criteria_status_post;

58% of this comment could be valid code. Did you maybe forget this after debugging?

        } elseif (1 == $GLOBALS['xoopsModuleConfig']['read_mode']) {
            $criteria_count->add(new \Criteria('post_time', (int)$last_visit, '>')); // irmtfan add new \Criteria

$criteria_count does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            $criteria_post->add(new \Criteria('p.post_time', (int)$last_visit, '>')); // irmtfan add new \Criteria

$criteria_post does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            // START irmtfan fix read_mode = 1 bugs - for all users (member and anon)
            $topics         = [];
            $topic_lastread = newbbGetCookie('LT', true);

$topic_lastread does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            if (count($topic_lastread) > 0) {

$topic_lastread does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                foreach ($topic_lastread as $id => $time) {

$topic_lastread does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

The expression $topic_lastread of type array|null is not guaranteed to be traversable. How about adding an additional type check?

                    if ($time > (int)$last_visit) {

$last_visit does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                        $topics[] = $id;
                    }
                }
            }
            if (count($topics) > 0) {
                $criteria_count->add(new \Criteria('topic_id', '(' . implode(',', $topics) . ')', 'NOT IN'));

$criteria_count does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

                $criteria_post->add(new \Criteria('p.topic_id', '(' . implode(',', $topics) . ')', 'NOT IN'));

$criteria_post does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            }
            // END irmtfan fix read_mode = 1 bugs - for all users (member and anon)
            //$criteria_status_count->add(new \Criteria("approved", 1));// irmtfan commented and removed

74% of this comment could be valid code. Did you maybe forget this after debugging?

            //$criteria_status_post->add(new \Criteria("p.approved", 1));// irmtfan commented and removed

74% of this comment could be valid code. Did you maybe forget this after debugging?

        }
        break;
    default:
        $criteria_count->add(new \Criteria('approved', 1)); // irmtfan add new \Criteria

$criteria_count does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $criteria_post->add(new \Criteria('p.approved', 1)); // irmtfan add new \Criteria

$criteria_post does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        break;
}
//$criteria_count->add($criteria_status_count); // irmtfan commented and removed

39% of this comment could be valid code. Did you maybe forget this after debugging?

//$criteria_post->add($criteria_status_post); // irmtfan commented and removed
// END irmtfan solve the status issues and specially status = new issue
///** @var Newbb\KarmaHandler $karmaHandler */
//$karmaHandler = Newbb\Helper::getInstance()->getHandler('Karma');
$user_karma = $karmaHandler->getUserKarma();

$user_karma does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$valid_modes     = ['flat', 'compact'];

$valid_modes does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$viewmode_cookie = newbbGetCookie('V');

$viewmode_cookie does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

if ('compact' === Request::getString('viewmode', '', 'GET')) {
    newbbSetCookie('V', 'compact', $forumCookie['expire']);
}

$viewmode = Request::getString('viewmode', (!empty($viewmode_cookie) ? $viewmode_cookie : (@$valid_modes[$GLOBALS['xoopsModuleConfig']['view_mode'] - 1])), 'GET');

$viewmode_cookie does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$viewmode = in_array($viewmode, $valid_modes) ? $viewmode : $valid_modes[0];

$valid_modes does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$postCount = $postHandler->getPostCount($criteria_count, $join);// irmtfan add join for read_mode = 2

$criteria_count does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$posts     = $postHandler->getPostsByLimit($criteria_post, $post_perpage, $start, $join);// irmtfan add join for read_mode = 2

$criteria_post does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$poster_array = [];

$poster_array does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

if (count($posts) > 0) {
    foreach (array_keys($posts) as $id) {
        /** @var Newbb\Post[] $posts */
        $poster_array[$posts[$id]->getVar('uid')] = 1;

$poster_array does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    }
}

$xoops_pagetitle                = $xoopsModule->getVar('name') . ' - ' . _MD_NEWBB_VIEWALLPOSTS;

$xoops_pagetitle does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$xoopsOption['xoops_pagetitle'] = $xoops_pagetitle;

$xoops_pagetitle does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$xoopsOption['template_main']   = 'newbb_viewpost.tpl';

include_once $GLOBALS['xoops']->path('header.php');
include_once __DIR__ . '/include/functions.time.php';
include_once __DIR__ . '/include/functions.render.php';

//global $xoTheme;

75% of this comment could be valid code. Did you maybe forget this after debugging?

//$xoTheme->addScript('/Frameworks/textsanitizer/xoops.js');

if (!empty($forum_id)) {

$forum_id does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    if (!$forumHandler->getPermission($forumObject, 'view')) {
        redirect_header(XOOPS_URL . '/index.php', 2, _MD_NEWBB_NORIGHTTOACCESS);
    }
    if ($forumObject->getVar('parent_forum')) {
        $parent_forumObject = $forumHandler->get($forumObject->getVar('parent_forum'), ['forum_name']);

$parent_forumObject does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $parentforum        = [
            'id'   => $forumObject->getVar('parent_forum'),
            'name' => $parent_forumObject->getVar('forum_name')

$parent_forumObject does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        ];
        unset($parent_forumObject);

$parent_forumObject does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $xoopsTpl->assign_by_ref('parentforum', $parentforum);
    }
    $xoopsTpl->assign('forum_name', $forumObject->getVar('forum_name'));
    $xoopsTpl->assign('forum_moderators', $forumObject->dispForumModerators());

    $xoops_pagetitle = $forumObject->getVar('forum_name') . ' - ' . _MD_NEWBB_VIEWALLPOSTS . ' [' . $xoopsModule->getVar('name') . ']';

$xoops_pagetitle does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $xoopsTpl->assign('forum_id', $forumObject->getVar('forum_id'));
    // irmtfan new method
    if (!empty($GLOBALS['xoopsModuleConfig']['rss_enable'])) {

This code seems to be duplicated across your project.

        $xoopsTpl->assign('xoops_module_header', '
            <link rel="alternate" type="application/xml+rss" title="' . $xoopsModule->getVar('name') . '-' . $forumObject->getVar('forum_name') . '" href="' . XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/rss.php?f=' . $forum_id . '" />

$forum_id does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            ' . @$xoopsTpl->get_template_vars('xoops_module_header'));
    }
} elseif (!empty($GLOBALS['xoopsModuleConfig']['rss_enable'])) {

This code seems to be duplicated across your project.

    $xoopsTpl->assign('xoops_module_header', '
        <link rel="alternate" type="application/xml+rss" title="' . $xoopsModule->getVar('name') . '" href="' . XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/rss.php" />
    ' . @$xoopsTpl->get_template_vars('xoops_module_header'));
}
// irmtfan remove and move to footer.php

36% of this comment could be valid code. Did you maybe forget this after debugging?

//$xoopsTpl->assign('xoops_module_header', $xoops_module_header);
$xoopsTpl->assign('xoops_pagetitle', $xoops_pagetitle);

$xoops_pagetitle does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

// irmtfan - remove icon_path and use newbbDisplayImage
$xoopsTpl->assign('anonym_avatar', newbbDisplayImage('anonym'));
$userid_array = [];

$userid_array does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

if (count($poster_array) > 0) {

$poster_array does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

This code seems to be duplicated across your project.

    /** @var \XoopsMembershipHandler $memberHandler */
    $memberHandler = xoops_getHandler('member');
    $userid_array  = array_keys($poster_array);

$userid_array does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $user_criteria = '(' . implode(',', $userid_array) . ')';

$user_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $users         = $memberHandler->getUsers(new \Criteria('uid', $user_criteria, 'IN'), true);

$user_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

} else {
    $user_criteria = '';

$user_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $users         = null;
}

$online = [];

if ($GLOBALS['xoopsModuleConfig']['wol_enabled']) {
    if (!empty($user_criteria)) {

$user_criteria does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        //        /** @var Newbb\OnlineHandler $onlineHandler */
        //        $onlineHandler = Newbb\Helper::getInstance()->getHandler('Online');

57% of this comment could be valid code. Did you maybe forget this after debugging?

        $onlineHandler->init($forum_id);

$forum_id does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    }
}

$viewtopic_users = [];

$viewtopic_users does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

if (count($userid_array) > 0) {

$userid_array does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

//    require $GLOBALS['xoops']->path('modules/' . $xoopsModule->getVar('dirname', 'n') . '/class/user.php');

63% of this comment could be valid code. Did you maybe forget this after debugging?

    $userHandler         = new Newbb\UserHandler($GLOBALS['xoopsModuleConfig']['groupbar_enabled'], $GLOBALS['xoopsModuleConfig']['wol_enabled']);
    $userHandler->users  = $users;
    $userHandler->online = $online;

The property online does not seem to exist. Did you mean enableOnline?

    $viewtopic_users     = $userHandler->getUsers();

$viewtopic_users does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

}

$pn = 0;
//$topicHandler = Newbb\Helper::getInstance()->getHandler('Topic');

60% of this comment could be valid code. Did you maybe forget this after debugging?

static $suspension = [];
foreach (array_keys($posts) as $id) {
    ++$pn;

    /** @var Newbb\Post $post */
    $post       = $posts[$id];
    $post_title = $post->getVar('subject');

$post_title does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    if ($posticon = $post->getVar('icon')) {
        $post_image = '<a name="' . $post->getVar('post_id') . '"><img src="' . XOOPS_URL . '/images/subject/' . htmlspecialchars($posticon) . '" alt="" /></a>';

$post_image does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    } else {
        $post_image = '<a name="' . $post->getVar('post_id') . '"><img src="' . XOOPS_URL . '/images/icons/no_posticon.gif" alt="" /></a>';

$post_image does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    }
    $poster = [
        'uid'  => 0,
        'name' => $post->getVar('poster_name') ?: $myts->htmlSpecialChars($GLOBALS['xoopsConfig']['anonymous']),
        'link' => $post->getVar('poster_name') ?: $myts->htmlSpecialChars($GLOBALS['xoopsConfig']['anonymous'])
    ];
    if ($post->getVar('uid') > 0 && isset($viewtopic_users[$post->getVar('uid')])) {

$viewtopic_users does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $poster = $viewtopic_users[$post->getVar('uid')];

$viewtopic_users does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    }
    if ($isAdmin || $post->checkIdentity()) {
        $post_text       = $post->getVar('post_text');

$post_text does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $post_attachment = $post->displayAttachment();

$post_attachment does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    } elseif ($GLOBALS['xoopsModuleConfig']['enable_karma'] && $post->getVar('post_karma') > $user_karma) {

$user_karma does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

This code seems to be duplicated across your project.

        $post_text       = "<div class='karma'>" . sprintf(_MD_NEWBB_KARMA_REQUIREMENT, $user_karma, $post->getVar('post_karma')) . '</div>';

$post_text does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $post_attachment = '';

$post_attachment does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    } elseif ($GLOBALS['xoopsModuleConfig']['allow_require_reply'] && $post->getVar('require_reply')) {
        $post_text       = "<div class='karma'>" . _MD_NEWBB_REPLY_REQUIREMENT . '</div>';

$post_text does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $post_attachment = '';

$post_attachment does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    } else {
        $post_text       = $post->getVar('post_text');

$post_text does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $post_attachment = $post->displayAttachment();

$post_attachment does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    }

    $thread_buttons = [];

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    if ($GLOBALS['xoopsModuleConfig']['enable_permcheck']) {
        if (!isset($suspension[$post->getVar('forum_id')])) {
            //            /** @var Newbb\ModerateHandler $moderateHandler */
            //            $moderateHandler                       = Newbb\Helper::getInstance()->getHandler('Moderate');

57% of this comment could be valid code. Did you maybe forget this after debugging?

            $suspension[$post->getVar('forum_id')] = !$moderateHandler->verifyUser(-1, '', $post->getVar('forum_id'));
        }

        if ($isAdmin

This code seems to be duplicated across your project.

            || (!$suspension[$post->getVar('forum_id')] && $post->checkIdentity()
                && $post->checkTimelimit('delete_timelimit'))) {
            $thread_buttons['delete']['image'] = newbbDisplayImage('p_delete', _DELETE);

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            $thread_buttons['delete']['link']  = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/delete.php?forum=' . $post->getVar('forum_id') . '&topic_id=' . $post->getVar('topic_id');

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            $thread_buttons['delete']['name']  = _DELETE;

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        }
        if ($isAdmin

This code seems to be duplicated across your project.

            || !$suspension[$post->getVar('forum_id')] && $post->checkIdentity()
               && $post->checkTimelimit('edit_timelimit')) {
            $thread_buttons['edit']['image'] = newbbDisplayImage('p_edit', _EDIT);

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            $thread_buttons['edit']['link']  = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/edit.php?forum=' . $post->getVar('forum_id') . '&topic_id=' . $post->getVar('topic_id');

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            $thread_buttons['edit']['name']  = _EDIT;

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        }
        if (is_object($GLOBALS['xoopsUser']) && !$suspension[$post->getVar('forum_id')]) {
            $thread_buttons['reply']['image'] = newbbDisplayImage('p_reply', _MD_NEWBB_REPLY);

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            $thread_buttons['reply']['link']  = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/reply.php?forum=' . $post->getVar('forum_id') . '&topic_id=' . $post->getVar('topic_id');

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            $thread_buttons['reply']['name']  = _MD_NEWBB_REPLY;

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            $thread_buttons['quote']['image'] = newbbDisplayImage('p_quote', _MD_NEWBB_QUOTE);

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            $thread_buttons['quote']['link']  = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/reply.php?forum=' . $post->getVar('forum_id') . '&topic_id=' . $post->getVar('topic_id') . '&quotedac=1';

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

            $thread_buttons['quote']['name']  = _MD_NEWBB_QUOTE;

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        }
    } else {
        $thread_buttons['delete']['image'] = newbbDisplayImage('p_delete', _DELETE);

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $thread_buttons['delete']['link']  = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/delete.php?forum=' . $post->getVar('forum_id') . '&topic_id=' . $post->getVar('topic_id');

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $thread_buttons['delete']['name']  = _DELETE;

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $thread_buttons['edit']['image']   = newbbDisplayImage('p_edit', _EDIT);

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $thread_buttons['edit']['link']    = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/edit.php?forum=' . $post->getVar('forum_id') . '&topic_id=' . $post->getVar('topic_id');

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $thread_buttons['edit']['name']    = _EDIT;

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $thread_buttons['reply']['image']  = newbbDisplayImage('p_reply', _MD_NEWBB_REPLY);

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $thread_buttons['reply']['link']   = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/reply.php?forum=' . $post->getVar('forum_id') . '&topic_id=' . $post->getVar('topic_id');

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $thread_buttons['reply']['name']   = _MD_NEWBB_REPLY;

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    }

    if (!$isAdmin && $GLOBALS['xoopsModuleConfig']['reportmod_enabled']) {
        $thread_buttons['report']['image'] = newbbDisplayImage('p_report', _MD_NEWBB_REPORT);

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $thread_buttons['report']['link']  = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/report.php?forum=' . $post->getVar('forum_id') . '&topic_id=' . $post->getVar('topic_id');

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        $thread_buttons['report']['name']  = _MD_NEWBB_REPORT;

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    }
    $thread_action = [];

$thread_action does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    $xoopsTpl->append('posts', [
        'post_id'         => $post->getVar('post_id'),
        'topic_id'        => $post->getVar('topic_id'),
        'forum_id'        => $post->getVar('forum_id'),
        'post_date'       => newbbFormatTimestamp($post->getVar('post_time')),
        'post_image'      => $post_image,

$post_image does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        'post_title'      => $post_title,

$post_title does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        'post_text'       => $post_text,

$post_text does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        'post_attachment' => $post_attachment,

$post_attachment does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        'post_edit'       => $post->displayPostEdit(),
        'post_no'         => $start + $pn,
        'post_signature'  => $post->getVar('attachsig') ? @$poster['signature'] : '',
        //                                 'poster_ip'       => ($isAdmin && $GLOBALS['xoopsModuleConfig']['show_ip']) ? long2ip($post->getVar('poster_ip')) : '',

69% of this comment could be valid code. Did you maybe forget this after debugging?

        'poster_ip'       => ($isAdmin && $GLOBALS['xoopsModuleConfig']['show_ip']) ? $post->getVar('poster_ip') : '',
        'thread_action'   => $thread_action,

$thread_action does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        'thread_buttons'  => $thread_buttons,

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        'poster'          => $poster
    ]);

    unset($thread_buttons, $poster);

$thread_buttons does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

}
unset($viewtopic_users, $forums);

$viewtopic_users does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

if (!empty($GLOBALS['xoopsModuleConfig']['show_jump'])) {

This code seems to be duplicated across your project.

    include_once __DIR__ . '/include/functions.forum.php';
    $xoopsTpl->assign('forum_jumpbox', newbbMakeJumpbox($forum_id));

$forum_id does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

}

if ($postCount > $post_perpage) {

$post_perpage does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    include $GLOBALS['xoops']->path('class/pagenav.php');
    $nav = new \XoopsPageNav($postCount, $post_perpage, $start, 'start', 'forum=' . $forum_id . '&viewmode=' . $viewmode . '&status=' . $status . '&uid=' . $uid . '&order=' . $order . '&mode=' . $mode);

$post_perpage does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

    //if (isset($GLOBALS['xoopsModuleConfig']['do_rewrite'])) $nav->url = formatURL(Request::getString('SERVER_NAME', '', 'SERVER')) . $nav->url;

66% of this comment could be valid code. Did you maybe forget this after debugging?

    if ('select' === $GLOBALS['xoopsModuleConfig']['pagenav_display']) {

This code seems to be duplicated across your project.

        $navi = $nav->renderSelect();
    } elseif ('image' === $GLOBALS['xoopsModuleConfig']['pagenav_display']) {
        $navi = $nav->renderImageNav(4);
    } else {
        $navi = $nav->renderNav(4);
    }

    $xoopsTpl->assign('pagenav', $navi);
} else {
    $xoopsTpl->assign('pagenav', '');
}

$xoopsTpl->assign('lang_forum_index', sprintf(_MD_NEWBB_FORUMINDEX, htmlspecialchars($GLOBALS['xoopsConfig']['sitename'], ENT_QUOTES)));

switch ($status) {
    case 'active':
        $lang_title = _MD_NEWBB_VIEWALLPOSTS . ' [' . _MD_NEWBB_TYPE_ADMIN . ']';

$lang_title does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        break;
    case 'pending':
        $lang_title = _MD_NEWBB_VIEWALLPOSTS . ' [' . _MD_NEWBB_TYPE_PENDING . ']';

$lang_title does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        break;
    case 'deleted':
        $lang_title = _MD_NEWBB_VIEWALLPOSTS . ' [' . _MD_NEWBB_TYPE_DELETED . ']';

$lang_title does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        break;
    case 'new':
        $lang_title = _MD_NEWBB_NEWPOSTS;

$lang_title does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        break;
    default:
        $lang_title = _MD_NEWBB_VIEWALLPOSTS;

$lang_title does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        break;
}
if ($uid > 0) {
    $lang_title .= ' (' . XoopsUser::getUnameFromId($uid) . ')';

$lang_title does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

}
$xoopsTpl->assign('lang_title', $lang_title);

$lang_title does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

// irmtfan up to p_up
$xoopsTpl->assign('p_up', newbbDisplayImage('up', _MD_NEWBB_TOP));
$xoopsTpl->assign('groupbar_enable', $GLOBALS['xoopsModuleConfig']['groupbar_enabled']);
$xoopsTpl->assign('anonymous_prefix', $GLOBALS['xoopsModuleConfig']['anonymous_prefix']);
$xoopsTpl->assign('down', newbbDisplayImage('down', _MD_NEWBB_BOTTOM));

$all_link       = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewpost.php?forum=' . $forum_id . "&start=$start";

$all_link does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$post_link      = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewpost.php?forum=' . $forum_id;

$post_link does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$newpost_link   = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewpost.php?forum=' . $forum_id . '&status=new';

$newpost_link does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$digest_link    = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewpost.php?forum=' . $forum_id . "&start=$start&status=digest";

$digest_link does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$unreplied_link = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewpost.php?forum=' . $forum_id . "&start=$start&status=unreplied";

$unreplied_link does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$unread_link    = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewpost.php?forum=' . $forum_id . "&start=$start&status=unread";

$unread_link does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$xoopsTpl->assign('all_link', $all_link);

$all_link does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$xoopsTpl->assign('post_link', $post_link);

$post_link does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$xoopsTpl->assign('newpost_link', $newpost_link);

$newpost_link does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$xoopsTpl->assign('digest_link', $digest_link);

$digest_link does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$xoopsTpl->assign('unreplied_link', $unreplied_link);

$unreplied_link does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$xoopsTpl->assign('unread_link', $unread_link);

$unread_link does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$viewmode_options = [];

$viewmode_options does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

if ('DESC' === $order) {

This code seems to be duplicated across your project.

    $viewmode_options[] = [

$viewmode_options does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        'link'  => XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewpost.php?viewmode=flat&order=ASC&forum=' . $forum_id,

$forum_id does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        'title' => _OLDESTFIRST
    ];
} else {
    $viewmode_options[] = [

$viewmode_options does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        'link'  => XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewpost.php?viewmode=flat&order=DESC&forum=' . $forum_id,

$forum_id does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

        'title' => _NEWESTFIRST
    ];
}

//$xoopsTpl->assign('viewmode_compact', ($viewmode=="compact")?1:0);

84% of this comment could be valid code. Did you maybe forget this after debugging?

$xoopsTpl->assign_by_ref('viewmode_options', $viewmode_options);

$viewmode_options does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$xoopsTpl->assign('menumode', $menumode);
$xoopsTpl->assign('menumode_other', $menumode_other);

$menumode_other does not seem to conform to the naming convention (^[a-z][a-zA-Z0-9]*$).

$xoopsTpl->assign('viewer_level', $isAdmin ? 2 : is_object($GLOBALS['xoopsUser']));
$xoopsTpl->assign('uid', $uid);
$xoopsTpl->assign('mode', $mode);
$xoopsTpl->assign('status', $status);
// irmtfan move to footer.php
include_once __DIR__ . '/footer.php';
include $GLOBALS['xoops']->path('footer.php');