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;

include_once __DIR__ . '/header.php';

$start    = Request::getInt('start', 0, 'GET');
$forum_id = Request::getInt('forum', 0, 'GET');
$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 \NewbbForumHandler $forumHandler */
$forumHandler = xoops_getModuleHandler('forum', 'newbb');
/** @var \NewbbPostHandler $postHandler */
$postHandler = xoops_getModuleHandler('post', 'newbb');

if (empty($forum_id)) {
    $forums       = $forumHandler->getByPermission(0, 'view');
    $accessForums = array_keys($forums);
    $isAdmin      = $GLOBALS['xoopsUserIsAdmin'];
} else {
    $forumObject         = $forumHandler->get($forum_id);
    $forums[$forum_id] = $forumObject;
    $accessForums      = [$forum_id];
    $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'];

$criteria_count = new CriteriaCompo(new Criteria('forum_id', '(' . implode(',', $accessForums) . ')', 'IN'));
$criteria_post  = new CriteriaCompo(new Criteria('p.forum_id', '(' . implode(',', $accessForums) . ')', 'IN'));
$criteria_post->setSort('p.post_id');
$criteria_post->setOrder($order);

if (!empty($uid)) {
    $criteria_count->add(new Criteria('uid', $uid));
    $criteria_post->add(new Criteria('p.uid', $uid));
}

$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_post->add(new Criteria('p.approved', 0)); // irmtfan add new criteria
        break;
    case 'deleted':
        $criteria_count->add(new Criteria('approved', -1)); // irmtfan add new criteria
        $criteria_post->add(new Criteria('p.approved', -1)); // irmtfan add new criteria
        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_post->add(new Criteria('p.approved', 1)); // irmtfan uncomment
        // 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;
            if (!empty($read_uid)) {
                $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
                $criteria_status_post = new CriteriaCompo();// irmtfan new criteria
                $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->add(new Criteria('r.read_id', null, 'IS NULL'), 'OR');// irmtfan corrected - should use "IS NULL" to render in XOOPS/class/criteria.php
                $criteria_post->add($criteria_status_post); // irmtfan add the status criteria to post criteria - move here
                $criteria_count = $criteria_post;// irmtfan criteria count is equal to criteria post - move here
            } 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_post->add(new Criteria('p.post_time', (int)$last_visit, '>')); // irmtfan add new criteria
            // START irmtfan fix read_mode = 1 bugs - for all users (member and anon)
            $topics         = [];
            $topic_lastread = newbbGetCookie('LT', true);
            if (count($topic_lastread) > 0) {
                foreach ($topic_lastread as $id => $time) {

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) {
                        $topics[] = $id;
                    }
                }
            }
            if (count($topics) > 0) {
                $criteria_count->add(new Criteria('topic_id', '(' . implode(',', $topics) . ')', 'NOT IN'));
                $criteria_post->add(new Criteria('p.topic_id', '(' . implode(',', $topics) . ')', 'NOT IN'));
            }
            // 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_post->add(new Criteria('p.approved', 1)); // irmtfan add new criteria
        break;
}
//$criteria_count->add($criteria_status_count); // irmtfan commented and removed
//$criteria_post->add($criteria_status_post); // irmtfan commented and removed
// END irmtfan solve the status issues and specially status = new issue
/** @var \NewbbKarmaHandler $karmaHandler */
$karmaHandler = xoops_getModuleHandler('karma', 'newbb');
$user_karma   = $karmaHandler->getUserKarma();

$valid_modes     = ['flat', 'compact'];
$viewmode_cookie = newbbGetCookie('V');

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 = in_array($viewmode, $valid_modes) ? $viewmode : $valid_modes[0];

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

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

$xoops_pagetitle                = $xoopsModule->getVar('name') . ' - ' . _MD_NEWBB_VIEWALLPOSTS;
$xoopsOption['xoops_pagetitle'] = $xoops_pagetitle;
$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)) {
    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']);
        $parentforum      = [
            'id'   => $forumObject->getVar('parent_forum'),
            'name' => $parent_forumObject->getVar('forum_name')
        ];
        unset($parent_forumObject);
        $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') . ']';
    $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 . '" />
            ' . @$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);
// irmtfan - remove icon_path and use newbbDisplayImage
$xoopsTpl->assign('anonym_avatar', newbbDisplayImage('anonym'));
$userid_array = [];
if (count($poster_array) > 0) {

This code seems to be duplicated across your project.

    /** @var \XoopsMembershipHandler $memberHandler */
    $memberHandler = xoops_getHandler('member');
    $userid_array  = array_keys($poster_array);
    $user_criteria = '(' . implode(',', $userid_array) . ')';
    $users         = $memberHandler->getUsers(new Criteria('uid', $user_criteria, 'IN'), true);
} else {
    $user_criteria = '';
    $users         = null;
}

$online = [];

if ($GLOBALS['xoopsModuleConfig']['wol_enabled']) {

This code seems to be duplicated across your project.

    if (!empty($user_criteria)) {
        /** @var \NewbbOnlineHandler $onlineHandler */
        $onlineHandler = xoops_getModuleHandler('online', 'newbb');
        $onlineHandler->init($forum_id);
    }
}

$viewtopic_users = [];

