XoopsModules\Newbb\Post::showPost()

Complexity

Conditions	41
Paths	> 20000

Size

Total Lines	240
Code Lines	176

Duplication

Lines	28
Ratio	11.67 %

Importance

Changes

0

<?php namespace XoopsModules\Newbb;
    public function showPost($isAdmin)

showPost uses the super-global variable $GLOBALS which is generally not recommended.

    {
        global $xoopsModule, $myts;

Use of global functionality is not recommended; it makes your code harder to test, and less reusable.

        global $forumUrl, $forumImage, $forumObject, $online, $viewmode;

Use of global functionality is not recommended; it makes your code harder to test, and less reusable.

        global $viewtopic_users, $viewtopic_posters, $topicObject, $user_karma;

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

Use of global functionality is not recommended; it makes your code harder to test, and less reusable.

        global $order, $start, $total_posts, $topic_status;

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

Use of global functionality is not recommended; it makes your code harder to test, and less reusable.

        static $post_NO = 0;
        static $name_anonymous;
        /** @var TopicHandler $topicHandler */
        $topicHandler = Newbb\Helper::getInstance()->getHandler('Topic');
        if (!isset($name_anonymous)) {

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

            $name_anonymous = $myts->htmlSpecialChars($GLOBALS['xoopsConfig']['anonymous']);

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

        }

        include_once __DIR__ . '/../include/functions.time.php';
        include_once __DIR__ . '/../include/functions.render.php';

        $post_id  = $this->getVar('post_id');

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

        $topic_id = $this->getVar('topic_id');

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

        $forum_id = $this->getVar('forum_id');

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

        $query_vars              = ['status', 'order', 'start', 'mode', 'viewmode'];

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

        $query_array             = [];

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

        $query_array['topic_id'] = "topic_id={$topic_id}";

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

        foreach ($query_vars as $var) {

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

            if (Request::getString($var, '', 'GET')) {
                $query_array[$var] = "{$var}=" . Request::getString($var, '', 'GET');

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

            }
        }
        $page_query = htmlspecialchars(implode('&', array_values($query_array)));

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

        $uid = is_object($GLOBALS['xoopsUser']) ? $GLOBALS['xoopsUser']->getVar('uid') : 0;

        ++$post_NO;

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

        if ('desc' === strtolower($order)) {
            $post_no = $total_posts - ($start + $post_NO) + 1;

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

        } else {
            $post_no = $start + $post_NO;

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

        }

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

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

            $post_attachment = $this->displayAttachment();

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

        } elseif ($GLOBALS['xoopsModuleConfig']['enable_karma'] && $this->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, $this->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'] && $this->getVar('require_reply')
                  && (!$uid || !in_array($uid, $viewtopic_posters))) {

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

            $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       = $this->getVar('post_text');

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

            $post_attachment = $this->displayAttachment();

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

        }

        // Hightlight search words
        $post_title = $this->getVar('subject');

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

        if ($keywords = Request::getString('keywords', '', 'GET')) {
            //$keywords   = $myts->htmlSpecialChars(trim(urldecode(Request::getString('keywords', '', 'GET'))));

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

            $post_text  = Highlighter::apply($keywords, $post_text, '<mark>', '</mark>');

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

            $post_title = Highlighter::apply($keywords, $post_title, '<mark>', '</mark>');

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

        }

        if (isset($viewtopic_users[$this->getVar('uid')])) {

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

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

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

        } else {
            $name   = ($post_name = $this->getVar('poster_name')) ? $post_name : $name_anonymous;

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

            $poster = [
                'poster_uid' => 0,
                'name'       => $name,
                'link'       => $name
            ];
        }

        if ($posticon = $this->getVar('icon')) {
            $post_image = '<a name="' . $post_id . '"><img src="' . XOOPS_URL . '/images/subject/' . $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_id . '"><img src="' . XOOPS_URL . '/images/icons/posticon.gif" alt="" /></a>';

$post_image 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]*$).

        $mod_buttons    = [];

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

        if ($isAdmin && ($GLOBALS['xoopsUser'] && $GLOBALS['xoopsUser']->getVar('uid') !== $this->getVar('uid'))
            && $this->getVar('uid') > 0) {
            $mod_buttons['bann']['image']    = newbbDisplayImage('p_bann', _MD_NEWBB_SUSPEND_MANAGEMENT);

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

            $mod_buttons['bann']['link']     = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/moderate.php?forum=' . $forum_id . '&uid=' . $this->getVar('uid');

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

            $mod_buttons['bann']['name']     = _MD_NEWBB_SUSPEND_MANAGEMENT;

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

            $thread_buttons['bann']['image'] = newbbDisplayImage('p_bann', _MD_NEWBB_SUSPEND_MANAGEMENT);

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

            $thread_buttons['bann']['link']  = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/moderate.php?forum=' . $forum_id . '&uid=' . $this->getVar('uid');

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

            $thread_buttons['bann']['name']  = _MD_NEWBB_SUSPEND_MANAGEMENT;

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

        }

        if ($GLOBALS['xoopsModuleConfig']['enable_permcheck']) {
            //            /** @var TopicHandler $topicHandler */
            //            $topicHandler =  Newbb\Helper::getInstance()->getHandler('Topic');

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

            $topic_status = $topicObject->getVar('topic_status');

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

            if ($topicHandler->getPermission($forum_id, $topic_status, 'edit')) {

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

This code seems to be duplicated across your project.

                $edit_ok = ($isAdmin || ($this->checkIdentity() && $this->checkTimelimit('edit_timelimit')));

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

                if ($edit_ok) {

$edit_ok 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?{$page_query}";

$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]*$).

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

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

                    $mod_buttons['edit']['link']     = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/edit.php?{$page_query}";

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

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

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

                }
            }

            if ($topicHandler->getPermission($forum_id, $topic_status, 'delete')) {

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

This code seems to be duplicated across your project.

                $delete_ok = ($isAdmin || ($this->checkIdentity() && $this->checkTimelimit('delete_timelimit')));

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

                if ($delete_ok) {

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

                    $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?{$page_query}";

$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]*$).

                    $mod_buttons['delete']['image']    = newbbDisplayImage('p_delete', _DELETE);

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

                    $mod_buttons['delete']['link']     = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/delete.php?{$page_query}";

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

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

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

                }
            }
            if ($topicHandler->getPermission($forum_id, $topic_status, 'reply')) {

$forum_id 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?{$page_query}";

$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?{$page_query}&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 {
            $mod_buttons['edit']['image'] = newbbDisplayImage('p_edit', _EDIT);

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

            $mod_buttons['edit']['link']  = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/edit.php?{$page_query}";

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

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

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

            $mod_buttons['delete']['image'] = newbbDisplayImage('p_delete', _DELETE);

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

            $mod_buttons['delete']['link']  = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . "/delete.php?{$page_query}";

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

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

$mod_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?{$page_query}";

$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?{$page_query}";

$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]*$).

        // irmtfan add pdf permission
        if (file_exists(XOOPS_ROOT_PATH . '/class/libraries/vendor/tecnickcom/tcpdf/tcpdf.php')
            && $topicHandler->getPermission($forum_id, $topic_status, 'pdf')) {

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

            $thread_action['pdf']['image']  = newbbDisplayImage('pdf', _MD_NEWBB_PDF);

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

            $thread_action['pdf']['link']   = XOOPS_URL . '/modules/newbb/makepdf.php?type=post&pageid=0';

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

            $thread_action['pdf']['name']   = _MD_NEWBB_PDF;

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

            $thread_action['pdf']['target'] = '_blank';

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

        }
        // irmtfan add print permission
        if ($topicHandler->getPermission($forum_id, $topic_status, 'print')) {

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

            $thread_action['print']['image']  = newbbDisplayImage('printer', _MD_NEWBB_PRINT);

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

            $thread_action['print']['link']   = XOOPS_URL . '/modules/newbb/print.php?form=2&forum=' . $forum_id . '&topic_id=' . $topic_id;

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

            $thread_action['print']['name']   = _MD_NEWBB_PRINT;

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

            $thread_action['print']['target'] = '_blank';

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

        }

        if ($GLOBALS['xoopsModuleConfig']['show_sociallinks']) {
            $full_title  = $this->getVar('subject');

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

            $clean_title = preg_replace('/[^A-Za-z0-9-]+/', '+', $this->getVar('subject'));

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

            $full_link   = XOOPS_URL . '/modules/newbb/viewtopic.php?post_id=' . $post_id;

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

            $thread_action['social_twitter']['image']  = newbbDisplayImage('twitter', _MD_NEWBB_SHARE_TWITTER);

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

            $thread_action['social_twitter']['link']   = 'http://twitter.com/share?text=' . $clean_title . '&url=' . $full_link;

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

            $thread_action['social_twitter']['name']   = _MD_NEWBB_SHARE_TWITTER;

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

            $thread_action['social_twitter']['target'] = '_blank';

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

            $thread_action['social_facebook']['image']  = newbbDisplayImage('facebook', _MD_NEWBB_SHARE_FACEBOOK);

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

            $thread_action['social_facebook']['link']   = 'http://www.facebook.com/sharer.php?u=' . $full_link;

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

            $thread_action['social_facebook']['name']   = _MD_NEWBB_SHARE_FACEBOOK;

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

            $thread_action['social_facebook']['target'] = '_blank';

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

            $thread_action['social_gplus']['image']  = newbbDisplayImage('googleplus', _MD_NEWBB_SHARE_GOOGLEPLUS);

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

            $thread_action['social_gplus']['link']   = 'https://plusone.google.com/_/+1/confirm?hl=en&url=' . $full_link;

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

            $thread_action['social_gplus']['name']   = _MD_NEWBB_SHARE_GOOGLEPLUS;

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

            $thread_action['social_gplus']['target'] = '_blank';

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

            $thread_action['social_linkedin']['image']  = newbbDisplayImage('linkedin', _MD_NEWBB_SHARE_LINKEDIN);

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

            $thread_action['social_linkedin']['link']   = 'http://www.linkedin.com/shareArticle?mini=true&title=' . $full_title . '&url=' . $full_link;

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

            $thread_action['social_linkedin']['name']   = _MD_NEWBB_SHARE_LINKEDIN;

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

            $thread_action['social_linkedin']['target'] = '_blank';

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

            $thread_action['social_delicious']['image']  = newbbDisplayImage('delicious', _MD_NEWBB_SHARE_DELICIOUS);

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

            $thread_action['social_delicious']['link']   = 'http://del.icio.us/post?title=' . $full_title . '&url=' . $full_link;

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

            $thread_action['social_delicious']['name']   = _MD_NEWBB_SHARE_DELICIOUS;

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

            $thread_action['social_delicious']['target'] = '_blank';

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

            $thread_action['social_digg']['image']  = newbbDisplayImage('digg', _MD_NEWBB_SHARE_DIGG);

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

            $thread_action['social_digg']['link']   = 'http://digg.com/submit?phase=2&title=' . $full_title . '&url=' . $full_link;

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

            $thread_action['social_digg']['name']   = _MD_NEWBB_SHARE_DIGG;

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

            $thread_action['social_digg']['target'] = '_blank';

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

            $thread_action['social_reddit']['image']  = newbbDisplayImage('reddit', _MD_NEWBB_SHARE_REDDIT);

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

            $thread_action['social_reddit']['link']   = 'http://reddit.com/submit?title=' . $full_title . '&url=' . $full_link;

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

            $thread_action['social_reddit']['name']   = _MD_NEWBB_SHARE_REDDIT;

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

            $thread_action['social_reddit']['target'] = '_blank';

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

            $thread_action['social_wong']['image']  = newbbDisplayImage('wong', _MD_NEWBB_SHARE_MRWONG);

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

            $thread_action['social_wong']['link']   = 'http://www.mister-wong.de/index.php?action=addurl&bm_url=' . $full_link;

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

            $thread_action['social_wong']['name']   = _MD_NEWBB_SHARE_MRWONG;

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

            $thread_action['social_wong']['target'] = '_blank';

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

        }

        $post = [
            'post_id'         => $post_id,

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

            'post_parent_id'  => $this->getVar('pid'),
            'post_date'       => newbbFormatTimestamp($this->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]*$).

            // irmtfan $post_title to add highlight keywords
            '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'       => $this->displayPostEdit(),
            'post_no'         => $post_no,

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

            'post_signature'  => $this->getVar('attachsig') ? @$poster['signature'] : '',
            //            'poster_ip'       => ($isAdmin && $GLOBALS['xoopsModuleConfig']['show_ip']) ? long2ip($this->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']) ? $this->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]*$).

            'mod_buttons'     => $mod_buttons,

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

            'poster'          => $poster,
            'post_permalink'  => '<a href="' . XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewtopic.php?post_id=' . $post_id . '"></a>'

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

        ];

        unset($thread_buttons, $mod_buttons, $eachposter);

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

        return $post;
    }
}