Issues in action.topic.php - Fixed Issues - Inspection of "Merge branch 'master' of https://github.com/mambax..." - mambax7/newbb5 - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 62af87...249590 )

by Michael

created 2018-01-13 06:18 UTC

action.topic.php (2 issues)

Labels

Severity

Minor 2

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?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.) <[email protected]>
 * @since          4.00
 * @package        module::newbb
 */

use Xmf\Request;

include_once __DIR__ . '/header.php';

$forum_id = Request::getInt('forum_id', 0, 'POST');
$topic_id = Request::getArray('topic_id', null, 'POST');

$op = Request::getCmd('op', '', 'POST');
$op = in_array($op, ['approve', 'delete', 'restore', 'move'], true) ? $op : '';

if (0 === count($topic_id) || 0 === count($op)) {
    // irmtfan - issue with javascript:history.go(-1)
    redirect_header(Request::getString('HTTP_REFERER', '', 'SERVER'), 2, _MD_NEWBB_NO_SELECTION);
}

$topic_id = array_values($topic_id);
///** @var \NewbbTopicHandler|XoopsPersistableObjectHandler $topicHandler */
//$topicHandler = xoops_getModuleHandler('topic', 'newbb');
///** @var \NewbbForumHandler|XoopsPersistableObjectHandler $forumHandler */
//$forumHandler = xoops_getModuleHandler('forum', 'newbb');

$isAdmin = newbbIsAdmin($forum_id);