if (count($userid_array) > 0) {

This code seems to be duplicated across your project.

    require $GLOBALS['xoops']->path('modules/' . $xoopsModule->getVar('dirname', 'n') . '/class/user.php');
    $userHandler         = new NewbbUserHandler($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();
}

$pn           = 0;
$topicHandler = xoops_getModuleHandler('topic', 'newbb');
static $suspension = [];
foreach (array_keys($posts) as $id) {
    ++$pn;

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

    if ($posticon = $post->getVar('icon')) {
        $post_image = '<a name="' . $post->getVar('post_id') . '"><img src="' . XOOPS_URL . '/images/subject/' . htmlspecialchars($posticon) . '" alt="" /></a>';
    } else {
        $post_image = '<a name="' . $post->getVar('post_id') . '"><img src="' . XOOPS_URL . '/images/icons/no_posticon.gif" alt="" /></a>';
    }
    $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')])) {
        $poster = $viewtopic_users[$post->getVar('uid')];
    }
    if ($isAdmin || $post->checkIdentity()) {
        $post_text       = $post->getVar('post_text');
        $post_attachment = $post->displayAttachment();
    } elseif ($GLOBALS['xoopsModuleConfig']['enable_karma'] && $post->getVar('post_karma') > $user_karma) {

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_attachment = '';
    } elseif ($GLOBALS['xoopsModuleConfig']['allow_require_reply'] && $post->getVar('require_reply')) {
        $post_text       = "<div class='karma'>" . _MD_NEWBB_REPLY_REQUIREMENT . '</div>';
        $post_attachment = '';
    } else {
        $post_text       = $post->getVar('post_text');
        $post_attachment = $post->displayAttachment();
    }

    $thread_buttons = [];

    if ($GLOBALS['xoopsModuleConfig']['enable_permcheck']) {
        if (!isset($suspension[$post->getVar('forum_id')])) {
            /** @var \NewbbModerateHandler $moderateHandler */
            $moderateHandler                       = xoops_getModuleHandler('moderate', 'newbb');
            $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['delete']['link']  = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/delete.php?forum=' . $post->getVar('forum_id') . '&topic_id=' . $post->getVar('topic_id');
            $thread_buttons['delete']['name']  = _DELETE;
        }
        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['edit']['link']  = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/edit.php?forum=' . $post->getVar('forum_id') . '&topic_id=' . $post->getVar('topic_id');
            $thread_buttons['edit']['name']  = _EDIT;
        }
        if (is_object($GLOBALS['xoopsUser']) && !$suspension[$post->getVar('forum_id')]) {
            $thread_buttons['reply']['image'] = newbbDisplayImage('p_reply', _MD_NEWBB_REPLY);
            $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['reply']['name']  = _MD_NEWBB_REPLY;

            $thread_buttons['quote']['image'] = newbbDisplayImage('p_quote', _MD_NEWBB_QUOTE);
            $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['quote']['name']  = _MD_NEWBB_QUOTE;
        }
    } else {
        $thread_buttons['delete']['image'] = newbbDisplayImage('p_delete', _DELETE);
        $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['delete']['name']  = _DELETE;
        $thread_buttons['edit']['image']   = newbbDisplayImage('p_edit', _EDIT);
        $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['edit']['name']    = _EDIT;
        $thread_buttons['reply']['image']  = newbbDisplayImage('p_reply', _MD_NEWBB_REPLY);
        $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['reply']['name']   = _MD_NEWBB_REPLY;
    }

    if (!$isAdmin && $GLOBALS['xoopsModuleConfig']['reportmod_enabled']) {
        $thread_buttons['report']['image'] = newbbDisplayImage('p_report', _MD_NEWBB_REPORT);
        $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['report']['name']  = _MD_NEWBB_REPORT;
    }
    $thread_action = [];

    $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_title'      => $post_title,
        'post_text'       => $post_text,
        'post_attachment' => $post_attachment,
        '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_buttons'  => $thread_buttons,
        'poster'          => $poster
    ]);

    unset($thread_buttons, $poster);
}
unset($viewtopic_users, $forums);

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

if ($postCount > $post_perpage) {
    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);
    //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 . ']';
        break;
    case 'pending':
        $lang_title = _MD_NEWBB_VIEWALLPOSTS . ' [' . _MD_NEWBB_TYPE_PENDING . ']';
        break;
    case 'deleted':
        $lang_title = _MD_NEWBB_VIEWALLPOSTS . ' [' . _MD_NEWBB_TYPE_DELETED . ']';
        break;
    case 'new':
        $lang_title = _MD_NEWBB_NEWPOSTS;
        break;
    default:
        $lang_title = _MD_NEWBB_VIEWALLPOSTS;
        break;
}
if ($uid > 0) {
    $lang_title .= ' (' . XoopsUser::getUnameFromId($uid) . ')';
}
$xoopsTpl->assign('lang_title', $lang_title);
// 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";
$post_link      = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewpost.php?forum=' . $forum_id;
$newpost_link   = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewpost.php?forum=' . $forum_id . '&status=new';
$digest_link    = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewpost.php?forum=' . $forum_id . "&start=$start&status=digest";
$unreplied_link = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewpost.php?forum=' . $forum_id . "&start=$start&status=unreplied";
$unread_link    = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewpost.php?forum=' . $forum_id . "&start=$start&status=unread";

$xoopsTpl->assign('all_link', $all_link);
$xoopsTpl->assign('post_link', $post_link);
$xoopsTpl->assign('newpost_link', $newpost_link);
$xoopsTpl->assign('digest_link', $digest_link);
$xoopsTpl->assign('unreplied_link', $unreplied_link);
$xoopsTpl->assign('unread_link', $unread_link);

$viewmode_options = [];
if ('DESC' === $order) {

This code seems to be duplicated across your project.

    $viewmode_options[] = [
        'link'  => XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewpost.php?viewmode=flat&order=ASC&forum=' . $forum_id,
        'title' => _OLDESTFIRST
    ];
} else {
    $viewmode_options[] = [
        'link'  => XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewpost.php?viewmode=flat&order=DESC&forum=' . $forum_id,
        '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);
$xoopsTpl->assign('menumode', $menumode);
$xoopsTpl->assign('menumode_other', $menumode_other);

$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');