mambax7 / newbb5

edit.php

<?php
/**
 * NewBB 5.0x,  the forum module for XOOPS project
 *
 * @copyright      XOOPS Project (https://xoops.org)
 * @license        GNU GPL 2 or later (http://www.gnu.org/licenses/gpl-2.0.html)
 * @author         Taiwen Jiang (phppp or D.J.) <phppp@users.sourceforge.net>
 * @since          4.00
 * @package        module::newbb
 */

use Xmf\Request;

include_once __DIR__ . '/header.php';

foreach (['forum', 'topic_id', 'post_id', 'order'] as $getint) {

This code seems to be duplicated across your project.

    ${$getint} = Request::getInt($getint, 0, 'GET');
}

if (!$topic_id && !$post_id) {

This code seems to be duplicated across your project.

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

    $redirect = empty($forum) ? 'index.php' : "viewforum.php?forum={$forum}";
    redirect_header($redirect, 2, _MD_NEWBB_ERRORTOPIC);
}

///** @var \NewbbForumHandler $forumHandler */
//$forumHandler = xoops_getModuleHandler('forum', 'newbb');
///** @var \NewbbTopicHandler $topicHandler */
//$topicHandler = xoops_getModuleHandler('topic', 'newbb');
///** @var \NewbbPostHandler $postHandler */
//$postHandler = xoops_getModuleHandler('post', 'newbb');

/** @var \NewbbPost $postObject */
$postObject  = $postHandler->get($post_id);

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

$topicObject = $topicHandler->get($postObject->getVar('topic_id'));
$forumObject = $forumHandler->get($postObject->getVar('forum_id'));
if (!$forumHandler->getPermission($forumObject)) {
    redirect_header('index.php', 2, _MD_NEWBB_NORIGHTTOACCESS);
}

if ($GLOBALS['xoopsModuleConfig']['wol_enabled']) {
//    /** @var \NewbbOnlineHandler $onlineHandler */
//    $onlineHandler = xoops_getModuleHandler('online', 'newbb');
    $onlineHandler->init($forumObject);
}
$isAdmin = newbbIsAdmin($forumObject);
$uid     = is_object($GLOBALS['xoopsUser']) ? $GLOBALS['xoopsUser']->getVar('uid') : 0;

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

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

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

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

$error_msg    = null;

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

if (!$topicHandler->getPermission($forumObject, $topic_status, 'edit') || (!$isAdmin && !$postObject->checkIdentity())) {

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

    $error_msg = _MD_NEWBB_NORIGHTTOEDIT;

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

} elseif (!$isAdmin && !$postObject->checkTimelimit('edit_timelimit')) {
    $error_msg = _MD_NEWBB_TIMEISUP;

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

}

if (!empty($error_msg)) {

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

    /*
     * Build the page query
     */
    $query_vars  = ['topic_id', 'post_id', 'forum', 'status', 'order', '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]*$).

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

    unset($query_array);

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

    redirect_header("viewtopic.php?{$page_query}", 2, $error_msg);

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

}

if ($GLOBALS['xoopsModuleConfig']['wol_enabled']) {
//    $onlineHandler = xoops_getModuleHandler('online', 'newbb');
    $onlineHandler->init($forumObject);
}

$xoopsOption['template_main']                                        = 'newbb_edit_post.tpl';
$GLOBALS['xoopsConfig']['module_cache'][$xoopsModule->getVar('mid')] = 0;
// irmtfan include header.php after defining $xoopsOption['template_main']
include_once $GLOBALS['xoops']->path('header.php');

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

$categoryHandler = xoops_getModuleHandler("category");
$categoryObject = $categoryHandler->get($forumObject->getVar('cat_id'), array("cat_title"));
$xoopsTpl->assign('category', array("id" => $forumObject->getVar('cat_id'), "title" => $categoryObject->getVar('cat_title')));

$form_title = _EDIT.": <a href=\"viewtopic.php?post_id={$post_id}\">".$postObject->getVar('subject');
$xoopsTpl->assign("form_title", $form_title);

$xoopsTpl->assign("parentforum", $forumHandler->getParents($forumObject));

$xoopsTpl->assign(array(
    'forum_id'             => $forumObject->getVar('forum_id'),
    'forum_name'         => $forumObject->getVar('forum_name'),
    ));
*/

$dohtml        = $postObject->getVar('dohtml');
$dosmiley      = $postObject->getVar('dosmiley');
$doxcode       = $postObject->getVar('doxcode');
$dobr          = $postObject->getVar('dobr');
$icon          = $postObject->getVar('icon');
$attachsig     = $postObject->getVar('attachsig');
$istopic       = $postObject->isTopic() ? 1 : 0;
$isedit        = 1;
$subject       = $postObject->getVar('subject', 'E');
$message       = $postObject->getVar('post_text', 'E');
$poster_name   = $postObject->getVar('poster_name', 'E');

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

$attachments   = $postObject->getAttachment();
$post_karma    = $postObject->getVar('post_karma');

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

$require_reply = $postObject->getVar('require_reply');

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

$xoopsTpl->assign('error_message', _MD_NEWBB_EDITEDBY . ' ' . $GLOBALS['xoopsUser']->uname());
include __DIR__ . '/include/form.post.php';

///** @var \NewbbKarmaHandler $karmaHandler */
//$karmaHandler = xoops_getModuleHandler('karma', 'newbb');
$user_karma   = $karmaHandler->getUserKarma();

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

$posts_context     = [];

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

$posts_contextObject = $istopic ? [] : [$postHandler->get($postObject->getVar('pid'))];

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

foreach ($posts_contextObject as $post_contextObject) {

This code seems to be duplicated across your project.

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

    if ($GLOBALS['xoopsModuleConfig']['enable_karma'] && $post_contextObject->getVar('post_karma') > 0) {

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

        $p_message = sprintf(_MD_NEWBB_KARMA_REQUIREMENT, '***', $post_contextObject->getVar('post_karma')) . '</div>';

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

    } elseif ($GLOBALS['xoopsModuleConfig']['allow_require_reply'] && $post_contextObject->getVar('require_reply')) {

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

        $p_message = _MD_NEWBB_REPLY_REQUIREMENT;

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

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

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

    }

    if ($post_contextObject->getVar('uid')) {

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

        $p_name = newbbGetUnameFromId($post_contextObject->getVar('uid'), $GLOBALS['xoopsModuleConfig']['show_realname']);

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

    } else {
        $poster_name = $post_contextObject->getVar('poster_name');

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

        $p_name      = empty($poster_name) ? htmlspecialchars($GLOBALS['xoopsConfig']['anonymous']) : $poster_name;

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

    }
    $p_date    = formatTimestamp($post_contextObject->getVar('post_time'));

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

    $p_subject = $post_contextObject->getVar('subject');

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

    $posts_context[] = [

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

        'subject' => $p_subject,

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

        'meta'    => _MD_NEWBB_BY . ' ' . $p_name . ' ' . _MD_NEWBB_ON . ' ' . $p_date,

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

        'content' => $p_message

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

    ];
}
$xoopsTpl->assign_by_ref('posts_context', $posts_context);

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

// irmtfan move to footer.php
include_once __DIR__ . '/footer.php';
include $GLOBALS['xoops']->path('footer.php');