if (!$isAdmin) {
    redirect_header(XOOPS_URL . '/index.php', 2, _MD_NEWBB_NORIGHTTOACCESS);
}
switch ($op) {
    case 'restore':
        $forums     = [];
        $topicsObject = $topicHandler->getAll(new Criteria('topic_id', '(' . implode(',', $topic_id) . ')', 'IN'));
        foreach (array_keys($topicsObject) as $id) {
            /** @var \Topic $topicObject */
            $topicObject = $topicsObject[$id];
            $topicHandler->approve($topicObject);
            $topicHandler->synchronization($topicObject);
            $forums[$topicObject->getVar('forum_id')] = 1;
        }
        $criteria_forum = new Criteria('forum_id', '(' . implode(',', array_keys($forums)) . ')', 'IN');
        $forumsObject     = $forumHandler->getAll($criteria_forum);
        foreach (array_keys($forumsObject) as $id) {
            $forumHandler->synchronization($forumsObject[$id]);
        }
        unset($topicsObject, $forumsObject);
        break;
    case 'approve':
        $forums     = [];
        $topicsObject = $topicHandler->getAll(new Criteria('topic_id', '(' . implode(',', $topic_id) . ')', 'IN'));
        foreach (array_keys($topicsObject) as $id) {
            /** @var \Topic $topicObject */
            $topicObject = $topicsObject[$id];
            $topicHandler->approve($topicObject);
            $topicHandler->synchronization($topicObject);
            $forums[$topicObject->getVar('forum_id')] = 1;
        }

        $criteria_forum = new Criteria('forum_id', '(' . implode(',', array_keys($forums)) . ')', 'IN');
        $forumsObject     = $forumHandler->getAll($criteria_forum);
        foreach (array_keys($forumsObject) as $id) {
            $forumHandler->synchronization($forumsObject[$id]);
        }

        if (empty($GLOBALS['xoopsModuleConfig']['notification_enabled'])) {
            break;
        }

        include_once __DIR__ . '/include/notification.inc.php';
        /** @var \XoopsNotificationHandler $notificationHandler */
        $notificationHandler = xoops_getHandler('notification');
        foreach (array_keys($topicsObject) as $id) {
            $topicObject           = $topicsObject[$id];
            $tags                = [];
            $tags['THREAD_NAME'] = $topicObject->getVar('topic_title');
            $tags['THREAD_URL']  = XOOPS_URL . '/modules/' . $moduleDirName . '/viewtopic.php?topic_id=' . $id . '&amp;forum=' . $topicObject->getVar('forum_id');
            /** @var \NewbbForum[] $forumsObject */
            $tags['FORUM_NAME'] = $forumsObject[$topicObject->getVar('forum_id')]->getVar('forum_name');
            $tags['FORUM_URL']  = XOOPS_URL . '/modules/' . $moduleDirName . '/viewforum.php?forum=' . $topicObject->getVar('forum_id');
            $notificationHandler->triggerEvent('global', 0, 'new_thread', $tags);
            $notificationHandler->triggerEvent('forum', $topicObject->getVar('forum_id'), 'new_thread', $tags);
            $postObject         = $topicHandler->getTopPost($id);
            $tags['POST_URL'] = $tags['THREAD_URL'] . '#forumpost' . $postObject->getVar('post_id');
            $notificationHandler->triggerEvent('thread', $id, 'new_post', $tags);
            $notificationHandler->triggerEvent('forum', $topicObject->getVar('forum_id'), 'new_post', $tags);
            $notificationHandler->triggerEvent('global', 0, 'new_post', $tags);
            $tags['POST_CONTENT'] = $postObject->getVar('post_text');
            $tags['POST_NAME']    = $postObject->getVar('subject');
            $notificationHandler->triggerEvent('global', 0, 'new_fullpost', $tags);
            $notificationHandler->triggerEvent('forum', $topicObject->getVar('forum_id'), 'new_fullpost', $tags);
            unset($postObject);
        }
        unset($topicsObject, $forumsObject);
        break;
    case 'delete':
        $forums     = [];
        /** @var \NewbbTopicHandler|XoopsPersistableObjectHandler $topicHandler */
        $topicsObject = $topicHandler->getAll(new Criteria('topic_id', '(' . implode(',', $topic_id) . ')', 'IN'));
        foreach (array_keys($topicsObject) as $id) {
            /** @var Topic $topicObject */
            $topicObject = $topicsObject[$id];
            // irmtfan should be set to false to not delete topic from database
            $topicHandler->delete($topicObject, false);
            $topicHandler->synchronization($topicObject);
            $forums[$topicObject->getVar('forum_id')] = 1;
        }

        $criteria_forum = new Criteria('forum_id', '(' . implode(',', array_keys($forums)) . ')', 'IN');
        $forumsObject     = $forumHandler->getAll($criteria_forum);
        foreach (array_keys($forumsObject) as $id) {
            $forumHandler->synchronization($forumsObject[$id]);
        }
        unset($topicsObject, $forumsObject);
        break;
    case 'move':
        if (Request::getInt('newforum', 0, 'POST')
            && Request::getInt('newforum', 0, 'POST') !== $forum_id
            && $forumHandler->getPermission(Request::getInt('newforum', 0, 'POST'), 'post')) {
            $criteria = new Criteria('topic_id', '(' . implode(',', $topic_id) . ')', 'IN');
//            /** @var \NewbbPostHandler $postHandler */

//            $postHandler = xoops_getModuleHandler('post', 'newbb');
            $postHandler->updateAll('forum_id', Request::getInt('newforum', 0, 'POST'), $criteria, true);
            $topicHandler->updateAll('forum_id', Request::getInt('newforum', 0, 'POST'), $criteria, true);
            $forumHandler->synchronization(Request::getInt('newforum', 0, 'POST'));
            $forumHandler->synchronization($forum_id);
        } else {
            include $GLOBALS['xoops']->path('header.php');
//            /** @var \NewbbCategoryHandler $categoryHandler */

//            $categoryHandler = xoops_getModuleHandler('category', 'newbb');
            $categories      = $categoryHandler->getByPermission('access');
            $forums          = $forumHandler->getForumsByCategory(array_keys($categories), 'post', false);

            $box = '<select name="newforum" size="1">';
            if (count($categories) > 0 && count($forums) > 0) {
                foreach (array_keys($forums) as $key) {

                    /** @var \NewbbCategory[] $categories */
                    $box .= "<option value='-1'>[" . $categories[$key]->getVar('cat_title') . ']</option>';
                    foreach ($forums[$key] as $forumid => $_forum) {
                        $box .= "<option value='" . $forumid . "'>-- " . $_forum['title'] . '</option>';
                        if (!isset($_forum['sub'])) {
                            continue;
                        }
                        foreach (array_keys($_forum['sub']) as $fid) {
                            $box .= "<option value='" . $fid . "'>---- " . $_forum['sub'][$fid]['title'] . '</option>';
                        }
                    }
                }
            } else {
                $box .= "<option value='-1'>" . _MD_NEWBB_NOFORUMINDB . '</option>';
            }
            $box .= '</select>';
            unset($forums, $categories);

            echo "<form action='" . Request::getString('PHP_SELF', '', 'SERVER') . "' method='post'>";
            echo "<table border='0' cellpadding='1' cellspacing='0' align='center' width='95%'>";
            echo "<tr><td class='bg2'>";
            echo "<table border='0' cellpadding='1' cellspacing='1' width='100%'>";
            echo '<tr><td class="bg3">' . _MD_NEWBB_MOVETOPICTO . '</td><td class="bg1">';
            echo $box;
            echo '</td></tr>';
            echo '<tr class="bg3"><td colspan="2" align="center">';
            echo "<input type='hidden' name='op' value='move' />";
            echo "<input type='hidden' name='forum_id' value='{$forum_id}' />";
            foreach ($topic_id as $id) {
                echo "<input type='hidden' name='topic_id[]' value='" . $id . "' />";
            }
            echo "<input type='submit' name='submit' value='" . _SUBMIT . "' />";
            echo '</td></tr></table></td></tr></table>';
            echo '</form>';
            include $GLOBALS['xoops']->path('footer.php');
            exit();
        }
        break;
}
///** @var \NewbbStatsHandler $statsHandler */
//$statsHandler = xoops_getModuleHandler('stats', 'newbb');
$statsHandler->reset();
if (empty($forum_id)) {
    redirect_header(XOOPS_URL . '/modules/newbb/list.topic.php', 2, _MD_NEWBB_DBUPDATED);
} else {
    redirect_header(XOOPS_URL . "/modules/newbb/viewforum.php?forum={$forum_id}", 2, _MD_NEWBB_DBUPDATED);
}
// irmtfan move to footer.php
include_once __DIR__ . '/footer.php';
include $GLOBALS['xoops']->path('footer.php');


1		<?php
2		/**
3		* NewBB 5.0x, the forum module for XOOPS project
4		*
5		* @copyright XOOPS Project (https://xoops.org)
6		* @license GNU GPL 2 or later (http://www.gnu.org/licenses/gpl-2.0.html)
7		* @author Taiwen Jiang (phppp or D.J.) <[email protected]>
8		* @since 4.00
9		* @package module::newbb
10		*/
11
12		use Xmf\Request;
13
14		include_once __DIR__ . '/header.php';
15
16		$forum_id = Request::getInt('forum_id', 0, 'POST');
17		$topic_id = Request::getArray('topic_id', null, 'POST');
18
19		$op = Request::getCmd('op', '', 'POST');
20		$op = in_array($op, ['approve', 'delete', 'restore', 'move'], true) ? $op : '';
21
22		if (0 === count($topic_id) \|\| 0 === count($op)) {
23		// irmtfan - issue with javascript:history.go(-1)
24		redirect_header(Request::getString('HTTP_REFERER', '', 'SERVER'), 2, _MD_NEWBB_NO_SELECTION);
25		}
26
27		$topic_id = array_values($topic_id);
28		///** @var \NewbbTopicHandler\|XoopsPersistableObjectHandler $topicHandler */
29		//$topicHandler = xoops_getModuleHandler('topic', 'newbb');
30		///** @var \NewbbForumHandler\|XoopsPersistableObjectHandler $forumHandler */
31		//$forumHandler = xoops_getModuleHandler('forum', 'newbb');
32
33		$isAdmin = newbbIsAdmin($forum_id);
34
35		if (!$isAdmin) {
36		redirect_header(XOOPS_URL . '/index.php', 2, _MD_NEWBB_NORIGHTTOACCESS);
37		}
38		switch ($op) {
39	View Code Duplication	case 'restore':
40		$forums = [];
41		$topicsObject = $topicHandler->getAll(new Criteria('topic_id', '(' . implode(',', $topic_id) . ')', 'IN'));
42		foreach (array_keys($topicsObject) as $id) {
43		/** @var \Topic $topicObject */
44		$topicObject = $topicsObject[$id];
45		$topicHandler->approve($topicObject);
46		$topicHandler->synchronization($topicObject);
47		$forums[$topicObject->getVar('forum_id')] = 1;
48		}
49		$criteria_forum = new Criteria('forum_id', '(' . implode(',', array_keys($forums)) . ')', 'IN');
50		$forumsObject = $forumHandler->getAll($criteria_forum);
51		foreach (array_keys($forumsObject) as $id) {
52		$forumHandler->synchronization($forumsObject[$id]);
53		}
54		unset($topicsObject, $forumsObject);
55		break;
56		case 'approve':
57		$forums = [];
58		$topicsObject = $topicHandler->getAll(new Criteria('topic_id', '(' . implode(',', $topic_id) . ')', 'IN'));
59		foreach (array_keys($topicsObject) as $id) {
60		/** @var \Topic $topicObject */
61		$topicObject = $topicsObject[$id];
62		$topicHandler->approve($topicObject);
63		$topicHandler->synchronization($topicObject);
64		$forums[$topicObject->getVar('forum_id')] = 1;
65		}
66
67		$criteria_forum = new Criteria('forum_id', '(' . implode(',', array_keys($forums)) . ')', 'IN');
68		$forumsObject = $forumHandler->getAll($criteria_forum);
69		foreach (array_keys($forumsObject) as $id) {
70		$forumHandler->synchronization($forumsObject[$id]);
71		}
72
73		if (empty($GLOBALS['xoopsModuleConfig']['notification_enabled'])) {
74		break;
75		}
76
77		include_once __DIR__ . '/include/notification.inc.php';
78		/** @var \XoopsNotificationHandler $notificationHandler */
79		$notificationHandler = xoops_getHandler('notification');
80		foreach (array_keys($topicsObject) as $id) {
81		$topicObject = $topicsObject[$id];
82		$tags = [];
83		$tags['THREAD_NAME'] = $topicObject->getVar('topic_title');
84		$tags['THREAD_URL'] = XOOPS_URL . '/modules/' . $moduleDirName . '/viewtopic.php?topic_id=' . $id . '&forum=' . $topicObject->getVar('forum_id');
85		/** @var \NewbbForum[] $forumsObject */
86		$tags['FORUM_NAME'] = $forumsObject[$topicObject->getVar('forum_id')]->getVar('forum_name');
87		$tags['FORUM_URL'] = XOOPS_URL . '/modules/' . $moduleDirName . '/viewforum.php?forum=' . $topicObject->getVar('forum_id');
88		$notificationHandler->triggerEvent('global', 0, 'new_thread', $tags);
89		$notificationHandler->triggerEvent('forum', $topicObject->getVar('forum_id'), 'new_thread', $tags);
90		$postObject = $topicHandler->getTopPost($id);
91		$tags['POST_URL'] = $tags['THREAD_URL'] . '#forumpost' . $postObject->getVar('post_id');
92		$notificationHandler->triggerEvent('thread', $id, 'new_post', $tags);
93		$notificationHandler->triggerEvent('forum', $topicObject->getVar('forum_id'), 'new_post', $tags);
94		$notificationHandler->triggerEvent('global', 0, 'new_post', $tags);
95		$tags['POST_CONTENT'] = $postObject->getVar('post_text');
96		$tags['POST_NAME'] = $postObject->getVar('subject');
97		$notificationHandler->triggerEvent('global', 0, 'new_fullpost', $tags);
98		$notificationHandler->triggerEvent('forum', $topicObject->getVar('forum_id'), 'new_fullpost', $tags);
99		unset($postObject);
100		}
101		unset($topicsObject, $forumsObject);
102		break;
103	View Code Duplication	case 'delete':
104		$forums = [];
105		/** @var \NewbbTopicHandler\|XoopsPersistableObjectHandler $topicHandler */
106		$topicsObject = $topicHandler->getAll(new Criteria('topic_id', '(' . implode(',', $topic_id) . ')', 'IN'));
107		foreach (array_keys($topicsObject) as $id) {
108		/** @var Topic $topicObject */
109		$topicObject = $topicsObject[$id];
110		// irmtfan should be set to false to not delete topic from database
111		$topicHandler->delete($topicObject, false);
112		$topicHandler->synchronization($topicObject);
113		$forums[$topicObject->getVar('forum_id')] = 1;
114		}
115
116		$criteria_forum = new Criteria('forum_id', '(' . implode(',', array_keys($forums)) . ')', 'IN');
117		$forumsObject = $forumHandler->getAll($criteria_forum);
118		foreach (array_keys($forumsObject) as $id) {
119		$forumHandler->synchronization($forumsObject[$id]);
120		}
121		unset($topicsObject, $forumsObject);
122		break;
123		case 'move':
124		if (Request::getInt('newforum', 0, 'POST')
125		&& Request::getInt('newforum', 0, 'POST') !== $forum_id
126		&& $forumHandler->getPermission(Request::getInt('newforum', 0, 'POST'), 'post')) {
127		$criteria = new Criteria('topic_id', '(' . implode(',', $topic_id) . ')', 'IN');
128		// /** @var \NewbbPostHandler $postHandler */
		0 ignored issues – show Unused Code Comprehensibility introduced 2017-09-15 15:10 UTC by Report Bug Copy Issue Report Show Similar Issues like this `42%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
129		// $postHandler = xoops_getModuleHandler('post', 'newbb');
130		$postHandler->updateAll('forum_id', Request::getInt('newforum', 0, 'POST'), $criteria, true);
131		$topicHandler->updateAll('forum_id', Request::getInt('newforum', 0, 'POST'), $criteria, true);
132		$forumHandler->synchronization(Request::getInt('newforum', 0, 'POST'));
133		$forumHandler->synchronization($forum_id);
134		} else {
135		include $GLOBALS['xoops']->path('header.php');
136		// /** @var \NewbbCategoryHandler $categoryHandler */
		0 ignored issues – show Unused Code Comprehensibility introduced 2017-09-15 15:10 UTC by Report Bug Copy Issue Report Show Similar Issues like this `42%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
137		// $categoryHandler = xoops_getModuleHandler('category', 'newbb');
138		$categories = $categoryHandler->getByPermission('access');
139		$forums = $forumHandler->getForumsByCategory(array_keys($categories), 'post', false);
140
141		$box = '<select name="newforum" size="1">';
142	View Code Duplication	if (count($categories) > 0 && count($forums) > 0) {
143		foreach (array_keys($forums) as $key) {
144
145		/** @var \NewbbCategory[] $categories */
146		$box .= "<option value='-1'>[" . $categories[$key]->getVar('cat_title') . ']</option>';
147		foreach ($forums[$key] as $forumid => $_forum) {
148		$box .= "<option value='" . $forumid . "'>-- " . $_forum['title'] . '</option>';
149		if (!isset($_forum['sub'])) {
150		continue;
151		}
152		foreach (array_keys($_forum['sub']) as $fid) {
153		$box .= "<option value='" . $fid . "'>---- " . $_forum['sub'][$fid]['title'] . '</option>';
154		}
155		}
156		}
157		} else {
158		$box .= "<option value='-1'>" . _MD_NEWBB_NOFORUMINDB . '</option>';
159		}
160		$box .= '</select>';
161		unset($forums, $categories);
162
163		echo "<form action='" . Request::getString('PHP_SELF', '', 'SERVER') . "' method='post'>";
164		echo "<table border='0' cellpadding='1' cellspacing='0' align='center' width='95%'>";
165		echo "<tr><td class='bg2'>";
166		echo "<table border='0' cellpadding='1' cellspacing='1' width='100%'>";
167		echo '<tr><td class="bg3">' . _MD_NEWBB_MOVETOPICTO . '</td><td class="bg1">';
168		echo $box;
169		echo '</td></tr>';
170		echo '<tr class="bg3"><td colspan="2" align="center">';
171		echo "<input type='hidden' name='op' value='move' />";
172		echo "<input type='hidden' name='forum_id' value='{$forum_id}' />";
173		foreach ($topic_id as $id) {
174		echo "<input type='hidden' name='topic_id[]' value='" . $id . "' />";
175		}
176		echo "<input type='submit' name='submit' value='" . _SUBMIT . "' />";
177		echo '</td></tr></table></td></tr></table>';
178		echo '</form>';
179		include $GLOBALS['xoops']->path('footer.php');
180		exit();
181		}
182		break;
183		}
184		///** @var \NewbbStatsHandler $statsHandler */
185		//$statsHandler = xoops_getModuleHandler('stats', 'newbb');
186		$statsHandler->reset();
187		if (empty($forum_id)) {
188		redirect_header(XOOPS_URL . '/modules/newbb/list.topic.php', 2, _MD_NEWBB_DBUPDATED);
189		} else {
190		redirect_header(XOOPS_URL . "/modules/newbb/viewforum.php?forum={$forum_id}", 2, _MD_NEWBB_DBUPDATED);
191		}
192		// irmtfan move to footer.php
193		include_once __DIR__ . '/footer.php';
194		include $GLOBALS['xoops']->path('footer.php');
195

mambax7 / newbb5

Push — master ( 62af87...249590 )

action.topic.php (2 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like