mambax7 / newbb5

moderate.php

Indentation +129 added lines, -129 removed lines

@@ -21,75 +21,75 @@ 
 $forum_id     = Request::getInt('forum', 0);
 $isAdmin      = newbbIsAdmin($forum_id);
 if (!$isAdmin) {
-    redirect_header(XOOPS_URL . '/index.php', 2, _MD_NEWBB_NORIGHTTOACCESS);
+	redirect_header(XOOPS_URL . '/index.php', 2, _MD_NEWBB_NORIGHTTOACCESS);
 }
 $is_administrator = $GLOBALS['xoopsUserIsAdmin'];
 ///** @var Newbb\ModerateHandler $moderateHandler */
 //$moderateHandler = Newbb\Helper::getInstance()->getHandler('Moderate');
 
 if (Request::hasVar('submit', 'POST') && Request::getInt('expire', 0, 'POST')) {
-    $ipWithMask = '';
-    if (0 == $forum_userid) {
-        $ipWithMask = Request::getString('ip', null, 'POST');
-        $mask       = '';
-        $ipParts    = explode('/', $ipWithMask);
-        $ip         = new \Xmf\IPAddress($ipParts[0]);
-        if (false === $ip->asReadable()) {
-            $ipWithMask = '';
-        } else {
-            $ipWithMask = $ip->asReadable();
-            $mask       = empty($ipParts[1]) ? 0 : (int)$ipParts[1];
-            $mask       = ($mask > ((4 === $ip->ipVersion()) ? 32 : 128) || $mask < 8) ? '' : $mask;
-            $ipWithMask .= empty($mask) ? '' : '/' . $mask;
-        }
-    }
-
-    $mod_end  = time() + Request::getInt('expire', 0, 'POST') * 3600 * 24;
-    $mod_desc = Request::getString('desc', '', 'POST');
-
-    $moderateObject = $moderateHandler->create();
-    $moderateObject->setVar('uid', $forum_userid);
-    $moderateObject->setVar('ip', $ipWithMask);
-    $moderateObject->setVar('forum_id', $forum_id);
-    $moderateObject->setVar('mod_start', time());
-    $moderateObject->setVar('mod_end', $mod_end);
-    $moderateObject->setVar('mod_desc', $mod_desc);
-    $res = $moderateHandler->insert($moderateObject);
-
-    redirect_header("moderate.php?forum={$forum_id}", 2, _MD_NEWBB_DBUPDATED);
+	$ipWithMask = '';
+	if (0 == $forum_userid) {
+		$ipWithMask = Request::getString('ip', null, 'POST');
+		$mask       = '';
+		$ipParts    = explode('/', $ipWithMask);
+		$ip         = new \Xmf\IPAddress($ipParts[0]);
+		if (false === $ip->asReadable()) {
+			$ipWithMask = '';
+		} else {
+			$ipWithMask = $ip->asReadable();
+			$mask       = empty($ipParts[1]) ? 0 : (int)$ipParts[1];
+			$mask       = ($mask > ((4 === $ip->ipVersion()) ? 32 : 128) || $mask < 8) ? '' : $mask;
+			$ipWithMask .= empty($mask) ? '' : '/' . $mask;
+		}
+	}
+
+	$mod_end  = time() + Request::getInt('expire', 0, 'POST') * 3600 * 24;
+	$mod_desc = Request::getString('desc', '', 'POST');
+
+	$moderateObject = $moderateHandler->create();
+	$moderateObject->setVar('uid', $forum_userid);
+	$moderateObject->setVar('ip', $ipWithMask);
+	$moderateObject->setVar('forum_id', $forum_id);
+	$moderateObject->setVar('mod_start', time());
+	$moderateObject->setVar('mod_end', $mod_end);
+	$moderateObject->setVar('mod_desc', $mod_desc);
+	$res = $moderateHandler->insert($moderateObject);
+
+	redirect_header("moderate.php?forum={$forum_id}", 2, _MD_NEWBB_DBUPDATED);
 } elseif (Request::hasVar('del')) {
-    $moderateObject = $moderateHandler->get(Request::getInt('del', 0, 'GET'));
-    if ($is_administrator || $moderateObject->getVar('forum_id') == $forum_id) {
-        $moderateHandler->delete($moderateObject, true);
-        redirect_header("moderate.php?forum={$forum_id}", 2, _MD_NEWBB_DBUPDATED);
-    }
+	$moderateObject = $moderateHandler->get(Request::getInt('del', 0, 'GET'));
+	if ($is_administrator || $moderateObject->getVar('forum_id') == $forum_id) {
+		$moderateHandler->delete($moderateObject, true);
+		redirect_header("moderate.php?forum={$forum_id}", 2, _MD_NEWBB_DBUPDATED);
+	}
 }
 
 $start    = Request::getInt('start', 0, 'GET');
 $sortname = Request::getString('sort', '', 'GET');
 
 switch ($sortname) {
-    case 'uid':
-        $sort  = 'uid ASC, ip';
-        $order = 'ASC';
-        break;
-    case 'start':
-        $sort  = 'mod_start';
-        $order = 'ASC';
-        break;
-    case 'expire':
-        $sort  = 'mod_end';
-        $order = 'DESC';
-        break;
-    default:
-        $sort  = 'forum_id ASC, uid ASC, ip';
-        $order = 'ASC';
-        break;
+	case 'uid':
+		$sort  = 'uid ASC, ip';
+		$order = 'ASC';
+		break;
+	case 'start':
+		$sort  = 'mod_start';
+		$order = 'ASC';
+		break;
+	case 'expire':
+		$sort  = 'mod_end';
+		$order = 'DESC';
+		break;
+	default:
+		$sort  = 'forum_id ASC, uid ASC, ip';
+		$order = 'ASC';
+		break;
 }
 // show all bans for module admin - for moderator just show its forum_id bans
 $criteria = new \CriteriaCompo();
 if (!$is_administrator) {
-    $criteria->add(new \Criteria('forum_id', $forum_id, '='));
+	$criteria->add(new \Criteria('forum_id', $forum_id, '='));
 }
 $criteria->setLimit($GLOBALS['xoopsModuleConfig']['topics_per_page']);
 $criteria->setStart($start);
@@ -100,75 +100,75 @@ 
 
 $url = 'moderate.php';
 if ($forum_id) {
-    $url .= '?forum=' . $forum_id;
+	$url .= '?forum=' . $forum_id;
 }
 $xoopsTpl->assign('moderate_url', $url);
 
 if (!empty($moderate_count)) {
-    $_users = [];
-    foreach (array_keys($moderateObjects) as $id) {
-        $_users[$moderateObjects[$id]->getVar('uid')] = 1;
-    }
-    $users = newbbGetUnameFromIds(array_keys($_users), $GLOBALS['xoopsModuleConfig']['show_realname'], true);
-
-    $columnHeaders ['uid']    = [
-        'url'    => 'moderate.php?forum=' . $forum_id . '&amp;start=' . $start . '&amp;sort=uid',
-        'header' => _MD_NEWBB_SUSPEND_UID,
-        'title'  => _MD_NEWBB_SUSPEND_UID,
-    ];
-    $columnHeaders ['start']  = [
-        'url'    => 'moderate.php?forum=' . $forum_id . '&amp;start=' . $start . '&amp;sort=start',
-        'header' => _MD_NEWBB_SUSPEND_START,
-        'title'  => _MD_NEWBB_SUSPEND_START,
-    ];
-    $columnHeaders['expire']  = [
-        'url'    => 'moderate.php?forum=' . $forum_id . '&amp;start=' . $start . '&amp;sort=expire',
-        'header' => _MD_NEWBB_SUSPEND_EXPIRE,
-        'title'  => _MD_NEWBB_SUSPEND_EXPIRE,
-    ];
-    $columnHeaders['forum']   = [
-        'url'    => 'moderate.php?forum=' . $forum_id . '&amp;start=' . $start . '&amp;sort=forum',
-        'header' => _MD_NEWBB_SUSPEND_SCOPE,
-        'title'  => _MD_NEWBB_SUSPEND_SCOPE,
-    ];
-    $columnHeaders['desc']    = [
-        'url'    => false,
-        'header' => _MD_NEWBB_SUSPEND_DESC,
-        'title'  => _MD_NEWBB_SUSPEND_DESC,
-    ];
-    $columnHeaders['options'] = [
-        'url'    => false,
-        'header' => _DELETE,
-        'title'  => _DELETE,
-    ];
-    $xoopsTpl->assign('columnHeaders', $columnHeaders);
-
-    //    /** @var Newbb\ForumHandler $forumHandler */
-    //    $forumHandler = Newbb\Helper::getInstance()->getHandler('Forum');
-    $forum_list = $forumHandler->getAll(null, ['forum_name'], false);
-
-    $columnRows = [];
-    foreach (array_keys($moderateObjects) as $id) {
-        // for anon, show ip instead
-        $row['uid']     = ($moderateObjects[$id]->getVar('uid') ? (isset($users[$moderateObjects[$id]->getVar('uid')]) ? $users[$moderateObjects[$id]->getVar('uid')] : $moderateObjects[$id]->getVar('uid')) : $moderateObjects[$id]->getVar('ip'));
-        $row['start']   = formatTimestamp($moderateObjects[$id]->getVar('mod_start'));
-        $row['expire']  = formatTimestamp($moderateObjects[$id]->getVar('mod_end'));
-        $row['forum']   = ($moderateObjects[$id]->getVar('forum_id') ? $forum_list[$moderateObjects[$id]->getVar('forum_id')]['forum_name'] : _ALL);
-        $row['desc']    = ($moderateObjects[$id]->getVar('mod_desc') ?: _NONE);
-        $row['options'] = (($is_administrator
-                            || $moderateObjects[$id]->getVar('forum_id') == $forum_id) ? '<a href="moderate.php?forum=' . $forum_id . '&amp;del=' . $moderateObjects[$id]->getVar('mod_id') . '">' . _DELETE . '</a>' : '');
-        $columnRows[]   = $row;
-    }
-    $xoopsTpl->assign('columnRows', $columnRows);
-
-    if ($moderate_count > $GLOBALS['xoopsModuleConfig']['topics_per_page']) {
-        require_once $GLOBALS['xoops']->path('class/pagenav.php');
-        $nav = new \XoopsPageNav($moderate_count, $GLOBALS['xoopsModuleConfig']['topics_per_page'], $start, 'start', 'forum=' . $forum_id . '&amp;sort=' . $sortname);
-        //if (isset($GLOBALS['xoopsModuleConfig']['do_rewrite'])) {
-        //    $nav->url = formatURL(Request::getString('SERVER_NAME', '', 'SERVER')) . ' /' . $nav->url;
-        //}
-        $xoopsTpl->assign('moderate_page_nav', $nav->renderNav());
-    }
+	$_users = [];
+	foreach (array_keys($moderateObjects) as $id) {
+		$_users[$moderateObjects[$id]->getVar('uid')] = 1;
+	}
+	$users = newbbGetUnameFromIds(array_keys($_users), $GLOBALS['xoopsModuleConfig']['show_realname'], true);
+
+	$columnHeaders ['uid']    = [
+		'url'    => 'moderate.php?forum=' . $forum_id . '&amp;start=' . $start . '&amp;sort=uid',
+		'header' => _MD_NEWBB_SUSPEND_UID,
+		'title'  => _MD_NEWBB_SUSPEND_UID,
+	];
+	$columnHeaders ['start']  = [
+		'url'    => 'moderate.php?forum=' . $forum_id . '&amp;start=' . $start . '&amp;sort=start',
+		'header' => _MD_NEWBB_SUSPEND_START,
+		'title'  => _MD_NEWBB_SUSPEND_START,
+	];
+	$columnHeaders['expire']  = [
+		'url'    => 'moderate.php?forum=' . $forum_id . '&amp;start=' . $start . '&amp;sort=expire',
+		'header' => _MD_NEWBB_SUSPEND_EXPIRE,
+		'title'  => _MD_NEWBB_SUSPEND_EXPIRE,
+	];
+	$columnHeaders['forum']   = [
+		'url'    => 'moderate.php?forum=' . $forum_id . '&amp;start=' . $start . '&amp;sort=forum',
+		'header' => _MD_NEWBB_SUSPEND_SCOPE,
+		'title'  => _MD_NEWBB_SUSPEND_SCOPE,
+	];
+	$columnHeaders['desc']    = [
+		'url'    => false,
+		'header' => _MD_NEWBB_SUSPEND_DESC,
+		'title'  => _MD_NEWBB_SUSPEND_DESC,
+	];
+	$columnHeaders['options'] = [
+		'url'    => false,
+		'header' => _DELETE,
+		'title'  => _DELETE,
+	];
+	$xoopsTpl->assign('columnHeaders', $columnHeaders);
+
+	//    /** @var Newbb\ForumHandler $forumHandler */
+	//    $forumHandler = Newbb\Helper::getInstance()->getHandler('Forum');
+	$forum_list = $forumHandler->getAll(null, ['forum_name'], false);
+
+	$columnRows = [];
+	foreach (array_keys($moderateObjects) as $id) {
+		// for anon, show ip instead
+		$row['uid']     = ($moderateObjects[$id]->getVar('uid') ? (isset($users[$moderateObjects[$id]->getVar('uid')]) ? $users[$moderateObjects[$id]->getVar('uid')] : $moderateObjects[$id]->getVar('uid')) : $moderateObjects[$id]->getVar('ip'));
+		$row['start']   = formatTimestamp($moderateObjects[$id]->getVar('mod_start'));
+		$row['expire']  = formatTimestamp($moderateObjects[$id]->getVar('mod_end'));
+		$row['forum']   = ($moderateObjects[$id]->getVar('forum_id') ? $forum_list[$moderateObjects[$id]->getVar('forum_id')]['forum_name'] : _ALL);
+		$row['desc']    = ($moderateObjects[$id]->getVar('mod_desc') ?: _NONE);
+		$row['options'] = (($is_administrator
+							|| $moderateObjects[$id]->getVar('forum_id') == $forum_id) ? '<a href="moderate.php?forum=' . $forum_id . '&amp;del=' . $moderateObjects[$id]->getVar('mod_id') . '">' . _DELETE . '</a>' : '');
+		$columnRows[]   = $row;
+	}
+	$xoopsTpl->assign('columnRows', $columnRows);
+
+	if ($moderate_count > $GLOBALS['xoopsModuleConfig']['topics_per_page']) {
+		require_once $GLOBALS['xoops']->path('class/pagenav.php');
+		$nav = new \XoopsPageNav($moderate_count, $GLOBALS['xoopsModuleConfig']['topics_per_page'], $start, 'start', 'forum=' . $forum_id . '&amp;sort=' . $sortname);
+		//if (isset($GLOBALS['xoopsModuleConfig']['do_rewrite'])) {
+		//    $nav->url = formatURL(Request::getString('SERVER_NAME', '', 'SERVER')) . ' /' . $nav->url;
+		//}
+		$xoopsTpl->assign('moderate_page_nav', $nav->renderNav());
+	}
 }
 
 require_once $GLOBALS['xoops']->path('class/xoopsformloader.php');
@@ -179,19 +179,19 @@ 
 $forum_form->addElement(new \XoopsFormText(_MD_NEWBB_SUSPEND_DESC, 'desc', 50, 255));
 require_once __DIR__ . '/include/functions.forum.php';
 if (newbbIsAdmin()) {
-    $forumSel = '<select name="forum">';// if user doesn't select, default is "0" all forums
-    $forumSel .= '<option value="0"';
-    if (0 == $forum_id) {
-        $forumSel .= ' selected';
-    }
-    $forumSel                         .= '>' . _ALL . '</option>';
-    $forumSel                         .= newbbForumSelectBox($forum_id, 'access', false); //$accessForums, $permission = "access", $delimitorCategory = true
-    $forumSel                         .= '</select>';
-    $forumEle                         = new \XoopsFormLabel(_MD_NEWBB_SELFORUM, $forumSel);
-    $forumEle->customValidationCode[] = 'if (document.suspend.forum.value < 0) {return false;} ';
-    $forum_form->addElement($forumEle);
+	$forumSel = '<select name="forum">';// if user doesn't select, default is "0" all forums
+	$forumSel .= '<option value="0"';
+	if (0 == $forum_id) {
+		$forumSel .= ' selected';
+	}
+	$forumSel                         .= '>' . _ALL . '</option>';
+	$forumSel                         .= newbbForumSelectBox($forum_id, 'access', false); //$accessForums, $permission = "access", $delimitorCategory = true
+	$forumSel                         .= '</select>';
+	$forumEle                         = new \XoopsFormLabel(_MD_NEWBB_SELFORUM, $forumSel);
+	$forumEle->customValidationCode[] = 'if (document.suspend.forum.value < 0) {return false;} ';
+	$forum_form->addElement($forumEle);
 } else {
-    $forum_form->addElement(new \XoopsFormHidden('forum', $forum_id));
+	$forum_form->addElement(new \XoopsFormHidden('forum', $forum_id));
 }
 $forum_form->addElement(new \XoopsFormButton('', 'submit', _SUBMIT, 'submit'));
 $forum_form->assign($xoopsTpl);

makepdf.php

Indentation +15 added lines, -15 removed lines

@@ -31,19 +31,19 @@ 
 $post_id   = Request::getInt('post_id', 0, 'GET');
 
 if (!is_file(XOOPS_ROOT_PATH . '/class/libraries/vendor/tecnickcom/tcpdf/tcpdf.php')) {
-    redirect_header(XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewtopic.php?topic_id=' . $topic_id, 3, 'TCPDF for Xoops not installed');
+	redirect_header(XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname') . '/viewtopic.php?topic_id=' . $topic_id, 3, 'TCPDF for Xoops not installed');
 } else {
-    require_once XOOPS_ROOT_PATH . '/class/libraries/vendor/tecnickcom/tcpdf/tcpdf.php';
+	require_once XOOPS_ROOT_PATH . '/class/libraries/vendor/tecnickcom/tcpdf/tcpdf.php';
 }
 
 if (empty($post_id)) {
-    exit(_MD_NEWBB_ERRORTOPIC);
+	exit(_MD_NEWBB_ERRORTOPIC);
 }
 ///** @var Newbb\PostHandler $postHandler */
 //$postHandler = Newbb\Helper::getInstance()->getHandler('Post');
 $post = $postHandler->get($post_id);
 if (!$approved = $post->getVar('approved')) {
-    exit(_MD_NEWBB_NORIGHTTOVIEW);
+	exit(_MD_NEWBB_NORIGHTTOVIEW);
 }
 $post_data = $postHandler->getPostForPDF($post);
 //$post_edit = $post->displayPostEdit();  //reserve for future versions to display edit records
@@ -52,7 +52,7 @@ 
 $forumtopic = $topicHandler->getByPost($post_id);
 $topic_id   = $forumtopic->getVar('topic_id');
 if (!$approved = $forumtopic->getVar('approved')) {
-    exit(_MD_NEWBB_NORIGHTTOVIEW);
+	exit(_MD_NEWBB_NORIGHTTOVIEW);
 }
 ///** @var Newbb\ForumHandler $forumHandler */
 //$forumHandler    = Newbb\Helper::getInstance()->getHandler('Forum');
@@ -62,19 +62,19 @@ 
 $parent_forums   = $forumHandler->getParents($viewtopic_forum);
 $pf_title        = '';
 if ($parent_forums) {
-    foreach ($parent_forums as $p_f) {
-        $pf_title .= $p_f['forum_name'] . ' - ';
-    }
+	foreach ($parent_forums as $p_f) {
+		$pf_title .= $p_f['forum_name'] . ' - ';
+	}
 }
 if (!$forumHandler->getPermission($viewtopic_forum)) {
-    exit(_MD_NEWBB_NORIGHTTOACCESS);
+	exit(_MD_NEWBB_NORIGHTTOACCESS);
 }
 if (!$topicHandler->getPermission($viewtopic_forum, $forumtopic->getVar('topic_status'), 'view')) {
-    exit(_MD_NEWBB_NORIGHTTOVIEW);
+	exit(_MD_NEWBB_NORIGHTTOVIEW);
 }
 // irmtfan add pdf permission
 if (!$topicHandler->getPermission($viewtopic_forum, $forumtopic->getVar('topic_status'), 'pdf')) {
-    exit(_MD_NEWBB_NORIGHTTOPDF);
+	exit(_MD_NEWBB_NORIGHTTOPDF);
 }
 //$categoryHandler = Newbb\Helper::getInstance()->getHandler('Category');
 $cat                                 = $viewtopic_forum->getVar('cat_id');
@@ -98,11 +98,11 @@ 
 $pdf_data['HeadSecondLine'] = _MD_NEWBB_FORUMHOME . ' - ' . $pdf_data['cat_title'] . ' - ' . $pdf_data['forum_title'] . ' - ' . $pdf_data['topic_title'];
 // START irmtfan to implement EMLH by GIJ
 if (function_exists('easiestml')) {
-    $pdf_data = easiestml($pdf_data);
+	$pdf_data = easiestml($pdf_data);
 // END irmtfan to implement EMLH by GIJ
-    // START irmtfan to implement Xlanguage by phppp(DJ)
+	// START irmtfan to implement Xlanguage by phppp(DJ)
 } elseif (function_exists('xlanguage_ml')) {
-    $pdf_data = xlanguage_ml($pdf_data);
+	$pdf_data = xlanguage_ml($pdf_data);
 }
 // END irmtfan to implement Xlanguage by phppp(DJ)
 
@@ -129,7 +129,7 @@ 
 // START irmtfan hack to add RTL-LTR local
 // until _RTL added to core 2.6.0
 if (!defined('_RTL')) {
-    define('_RTL', false);
+	define('_RTL', false);
 }
 $pdf->setRTL(_RTL);
 // END irmtfan hack to add RTL-LTR local

digest.php

Indentation +3 added lines, -3 removed lines

@@ -13,14 +13,14 @@
 
 // Why the skip-DB-security check defined only for XMLRPC? We also need it!!! ~_*
 if (!defined('XOOPS_XMLRPC')) {
-    define('XOOPS_XMLRPC', 1);
+	define('XOOPS_XMLRPC', 1);
 }
 ob_start();
 require_once __DIR__ . '/header.php';
 if (0 == $GLOBALS['xoopsModuleConfig']['email_digest']) {
-    echo '<br>Not set';
+	echo '<br>Not set';
 
-    return false;
+	return false;
 }
 /** @var Newbb\DigestHandler $digestHandler */
 $digestHandler = Newbb\Helper::getInstance()->getHandler('Digest');

blocks/list_topic.php

Indentation +156 added lines, -156 removed lines

@@ -15,7 +15,7 @@ 
 // defined('XOOPS_ROOT_PATH') || die('Restricted access');
 
 if (defined('LIST_TOPIC_DEFINED')) {
-    return;
+	return;
 }
 define('LIST_TOPIC_DEFINED', true);
 
@@ -51,47 +51,47 @@ 
  */
 function newbb_list_topic_show($options)
 {
-    $newbbConfig = newbbLoadConfig(); // load all newbb configs
-
-    $topicRenderer            = new Newbb\TopicRenderer();
-    $topicRenderer->userlevel = $GLOBALS['xoopsUserIsAdmin'] ? 2 : is_object($GLOBALS['xoopsUser']); // Vistitor's level: 0 - anonymous; 1 - user; 2 - moderator or admin
-
-    $topicRenderer->force = true; // force against static vars for parse
-
-    $topicRenderer->is_multiple = true; // is it for multiple forums
-    $topicRenderer->config      =& $newbbConfig; // get all configs
-    if (!empty($options[6])) {
-        $topicRenderer->config['topics_per_page'] = (int)$options[6]; // number of topics (items) to display
-    }
-    $topicRenderer->config['topic_title_excerpt'] = (int)$options[10]; // topic title length 0 = dont excerpt
-    $topicRenderer->config['post_excerpt']        = (int)$options[11]; // post text excerpt 0 = no post text
-
-    $optionsStatus = explode(',', $options[0]); // status in where claus
-    $optionsForum  = explode(',', $options[12]);
-
-    // set and parse values:
-    // forum: parse positive values to forum IDs and negative values to category IDs. value=0 => all valid forums
-    $topicRenderer->setVars([
-                                'status'     => $optionsStatus,
-                                'uid'        => $options[1],
-                                'lastposter' => $options[2],
-                                'type'       => $options[3],
-                                'sort'       => $options[4],
-                                'order'      => $options[5],
-                                'since'      => $options[7],
-                                'forum'      => $optionsForum
-                            ]);
-    $block = [];
-    // headers to display in block
-    $block['headers'] = $topicRenderer->getHeader($options[8]);
-
-    // render a list of topics using all above criterias
-    list($block['topics'], $block['sticky']) = $topicRenderer->renderTopics();
-
-    // show index navigation
-    $block['indexNav'] = !empty($options[9]);
-
-    return $block;
+	$newbbConfig = newbbLoadConfig(); // load all newbb configs
+
+	$topicRenderer            = new Newbb\TopicRenderer();
+	$topicRenderer->userlevel = $GLOBALS['xoopsUserIsAdmin'] ? 2 : is_object($GLOBALS['xoopsUser']); // Vistitor's level: 0 - anonymous; 1 - user; 2 - moderator or admin
+
+	$topicRenderer->force = true; // force against static vars for parse
+
+	$topicRenderer->is_multiple = true; // is it for multiple forums
+	$topicRenderer->config      =& $newbbConfig; // get all configs
+	if (!empty($options[6])) {
+		$topicRenderer->config['topics_per_page'] = (int)$options[6]; // number of topics (items) to display
+	}
+	$topicRenderer->config['topic_title_excerpt'] = (int)$options[10]; // topic title length 0 = dont excerpt
+	$topicRenderer->config['post_excerpt']        = (int)$options[11]; // post text excerpt 0 = no post text
+
+	$optionsStatus = explode(',', $options[0]); // status in where claus
+	$optionsForum  = explode(',', $options[12]);
+
+	// set and parse values:
+	// forum: parse positive values to forum IDs and negative values to category IDs. value=0 => all valid forums
+	$topicRenderer->setVars([
+								'status'     => $optionsStatus,
+								'uid'        => $options[1],
+								'lastposter' => $options[2],
+								'type'       => $options[3],
+								'sort'       => $options[4],
+								'order'      => $options[5],
+								'since'      => $options[7],
+								'forum'      => $optionsForum
+							]);
+	$block = [];
+	// headers to display in block
+	$block['headers'] = $topicRenderer->getHeader($options[8]);
+
+	// render a list of topics using all above criterias
+	list($block['topics'], $block['sticky']) = $topicRenderer->renderTopics();
+
+	// show index navigation
+	$block['indexNav'] = !empty($options[9]);
+
+	return $block;
 }
 
 /**
@@ -100,118 +100,118 @@ 
  */
 function newbb_list_topic_edit($options)
 {
-    // require_once $GLOBALS['xoops']->path('class/blockform.php'); //reserve for 2.6
-    xoops_load('XoopsFormLoader');
-    // $form = new \XoopsBlockForm(); //reserve for 2.6
-    $form = new \XoopsThemeForm(_MB_NEWBB_DISPLAYMODE_DESC, 'list_topic', '');
-
-    $topicRenderer            = new Newbb\TopicRenderer();
-    $topicRenderer->userlevel = 2; // 2 - moderator or admin
-
-    // status element
-    $optionsStatus = explode(',', $options[0]);
-    $statusEle     = new \XoopsFormSelect(_MB_NEWBB_CRITERIA, 'options[0]', $optionsStatus, 5, true);
-    $status        = $topicRenderer->getStatus($topicRenderer->userlevel); // get all public status + admin status (admin mode, pending deleted)
-    $statusEle->addOptionArray($status);
-    $statusEle->setExtra("onchange = \"validate('options[0][]','select', true)\""); // if user dont select any option it select "all"
-    $statusEle->setDescription(_MB_NEWBB_CRITERIA_DESC);
-
-    // topic_poster element
-    $topicPosterRadioEle = new \XoopsFormRadio(_MB_NEWBB_AUTHOR, 'options[1]', $options[1]);
-    $topicPosterRadioEle->addOption(-1, _MD_NEWBB_TOTALUSER);
-    $topicPosterRadioEle->addOption((-1 !== $options[1]) ? $options[1] : 0, _SELECT); // if no user in selection box it select uid=0 anon users
-    $topicPosterRadioEle->setExtra("onchange=\"var el=document.getElementById('options[1]'); el.disabled=(this.id == 'options[1]1'); if (!el.value) {el.value= this.value}\""); // if user dont select any option it select "all"
-    $topicPosterSelectEle = new \XoopsFormSelectUser(_MB_NEWBB_AUTHOR, 'options[1]', true, explode(',', $options[1]), 5, true);// show $limit = 200 users when no user is selected;
-    $topicPosterEle       = new \XoopsFormLabel(_MB_NEWBB_AUTHOR, $topicPosterRadioEle->render() . $topicPosterSelectEle->render());
-
-    // lastposter element
-    $lastPosterRadioEle = new \XoopsFormRadio(_MD_NEWBB_POSTER, 'options[2]', $options[2]);
-    $lastPosterRadioEle->addOption(-1, _MD_NEWBB_TOTALUSER);
-    $lastPosterRadioEle->addOption((-1 !== $options[2]) ? $options[2] : 0, _SELECT); // if no user in selection box it select uid=1
-    $lastPosterRadioEle->setExtra("onchange=\"var el=document.getElementById('options[2]'); el.disabled=(this.id == 'options[2]1'); if (!el.value) {el.value= this.value}\""); // if user dont select any option it select "all"
-    $lastPosterSelectEle = new \XoopsFormSelectUser(_MD_NEWBB_POSTER, 'options[2]', true, explode(',', $options[2]), 5, true);// show $limit = 200 users when no user is selected;
-    $lastPosterEle       = new \XoopsFormLabel(_MD_NEWBB_POSTER, $lastPosterRadioEle->render() . $lastPosterSelectEle->render());
-
-    // type element
-    $types   = $topicRenderer->getTypes(); // get all available types in all forums
-    $typeEle = new \XoopsFormSelect(_MD_NEWBB_TYPE, 'options[3]', $options[3]);
-    $typeEle->addOption(0, _NONE);
-    if (!empty($types)) {
-        foreach ($types as $type_id => $type) {
-            $typeEle->addOption($type_id, $type['type_name']);
-        }
-    }
-
-    // sort element
-    $sortEle = new \XoopsFormSelect(_MD_NEWBB_SORTBY, 'options[4]', $options[4]);
-    $sortEle->setDescription(_MB_NEWBB_CRITERIA_SORT_DESC);
-    $sorts = $topicRenderer->getSort('', 'title');
-    $sortEle->addOptionArray($sorts);
-
-    // order element
-    $orderEle = new \XoopsFormSelect(_MB_NEWBB_CRITERIA_ORDER, 'options[5]', $options[5]);
-    $orderEle->addOption(0, _DESCENDING);
-    $orderEle->addOption(1, _ASCENDING);
-
-    // number of topics to display element
-    $numdispEle = new \XoopsFormText(_MB_NEWBB_DISPLAY, 'options[6]', 10, 255, (int)$options[6]);
-
-    $timeEle = new \XoopsFormText(_MB_NEWBB_TIME, 'options[7]', 10, 255, $options[7]);
-    $timeEle->setDescription(_MB_NEWBB_TIME_DESC);
-
-    // mode disp element
-    $options_headers = explode(',', $options[8]);
-    $modeEle         = new \XoopsFormCheckBox(_MB_NEWBB_DISPLAYMODE, 'options[8][]', $options_headers);
-    $modeEle->setDescription(_MB_NEWBB_DISPLAYMODE_DESC);
-    $modeEle->columns = 4;
-    $disps            = $topicRenderer->getHeader();
-    $modeEle->addOptionArray($disps);
-    $modeEle->setExtra("onchange = \"validate('options[8][]','checkbox', true)\""); // prevent user select no option
-    // Index navigation element
-    $navEle = new \XoopsFormRadioYN(_MB_NEWBB_INDEXNAV, 'options[9]', !empty($options[9]));
-
-    // Topic title element
-    $lengthEle = new \XoopsFormText(_MB_NEWBB_TITLE_LENGTH, 'options[10]', 10, 255, (int)$options[10]);
-    $lengthEle->setDescription(_MB_NEWBB_TITLE_LENGTH_DESC);
-
-    // Post text element
-    $postExcerptEle = new \XoopsFormText(_MB_NEWBB_POST_EXCERPT, 'options[11]', 10, 255, (int)$options[11]);
-    $postExcerptEle->setDescription(_MB_NEWBB_POST_EXCERPT_DESC);
-
-    //  forum element
-    $optionsForum = explode(',', $options[12]);
-    require_once  dirname(__DIR__) . '/include/functions.forum.php';
-    /** @var Newbb\ForumHandler $forumHandler */
-    $forumHandler = Newbb\Helper::getInstance()->getHandler('Forum');
-    //get forum Ids by values. parse positive values to forum IDs and negative values to category IDs. value=0 => all valid forums
-    // Get accessible forums
-    $accessForums = $forumHandler->getIdsByValues(array_map('intval', $optionsForum));
-    $isAll        = (0 === count($optionsForum) || empty($optionsForum[0]));
-    $forumSel     = "<select name=\"options[12][]\" multiple=\"multiple\" onchange = \"validate('options[12][]','select', true)\">";// if user dont select any it select "0"
-    $forumSel     .= '<option value="0" ';
-    if ($isAll) {
-        $forumSel     .= ' selected';
-        $accessForums = null; // just select _ALL option
-    }
-    $forumSel .= '>' . _ALL . '</option>';
-    $forumSel .= newbbForumSelectBox($accessForums, 'access', false); //$accessForums, $permission = "access", $delimitorCategory = false
-    $forumSel .= '</select>';
-    $forumEle = new \XoopsFormLabel(_MB_NEWBB_FORUMLIST, $forumSel);
-
-    // add all elements to form
-    $form->addElement($statusEle);
-    $form->addElement($topicPosterEle);
-    $form->addElement($lastPosterEle);
-    $form->addElement($typeEle);
-    $form->addElement($sortEle);
-    $form->addElement($orderEle);
-    $form->addElement($numdispEle);
-    $form->addElement($timeEle);
-    $form->addElement($modeEle, true); // required: user should select at least one otherwise it will select the first one
-    $form->addElement($navEle);
-    $form->addElement($lengthEle);
-    $form->addElement($postExcerptEle);
-    $form->addElement($forumEle);
-
-    return $form->render();
+	// require_once $GLOBALS['xoops']->path('class/blockform.php'); //reserve for 2.6
+	xoops_load('XoopsFormLoader');
+	// $form = new \XoopsBlockForm(); //reserve for 2.6
+	$form = new \XoopsThemeForm(_MB_NEWBB_DISPLAYMODE_DESC, 'list_topic', '');
+
+	$topicRenderer            = new Newbb\TopicRenderer();
+	$topicRenderer->userlevel = 2; // 2 - moderator or admin
+
+	// status element
+	$optionsStatus = explode(',', $options[0]);
+	$statusEle     = new \XoopsFormSelect(_MB_NEWBB_CRITERIA, 'options[0]', $optionsStatus, 5, true);
+	$status        = $topicRenderer->getStatus($topicRenderer->userlevel); // get all public status + admin status (admin mode, pending deleted)
+	$statusEle->addOptionArray($status);
+	$statusEle->setExtra("onchange = \"validate('options[0][]','select', true)\""); // if user dont select any option it select "all"
+	$statusEle->setDescription(_MB_NEWBB_CRITERIA_DESC);
+
+	// topic_poster element
+	$topicPosterRadioEle = new \XoopsFormRadio(_MB_NEWBB_AUTHOR, 'options[1]', $options[1]);
+	$topicPosterRadioEle->addOption(-1, _MD_NEWBB_TOTALUSER);
+	$topicPosterRadioEle->addOption((-1 !== $options[1]) ? $options[1] : 0, _SELECT); // if no user in selection box it select uid=0 anon users
+	$topicPosterRadioEle->setExtra("onchange=\"var el=document.getElementById('options[1]'); el.disabled=(this.id == 'options[1]1'); if (!el.value) {el.value= this.value}\""); // if user dont select any option it select "all"
+	$topicPosterSelectEle = new \XoopsFormSelectUser(_MB_NEWBB_AUTHOR, 'options[1]', true, explode(',', $options[1]), 5, true);// show $limit = 200 users when no user is selected;
+	$topicPosterEle       = new \XoopsFormLabel(_MB_NEWBB_AUTHOR, $topicPosterRadioEle->render() . $topicPosterSelectEle->render());
+
+	// lastposter element
+	$lastPosterRadioEle = new \XoopsFormRadio(_MD_NEWBB_POSTER, 'options[2]', $options[2]);
+	$lastPosterRadioEle->addOption(-1, _MD_NEWBB_TOTALUSER);
+	$lastPosterRadioEle->addOption((-1 !== $options[2]) ? $options[2] : 0, _SELECT); // if no user in selection box it select uid=1
+	$lastPosterRadioEle->setExtra("onchange=\"var el=document.getElementById('options[2]'); el.disabled=(this.id == 'options[2]1'); if (!el.value) {el.value= this.value}\""); // if user dont select any option it select "all"
+	$lastPosterSelectEle = new \XoopsFormSelectUser(_MD_NEWBB_POSTER, 'options[2]', true, explode(',', $options[2]), 5, true);// show $limit = 200 users when no user is selected;
+	$lastPosterEle       = new \XoopsFormLabel(_MD_NEWBB_POSTER, $lastPosterRadioEle->render() . $lastPosterSelectEle->render());
+
+	// type element
+	$types   = $topicRenderer->getTypes(); // get all available types in all forums
+	$typeEle = new \XoopsFormSelect(_MD_NEWBB_TYPE, 'options[3]', $options[3]);
+	$typeEle->addOption(0, _NONE);
+	if (!empty($types)) {
+		foreach ($types as $type_id => $type) {
+			$typeEle->addOption($type_id, $type['type_name']);
+		}
+	}
+
+	// sort element
+	$sortEle = new \XoopsFormSelect(_MD_NEWBB_SORTBY, 'options[4]', $options[4]);
+	$sortEle->setDescription(_MB_NEWBB_CRITERIA_SORT_DESC);
+	$sorts = $topicRenderer->getSort('', 'title');
+	$sortEle->addOptionArray($sorts);
+
+	// order element
+	$orderEle = new \XoopsFormSelect(_MB_NEWBB_CRITERIA_ORDER, 'options[5]', $options[5]);
+	$orderEle->addOption(0, _DESCENDING);
+	$orderEle->addOption(1, _ASCENDING);
+
+	// number of topics to display element
+	$numdispEle = new \XoopsFormText(_MB_NEWBB_DISPLAY, 'options[6]', 10, 255, (int)$options[6]);
+
+	$timeEle = new \XoopsFormText(_MB_NEWBB_TIME, 'options[7]', 10, 255, $options[7]);
+	$timeEle->setDescription(_MB_NEWBB_TIME_DESC);
+
+	// mode disp element
+	$options_headers = explode(',', $options[8]);
+	$modeEle         = new \XoopsFormCheckBox(_MB_NEWBB_DISPLAYMODE, 'options[8][]', $options_headers);
+	$modeEle->setDescription(_MB_NEWBB_DISPLAYMODE_DESC);
+	$modeEle->columns = 4;
+	$disps            = $topicRenderer->getHeader();
+	$modeEle->addOptionArray($disps);
+	$modeEle->setExtra("onchange = \"validate('options[8][]','checkbox', true)\""); // prevent user select no option
+	// Index navigation element
+	$navEle = new \XoopsFormRadioYN(_MB_NEWBB_INDEXNAV, 'options[9]', !empty($options[9]));
+
+	// Topic title element
+	$lengthEle = new \XoopsFormText(_MB_NEWBB_TITLE_LENGTH, 'options[10]', 10, 255, (int)$options[10]);
+	$lengthEle->setDescription(_MB_NEWBB_TITLE_LENGTH_DESC);
+
+	// Post text element
+	$postExcerptEle = new \XoopsFormText(_MB_NEWBB_POST_EXCERPT, 'options[11]', 10, 255, (int)$options[11]);
+	$postExcerptEle->setDescription(_MB_NEWBB_POST_EXCERPT_DESC);
+
+	//  forum element
+	$optionsForum = explode(',', $options[12]);
+	require_once  dirname(__DIR__) . '/include/functions.forum.php';
+	/** @var Newbb\ForumHandler $forumHandler */
+	$forumHandler = Newbb\Helper::getInstance()->getHandler('Forum');
+	//get forum Ids by values. parse positive values to forum IDs and negative values to category IDs. value=0 => all valid forums
+	// Get accessible forums
+	$accessForums = $forumHandler->getIdsByValues(array_map('intval', $optionsForum));
+	$isAll        = (0 === count($optionsForum) || empty($optionsForum[0]));
+	$forumSel     = "<select name=\"options[12][]\" multiple=\"multiple\" onchange = \"validate('options[12][]','select', true)\">";// if user dont select any it select "0"
+	$forumSel     .= '<option value="0" ';
+	if ($isAll) {
+		$forumSel     .= ' selected';
+		$accessForums = null; // just select _ALL option
+	}
+	$forumSel .= '>' . _ALL . '</option>';
+	$forumSel .= newbbForumSelectBox($accessForums, 'access', false); //$accessForums, $permission = "access", $delimitorCategory = false
+	$forumSel .= '</select>';
+	$forumEle = new \XoopsFormLabel(_MB_NEWBB_FORUMLIST, $forumSel);
+
+	// add all elements to form
+	$form->addElement($statusEle);
+	$form->addElement($topicPosterEle);
+	$form->addElement($lastPosterEle);
+	$form->addElement($typeEle);
+	$form->addElement($sortEle);
+	$form->addElement($orderEle);
+	$form->addElement($numdispEle);
+	$form->addElement($timeEle);
+	$form->addElement($modeEle, true); // required: user should select at least one otherwise it will select the first one
+	$form->addElement($navEle);
+	$form->addElement($lengthEle);
+	$form->addElement($postExcerptEle);
+	$form->addElement($forumEle);
+
+	return $form->render();
 }

blocks/newbb_block.php

Indentation +856 added lines, -856 removed lines

@@ -26,7 +26,7 @@ 
 require_once $GLOBALS['xoops']->path('modules/newbb/include/functions.ini.php');
 
 if (defined('NEWBB_BLOCK_DEFINED')) {
-    return;
+	return;
 }
 define('NEWBB_BLOCK_DEFINED', true);
 
@@ -36,7 +36,7 @@ 
  */
 function b_newbb_array_filter($var)
 {
-    return $var > 0;
+	return $var > 0;
 }
 
 // options[0] - Citeria valid: time(by default)
@@ -53,166 +53,166 @@ 
  */
 function b_newbb_show($options)
 {
-    global $accessForums;
-    global $xoopsLogger;
-
-    require_once  dirname(__DIR__) . '/include/functions.config.php';
-    require_once  dirname(__DIR__) . '/include/functions.time.php';
-
-    $myts          = \MyTextSanitizer::getInstance();
-    $block         = [];
-    $i             = 0;
-    $order         = '';
-    $extraCriteria = '';
-    if (!empty($options[2])) {
-        //require_once  dirname(__DIR__) . '/include/functions.time.php';
-        $extraCriteria .= ' AND p.post_time>' . (time() - newbbGetSinceTime($options[2]));
-    }
-    switch ($options[0]) {
-        case 'time':
-        default:
-            $order = 't.topic_last_post_id';
-            break;
-    }
-
-    if (!isset($accessForums)) {
-        /** var Newbb\PermissionHandler $permHandler */
-        $permHandler = Newbb\Helper::getInstance()->getHandler('Permission');
-        if (!$accessForums = $permHandler->getForums()) {
-            return $block;
-        }
-    }
-    if (!empty($options[6])) {
-        $myallowedForums = array_filter(array_slice($options, 6), 'b_newbb_array_filter'); // get allowed forums
-        $allowedForums   = array_intersect($myallowedForums, $accessForums);
-    } else {
-        $allowedForums = $accessForums;
-    }
-    if (empty($allowedForums)) {
-        return $block;
-    }
-
-    $forumCriteria   = ' AND t.forum_id IN (' . implode(',', $allowedForums) . ')';
-    $approveCriteria = ' AND t.approved = 1';
-
-    $newbbConfig = newbbLoadConfig();
-    if (!empty($newbbConfig['do_rewrite'])) {
-        require_once $GLOBALS['xoops']->path('modules/newbb/seo_url.php');
-    } else {
-        if (!defined('SEO_MODULE_NAME')) {
-            define('SEO_MODULE_NAME', 'modules/newbb');
-        }
-    }
-
-    $query = 'SELECT'
-             . '    t.topic_id, t.topic_replies, t.forum_id, t.topic_title, t.topic_views, t.type_id,'
-             . '    f.forum_name,t.topic_status,'
-             . '    p.post_id, p.post_time, p.icon, p.uid, p.poster_name'
-             . '    FROM '
-             . $GLOBALS['xoopsDB']->prefix('newbb_topics')
-             . ' AS t '
-             . '    LEFT JOIN '
-             . $GLOBALS['xoopsDB']->prefix('newbb_posts')
-             . ' AS p ON t.topic_last_post_id=p.post_id'
-             . '    LEFT JOIN '
-             . $GLOBALS['xoopsDB']->prefix('newbb_forums')
-             . ' AS f ON f.forum_id=t.forum_id'
-             . '    WHERE 1=1 '
-             . $forumCriteria
-             . $approveCriteria
-             . $extraCriteria
-             . ' ORDER BY '
-             . $order
-             . ' DESC';
-
-    $result = $GLOBALS['xoopsDB']->query($query, $options[1], 0);
-
-    if (!$result) {
-        //xoops_error($GLOBALS['xoopsDB']->error());
-        return false;
-    }
-    $block['disp_mode'] = $options[3]; // 0 - full view; 1 - compact view; 2 - lite view;
-    $rows               = [];
-    $author             = [];
-    $types              = [];
-
-    while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
-        $rows[]              = $row;
-        $author[$row['uid']] = 1;
-        if ($row['type_id'] > 0) {
-            $types[$row['type_id']] = 1;
-        }
-    }
-
-    if (count($rows) < 1) {
-        return $block;
-    }
-
-    require_once  dirname(__DIR__) . '/include/functions.user.php';
-    $author_name = newbbGetUnameFromIds(array_keys($author), $newbbConfig['show_realname'], true);
-
-    if (count($types) > 0) {
-        /** @var Newbb\TypeHandler $typeHandler */
-        $typeHandler = Newbb\Helper::getInstance()->getHandler('Type');
-        $type_list   = $typeHandler->getList(new \Criteria('type_id', '(' . implode(', ', array_keys($types)) . ')', 'IN'));
-    }
-
-    foreach ($rows as $arr) {
-        // irmtfan add lastposticon - load main lang
-        xoops_loadLanguage('main', 'newbb');
-        $topic                  = [];
-        $topic_page_jump        = newbbDisplayImage('lastposticon', _MD_NEWBB_GOTOLASTPOST);
-        $topic['topic_subject'] = empty($type_list[$arr['type_id']]) ? '' : '[' . $type_list[$arr['type_id']] . ']';
-
-        $topic['post_id']      = $arr['post_id'];
-        $topic['topic_status'] = $arr['topic_status'];
-        $topic['forum_id']     = $arr['forum_id'];
-        $topic['forum_name']   = $myts->htmlSpecialChars($arr['forum_name']);
-        $topic['id']           = $arr['topic_id'];
-
-        $title = $myts->htmlSpecialChars($arr['topic_title']);
-        if (!empty($options[5])) {
-            $title = xoops_substr($title, 0, $options[5]);
-        }
-        $topic['title']   = $topic['topic_subject'] . ' ' . $title;
-        $topic['replies'] = $arr['topic_replies'];
-        $topic['views']   = $arr['topic_views'];
-        $topic['time']    = newbbFormatTimestamp($arr['post_time']);
-        if (!empty($author_name[$arr['uid']])) {
-            $topic_poster = $author_name[$arr['uid']];
-        } else {
-            $topic_poster = $myts->htmlSpecialChars($arr['poster_name'] ?: $GLOBALS['xoopsConfig']['anonymous']);
-        }
-        $topic['topic_poster']    = $topic_poster;
-        $topic['topic_page_jump'] = $topic_page_jump;
-        // START irmtfan remove hardcoded html in URLs - add $seo_topic_url
-        $seo_url       = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewtopic.php?post_id=' . $topic['post_id'];
-        $seo_topic_url = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewtopic.php?topic_id=' . $topic['id'];
-        $seo_forum_url = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewforum.php?forum=' . $topic['forum_id'];
-        if (!empty($newbbConfig['do_rewrite'])) {
-            $topic['seo_url']       = seo_urls($seo_url);
-            $topic['seo_topic_url'] = seo_urls($seo_topic_url);
-            $topic['seo_forum_url'] = seo_urls($seo_forum_url);
-        } else {
-            $topic['seo_url']       = $seo_url;
-            $topic['seo_topic_url'] = $seo_topic_url;
-            $topic['seo_forum_url'] = $seo_forum_url;
-        }
-        // END irmtfan remove hardcoded html in URLs - add $seo_topic_url
-        $block['topics'][] = $topic;
-        unset($topic);
-    }
-    // START irmtfan remove hardcoded html in URLs
-    $seo_top_allforums          = XOOPS_URL . '/' . SEO_MODULE_NAME;
-    $block['seo_top_allforums'] = !empty($newbbConfig['do_rewrite']) ? seo_urls($seo_top_allforums) : $seo_top_allforums;
-    $seo_top_allforums          = XOOPS_URL . '/' . SEO_MODULE_NAME . '/list.topic.php';
-    $block['seo_top_alltopics'] = !empty($newbbConfig['do_rewrite']) ? seo_urls($seo_top_allforums) : $seo_top_allforums;
-    $seo_top_allforums          = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewpost.php';
-    $block['seo_top_allposts']  = !empty($newbbConfig['do_rewrite']) ? seo_urls($seo_top_allforums) : $seo_top_allforums;
-    // END irmtfan remove hardcoded html in URLs
-    $block['indexNav'] = (int)$options[4];
-
-    return $block;
+	global $accessForums;
+	global $xoopsLogger;
+
+	require_once  dirname(__DIR__) . '/include/functions.config.php';
+	require_once  dirname(__DIR__) . '/include/functions.time.php';
+
+	$myts          = \MyTextSanitizer::getInstance();
+	$block         = [];
+	$i             = 0;
+	$order         = '';
+	$extraCriteria = '';
+	if (!empty($options[2])) {
+		//require_once  dirname(__DIR__) . '/include/functions.time.php';
+		$extraCriteria .= ' AND p.post_time>' . (time() - newbbGetSinceTime($options[2]));
+	}
+	switch ($options[0]) {
+		case 'time':
+		default:
+			$order = 't.topic_last_post_id';
+			break;
+	}
+
+	if (!isset($accessForums)) {
+		/** var Newbb\PermissionHandler $permHandler */
+		$permHandler = Newbb\Helper::getInstance()->getHandler('Permission');
+		if (!$accessForums = $permHandler->getForums()) {
+			return $block;
+		}
+	}
+	if (!empty($options[6])) {
+		$myallowedForums = array_filter(array_slice($options, 6), 'b_newbb_array_filter'); // get allowed forums
+		$allowedForums   = array_intersect($myallowedForums, $accessForums);
+	} else {
+		$allowedForums = $accessForums;
+	}
+	if (empty($allowedForums)) {
+		return $block;
+	}
+
+	$forumCriteria   = ' AND t.forum_id IN (' . implode(',', $allowedForums) . ')';
+	$approveCriteria = ' AND t.approved = 1';
+
+	$newbbConfig = newbbLoadConfig();
+	if (!empty($newbbConfig['do_rewrite'])) {
+		require_once $GLOBALS['xoops']->path('modules/newbb/seo_url.php');
+	} else {
+		if (!defined('SEO_MODULE_NAME')) {
+			define('SEO_MODULE_NAME', 'modules/newbb');
+		}
+	}
+
+	$query = 'SELECT'
+			 . '    t.topic_id, t.topic_replies, t.forum_id, t.topic_title, t.topic_views, t.type_id,'
+			 . '    f.forum_name,t.topic_status,'
+			 . '    p.post_id, p.post_time, p.icon, p.uid, p.poster_name'
+			 . '    FROM '
+			 . $GLOBALS['xoopsDB']->prefix('newbb_topics')
+			 . ' AS t '
+			 . '    LEFT JOIN '
+			 . $GLOBALS['xoopsDB']->prefix('newbb_posts')
+			 . ' AS p ON t.topic_last_post_id=p.post_id'
+			 . '    LEFT JOIN '
+			 . $GLOBALS['xoopsDB']->prefix('newbb_forums')
+			 . ' AS f ON f.forum_id=t.forum_id'
+			 . '    WHERE 1=1 '
+			 . $forumCriteria
+			 . $approveCriteria
+			 . $extraCriteria
+			 . ' ORDER BY '
+			 . $order
+			 . ' DESC';
+
+	$result = $GLOBALS['xoopsDB']->query($query, $options[1], 0);
+
+	if (!$result) {
+		//xoops_error($GLOBALS['xoopsDB']->error());
+		return false;
+	}
+	$block['disp_mode'] = $options[3]; // 0 - full view; 1 - compact view; 2 - lite view;
+	$rows               = [];
+	$author             = [];
+	$types              = [];
+
+	while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
+		$rows[]              = $row;
+		$author[$row['uid']] = 1;
+		if ($row['type_id'] > 0) {
+			$types[$row['type_id']] = 1;
+		}
+	}
+
+	if (count($rows) < 1) {
+		return $block;
+	}
+
+	require_once  dirname(__DIR__) . '/include/functions.user.php';
+	$author_name = newbbGetUnameFromIds(array_keys($author), $newbbConfig['show_realname'], true);
+
+	if (count($types) > 0) {
+		/** @var Newbb\TypeHandler $typeHandler */
+		$typeHandler = Newbb\Helper::getInstance()->getHandler('Type');
+		$type_list   = $typeHandler->getList(new \Criteria('type_id', '(' . implode(', ', array_keys($types)) . ')', 'IN'));
+	}
+
+	foreach ($rows as $arr) {
+		// irmtfan add lastposticon - load main lang
+		xoops_loadLanguage('main', 'newbb');
+		$topic                  = [];
+		$topic_page_jump        = newbbDisplayImage('lastposticon', _MD_NEWBB_GOTOLASTPOST);
+		$topic['topic_subject'] = empty($type_list[$arr['type_id']]) ? '' : '[' . $type_list[$arr['type_id']] . ']';
+
+		$topic['post_id']      = $arr['post_id'];
+		$topic['topic_status'] = $arr['topic_status'];
+		$topic['forum_id']     = $arr['forum_id'];
+		$topic['forum_name']   = $myts->htmlSpecialChars($arr['forum_name']);
+		$topic['id']           = $arr['topic_id'];
+
+		$title = $myts->htmlSpecialChars($arr['topic_title']);
+		if (!empty($options[5])) {
+			$title = xoops_substr($title, 0, $options[5]);
+		}
+		$topic['title']   = $topic['topic_subject'] . ' ' . $title;
+		$topic['replies'] = $arr['topic_replies'];
+		$topic['views']   = $arr['topic_views'];
+		$topic['time']    = newbbFormatTimestamp($arr['post_time']);
+		if (!empty($author_name[$arr['uid']])) {
+			$topic_poster = $author_name[$arr['uid']];
+		} else {
+			$topic_poster = $myts->htmlSpecialChars($arr['poster_name'] ?: $GLOBALS['xoopsConfig']['anonymous']);
+		}
+		$topic['topic_poster']    = $topic_poster;
+		$topic['topic_page_jump'] = $topic_page_jump;
+		// START irmtfan remove hardcoded html in URLs - add $seo_topic_url
+		$seo_url       = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewtopic.php?post_id=' . $topic['post_id'];
+		$seo_topic_url = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewtopic.php?topic_id=' . $topic['id'];
+		$seo_forum_url = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewforum.php?forum=' . $topic['forum_id'];
+		if (!empty($newbbConfig['do_rewrite'])) {
+			$topic['seo_url']       = seo_urls($seo_url);
+			$topic['seo_topic_url'] = seo_urls($seo_topic_url);
+			$topic['seo_forum_url'] = seo_urls($seo_forum_url);
+		} else {
+			$topic['seo_url']       = $seo_url;
+			$topic['seo_topic_url'] = $seo_topic_url;
+			$topic['seo_forum_url'] = $seo_forum_url;
+		}
+		// END irmtfan remove hardcoded html in URLs - add $seo_topic_url
+		$block['topics'][] = $topic;
+		unset($topic);
+	}
+	// START irmtfan remove hardcoded html in URLs
+	$seo_top_allforums          = XOOPS_URL . '/' . SEO_MODULE_NAME;
+	$block['seo_top_allforums'] = !empty($newbbConfig['do_rewrite']) ? seo_urls($seo_top_allforums) : $seo_top_allforums;
+	$seo_top_allforums          = XOOPS_URL . '/' . SEO_MODULE_NAME . '/list.topic.php';
+	$block['seo_top_alltopics'] = !empty($newbbConfig['do_rewrite']) ? seo_urls($seo_top_allforums) : $seo_top_allforums;
+	$seo_top_allforums          = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewpost.php';
+	$block['seo_top_allposts']  = !empty($newbbConfig['do_rewrite']) ? seo_urls($seo_top_allforums) : $seo_top_allforums;
+	// END irmtfan remove hardcoded html in URLs
+	$block['indexNav'] = (int)$options[4];
+
+	return $block;
 }
 
 // options[0] - Citeria valid: time(by default), views, replies, digest, sticky
@@ -229,168 +229,168 @@ 
  */
 function b_newbb_topic_show($options)
 {
-    global $accessForums;
-    require_once  dirname(__DIR__) . '/include/functions.time.php';
-    $myts          = \MyTextSanitizer::getInstance();
-    $block         = [];
-    $i             = 0;
-    $order         = '';
-    $extraCriteria = '';
-    $time_criteria = null;
-    if (!empty($options[2])) {
-        $time_criteria = time() - newbbGetSinceTime($options[2]);
-        $extraCriteria = ' AND t.topic_time>' . $time_criteria;
-    }
-    switch ($options[0]) {
-        case 'views':
-            $order = 't.topic_views';
-            break;
-        case 'replies':
-            $order = 't.topic_replies';
-            break;
-        case 'digest':
-            $order         = 't.digest_time';
-            $extraCriteria = ' AND t.topic_digest=1';
-            if (null !== $time_criteria) {
-                $extraCriteria .= ' AND t.digest_time>' . $time_criteria;
-            }
-            break;
-        case 'sticky':
-            $order         = 't.topic_id';
-            $extraCriteria .= ' AND t.topic_sticky=1';
-            break;
-        case 'time':
-        default:
-            $order = 't.topic_id';
-            break;
-    }
-
-    $newbbConfig = newbbLoadConfig();
-    if (!empty($newbbConfig['do_rewrite'])) {
-        require_once $GLOBALS['xoops']->path('modules/newbb/seo_url.php');
-    } else {
-        if (!defined('SEO_MODULE_NAME')) {
-            define('SEO_MODULE_NAME', 'modules/newbb');
-        }
-    }
-
-    if (!isset($accessForums)) {
-        /** var Newbb\PermissionHandler $permHandler */
-        $permHandler = Newbb\Helper::getInstance()->getHandler('Permission');
-        if (!$accessForums = $permHandler->getForums()) {
-            return $block;
-        }
-    }
-
-    if (!empty($options[6])) {
-        $myallowedForums = array_filter(array_slice($options, 6), 'b_newbb_array_filter'); // get allowed forums
-        $allowedForums   = array_intersect($myallowedForums, $accessForums);
-    } else {
-        $allowedForums = $accessForums;
-    }
-    if (empty($allowedForums)) {
-        return false;
-    }
-
-    $forumCriteria   = ' AND t.forum_id IN (' . implode(',', $allowedForums) . ')';
-    $approveCriteria = ' AND t.approved = 1';
-
-    $query = 'SELECT'
-             . '    t.topic_id, t.topic_replies, t.forum_id, t.topic_title, t.topic_views, t.type_id, t.topic_time, t.topic_poster, t.poster_name,'
-             . '    f.forum_name'
-             . '    FROM '
-             . $GLOBALS['xoopsDB']->prefix('newbb_topics')
-             . ' AS t '
-             . '    LEFT JOIN '
-             . $GLOBALS['xoopsDB']->prefix('newbb_forums')
-             . ' AS f ON f.forum_id=t.forum_id'
-             . '    WHERE 1=1 '
-             . $forumCriteria
-             . $approveCriteria
-             . $extraCriteria
-             . ' ORDER BY '
-             . $order
-             . ' DESC';
-
-    $result = $GLOBALS['xoopsDB']->query($query, $options[1], 0);
-
-    if (!$result) {
-        //xoops_error($GLOBALS['xoopsDB']->error());
-        return $block;
-    }
-    $block['disp_mode'] = $options[3]; // 0 - full view; 1 - compact view; 2 - lite view;
-    $rows               = [];
-    $author             = [];
-    $types              = [];
-    while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
-        $rows[]                       = $row;
-        $author[$row['topic_poster']] = 1;
-        if ($row['type_id'] > 0) {
-            $types[$row['type_id']] = 1;
-        }
-    }
-    if (count($rows) < 1) {
-        return $block;
-    }
-    require_once  dirname(__DIR__) . '/include/functions.user.php';
-    $author_name = newbbGetUnameFromIds(array_keys($author), $newbbConfig['show_realname'], true);
-    if (count($types) > 0) {
-        /** @var Newbb\TypeHandler $typeHandler */
-        $typeHandler = Newbb\Helper::getInstance()->getHandler('Type');
-        $type_list   = $typeHandler->getList(new \Criteria('type_id', '(' . implode(', ', array_keys($types)) . ')', 'IN'));
-    }
-
-    foreach ($rows as $arr) {
-        // irmtfan remove $topic_page_jump because there is no last post
-        //$topic_page_jump = '';
-        $topic                  = [];
-        $topic['topic_subject'] = empty($type_list[$arr['type_id']]) ? '' : '[' . $type_list[$arr['type_id']] . '] ';
-        $topic['forum_id']      = $arr['forum_id'];
-        $topic['forum_name']    = $myts->htmlSpecialChars($arr['forum_name']);
-        $topic['id']            = $arr['topic_id'];
-
-        $title = $myts->htmlSpecialChars($arr['topic_title']);
-        if (!empty($options[5])) {
-            $title = xoops_substr($title, 0, $options[5]);
-        }
-        $topic['title']   = $topic['topic_subject'] . $title;
-        $topic['replies'] = $arr['topic_replies'];
-        $topic['views']   = $arr['topic_views'];
-        $topic['time']    = newbbFormatTimestamp($arr['topic_time']);
-        if (!empty($author_name[$arr['topic_poster']])) {
-            $topic_poster = $author_name[$arr['topic_poster']];
-        } else {
-            $topic_poster = $myts->htmlSpecialChars($arr['poster_name'] ?: $GLOBALS['xoopsConfig']['anonymous']);
-        }
-        $topic['topic_poster'] = $topic_poster;
-        // irmtfan remove $topic_page_jump because there is no last post
-        //$topic['topic_page_jump'] = $topic_page_jump;
-        // START irmtfan remove hardcoded html in URLs - add $seo_topic_url
-        $seo_topic_url = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewtopic.php?topic_id=' . $topic['id'];
-        $seo_forum_url = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewforum.php?forum=' . $topic['forum_id'];
-
-        if (!empty($newbbConfig['do_rewrite'])) {
-            $topic['seo_topic_url'] = seo_urls($seo_topic_url);
-            $topic['seo_forum_url'] = seo_urls($seo_forum_url);
-        } else {
-            $topic['seo_topic_url'] = $seo_topic_url;
-            $topic['seo_forum_url'] = $seo_forum_url;
-        }
-        // END irmtfan remove hardcoded html in URLs - add $seo_topic_url
-        $block['topics'][] = $topic;
-        unset($topic);
-    }
-    // START irmtfan remove hardcoded html in URLs
-    $seo_top_allforums          = XOOPS_URL . '/' . SEO_MODULE_NAME;
-    $block['seo_top_allforums'] = !empty($newbbConfig['do_rewrite']) ? seo_urls($seo_top_allforums) : $seo_top_allforums;
-    $seo_top_allforums          = XOOPS_URL . '/' . SEO_MODULE_NAME . '/list.topic.php';
-    $block['seo_top_alltopics'] = !empty($newbbConfig['do_rewrite']) ? seo_urls($seo_top_allforums) : $seo_top_allforums;
-    $seo_top_allforums          = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewpost.php';
-    $block['seo_top_allposts']  = !empty($newbbConfig['do_rewrite']) ? seo_urls($seo_top_allforums) : $seo_top_allforums;
-    // END irmtfan remove hardcoded html in URLs
-    $block['indexNav'] = (int)$options[4];
-
-    return $block;
+	global $accessForums;
+	require_once  dirname(__DIR__) . '/include/functions.time.php';
+	$myts          = \MyTextSanitizer::getInstance();
+	$block         = [];
+	$i             = 0;
+	$order         = '';
+	$extraCriteria = '';
+	$time_criteria = null;
+	if (!empty($options[2])) {
+		$time_criteria = time() - newbbGetSinceTime($options[2]);
+		$extraCriteria = ' AND t.topic_time>' . $time_criteria;
+	}
+	switch ($options[0]) {
+		case 'views':
+			$order = 't.topic_views';
+			break;
+		case 'replies':
+			$order = 't.topic_replies';
+			break;
+		case 'digest':
+			$order         = 't.digest_time';
+			$extraCriteria = ' AND t.topic_digest=1';
+			if (null !== $time_criteria) {
+				$extraCriteria .= ' AND t.digest_time>' . $time_criteria;
+			}
+			break;
+		case 'sticky':
+			$order         = 't.topic_id';
+			$extraCriteria .= ' AND t.topic_sticky=1';
+			break;
+		case 'time':
+		default:
+			$order = 't.topic_id';
+			break;
+	}
+
+	$newbbConfig = newbbLoadConfig();
+	if (!empty($newbbConfig['do_rewrite'])) {
+		require_once $GLOBALS['xoops']->path('modules/newbb/seo_url.php');
+	} else {
+		if (!defined('SEO_MODULE_NAME')) {
+			define('SEO_MODULE_NAME', 'modules/newbb');
+		}
+	}
+
+	if (!isset($accessForums)) {
+		/** var Newbb\PermissionHandler $permHandler */
+		$permHandler = Newbb\Helper::getInstance()->getHandler('Permission');
+		if (!$accessForums = $permHandler->getForums()) {
+			return $block;
+		}
+	}
+
+	if (!empty($options[6])) {
+		$myallowedForums = array_filter(array_slice($options, 6), 'b_newbb_array_filter'); // get allowed forums
+		$allowedForums   = array_intersect($myallowedForums, $accessForums);
+	} else {
+		$allowedForums = $accessForums;
+	}
+	if (empty($allowedForums)) {
+		return false;
+	}
+
+	$forumCriteria   = ' AND t.forum_id IN (' . implode(',', $allowedForums) . ')';
+	$approveCriteria = ' AND t.approved = 1';
+
+	$query = 'SELECT'
+			 . '    t.topic_id, t.topic_replies, t.forum_id, t.topic_title, t.topic_views, t.type_id, t.topic_time, t.topic_poster, t.poster_name,'
+			 . '    f.forum_name'
+			 . '    FROM '
+			 . $GLOBALS['xoopsDB']->prefix('newbb_topics')
+			 . ' AS t '
+			 . '    LEFT JOIN '
+			 . $GLOBALS['xoopsDB']->prefix('newbb_forums')
+			 . ' AS f ON f.forum_id=t.forum_id'
+			 . '    WHERE 1=1 '
+			 . $forumCriteria
+			 . $approveCriteria
+			 . $extraCriteria
+			 . ' ORDER BY '
+			 . $order
+			 . ' DESC';
+
+	$result = $GLOBALS['xoopsDB']->query($query, $options[1], 0);
+
+	if (!$result) {
+		//xoops_error($GLOBALS['xoopsDB']->error());
+		return $block;
+	}
+	$block['disp_mode'] = $options[3]; // 0 - full view; 1 - compact view; 2 - lite view;
+	$rows               = [];
+	$author             = [];
+	$types              = [];
+	while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
+		$rows[]                       = $row;
+		$author[$row['topic_poster']] = 1;
+		if ($row['type_id'] > 0) {
+			$types[$row['type_id']] = 1;
+		}
+	}
+	if (count($rows) < 1) {
+		return $block;
+	}
+	require_once  dirname(__DIR__) . '/include/functions.user.php';
+	$author_name = newbbGetUnameFromIds(array_keys($author), $newbbConfig['show_realname'], true);
+	if (count($types) > 0) {
+		/** @var Newbb\TypeHandler $typeHandler */
+		$typeHandler = Newbb\Helper::getInstance()->getHandler('Type');
+		$type_list   = $typeHandler->getList(new \Criteria('type_id', '(' . implode(', ', array_keys($types)) . ')', 'IN'));
+	}
+
+	foreach ($rows as $arr) {
+		// irmtfan remove $topic_page_jump because there is no last post
+		//$topic_page_jump = '';
+		$topic                  = [];
+		$topic['topic_subject'] = empty($type_list[$arr['type_id']]) ? '' : '[' . $type_list[$arr['type_id']] . '] ';
+		$topic['forum_id']      = $arr['forum_id'];
+		$topic['forum_name']    = $myts->htmlSpecialChars($arr['forum_name']);
+		$topic['id']            = $arr['topic_id'];
+
+		$title = $myts->htmlSpecialChars($arr['topic_title']);
+		if (!empty($options[5])) {
+			$title = xoops_substr($title, 0, $options[5]);
+		}
+		$topic['title']   = $topic['topic_subject'] . $title;
+		$topic['replies'] = $arr['topic_replies'];
+		$topic['views']   = $arr['topic_views'];
+		$topic['time']    = newbbFormatTimestamp($arr['topic_time']);
+		if (!empty($author_name[$arr['topic_poster']])) {
+			$topic_poster = $author_name[$arr['topic_poster']];
+		} else {
+			$topic_poster = $myts->htmlSpecialChars($arr['poster_name'] ?: $GLOBALS['xoopsConfig']['anonymous']);
+		}
+		$topic['topic_poster'] = $topic_poster;
+		// irmtfan remove $topic_page_jump because there is no last post
+		//$topic['topic_page_jump'] = $topic_page_jump;
+		// START irmtfan remove hardcoded html in URLs - add $seo_topic_url
+		$seo_topic_url = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewtopic.php?topic_id=' . $topic['id'];
+		$seo_forum_url = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewforum.php?forum=' . $topic['forum_id'];
+
+		if (!empty($newbbConfig['do_rewrite'])) {
+			$topic['seo_topic_url'] = seo_urls($seo_topic_url);
+			$topic['seo_forum_url'] = seo_urls($seo_forum_url);
+		} else {
+			$topic['seo_topic_url'] = $seo_topic_url;
+			$topic['seo_forum_url'] = $seo_forum_url;
+		}
+		// END irmtfan remove hardcoded html in URLs - add $seo_topic_url
+		$block['topics'][] = $topic;
+		unset($topic);
+	}
+	// START irmtfan remove hardcoded html in URLs
+	$seo_top_allforums          = XOOPS_URL . '/' . SEO_MODULE_NAME;
+	$block['seo_top_allforums'] = !empty($newbbConfig['do_rewrite']) ? seo_urls($seo_top_allforums) : $seo_top_allforums;
+	$seo_top_allforums          = XOOPS_URL . '/' . SEO_MODULE_NAME . '/list.topic.php';
+	$block['seo_top_alltopics'] = !empty($newbbConfig['do_rewrite']) ? seo_urls($seo_top_allforums) : $seo_top_allforums;
+	$seo_top_allforums          = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewpost.php';
+	$block['seo_top_allposts']  = !empty($newbbConfig['do_rewrite']) ? seo_urls($seo_top_allforums) : $seo_top_allforums;
+	// END irmtfan remove hardcoded html in URLs
+	$block['indexNav'] = (int)$options[4];
+
+	return $block;
 }
 
 // options[0] - Citeria valid: title(by default), text
@@ -407,155 +407,155 @@ 
  */
 function b_newbb_post_show($options)
 {
-    global $accessForums;
-    global $newbbConfig;
-
-    require_once  dirname(__DIR__) . '/include/functions.time.php';
-    $myts          = \MyTextSanitizer::getInstance();
-    $block         = [];
-    $i             = 0;
-    $order         = '';
-    $extraCriteria = '';
-    $time_criteria = null;
-    if (!empty($options[2])) {
-        $time_criteria = time() - newbbGetSinceTime($options[2]);
-        $extraCriteria = ' AND p.post_time>' . $time_criteria;
-    }
-
-    switch ($options[0]) {
-        case 'text':
-            if (!empty($newbbConfig['enable_karma'])) {
-                $extraCriteria .= ' AND p.post_karma = 0';
-            }
-            if (!empty($newbbConfig['allow_require_reply'])) {
-                $extraCriteria .= ' AND p.require_reply = 0';
-            }
-        // no break
-        default:
-            $order = 'p.post_id';
-            break;
-    }
-
-    if (!isset($accessForums)) {
-        /** var Newbb\PermissionHandler $permHandler */
-        $permHandler = Newbb\Helper::getInstance()->getHandler('Permission');
-        if (!$accessForums = $permHandler->getForums()) {
-            return $block;
-        }
-    }
-
-    $newbbConfig = newbbLoadConfig();
-    if (!empty($newbbConfig['do_rewrite'])) {
-        require_once $GLOBALS['xoops']->path('modules/newbb/seo_url.php');
-    } else {
-        if (!defined('SEO_MODULE_NAME')) {
-            define('SEO_MODULE_NAME', 'modules/newbb');
-        }
-    }
-
-    if (!empty($options[6])) {
-        $myallowedForums = array_filter(array_slice($options, 6), 'b_newbb_array_filter'); // get allowed forums
-        $allowedForums   = array_intersect($myallowedForums, $accessForums);
-    } else {
-        $allowedForums = $accessForums;
-    }
-    if (empty($allowedForums)) {
-        return $block;
-    }
-
-    $forumCriteria   = ' AND p.forum_id IN (' . implode(',', $allowedForums) . ')';
-    $approveCriteria = ' AND p.approved = 1';
-
-    $query = 'SELECT';
-    $query .= '    p.post_id, p.subject, p.post_time, p.icon, p.uid, p.poster_name,';
-    if ('text' === $options[0]) {
-        $query .= '    pt.dohtml, pt.dosmiley, pt.doxcode, pt.dobr, pt.post_text,';
-    }
-    $query .= '    f.forum_id, f.forum_name' . '    FROM ' . $GLOBALS['xoopsDB']->prefix('newbb_posts') . ' AS p ' . '    LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('newbb_forums') . ' AS f ON f.forum_id=p.forum_id';
-    if ('text' === $options[0]) {
-        $query .= '    LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('newbb_posts_text') . ' AS pt ON pt.post_id=p.post_id';
-    }
-    $query .= '    WHERE 1=1 ' . $forumCriteria . $approveCriteria . $extraCriteria . ' ORDER BY ' . $order . ' DESC';
-
-    $result = $GLOBALS['xoopsDB']->query($query, $options[1], 0);
-    if (!$result) {
-        //xoops_error($GLOBALS['xoopsDB']->error());
-        return $block;
-    }
-    $block['disp_mode'] = ('text' === $options[0]) ? 3 : $options[3]; // 0 - full view; 1 - compact view; 2 - lite view;
-    $rows               = [];
-    $author             = [];
-    while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
-        $rows[]              = $row;
-        $author[$row['uid']] = 1;
-    }
-    if (count($rows) < 1) {
-        return $block;
-    }
-    require_once  dirname(__DIR__) . '/include/functions.user.php';
-    $author_name = newbbGetUnameFromIds(array_keys($author), $newbbConfig['show_realname'], true);
-
-    foreach ($rows as $arr) {
-        //if ($arr['icon'] && is_file($GLOBALS['xoops']->path('images/subject/' . $arr['icon']))) {
-        if (!empty($arr['icon'])) {
-            $last_post_icon = '<img src="' . XOOPS_URL . '/images/subject/' . htmlspecialchars($arr['icon'], ENT_QUOTES | ENT_HTML5) . '" alt="" />';
-        } else {
-            $last_post_icon = '<img src="' . XOOPS_URL . '/images/subject/icon1.gif" alt="" />';
-        }
-        //$topic['jump_post'] = "<a href='" . XOOPS_URL . "/modules/newbb/viewtopic.php?post_id=" . $arr['post_id'] ."#forumpost" . $arr['post_id'] . "'>" . $last_post_icon . '</a>';
-        $topic               = [];
-        $topic['forum_id']   = $arr['forum_id'];
-        $topic['forum_name'] = $myts->htmlSpecialChars($arr['forum_name']);
-        //$topic['id'] = $arr['topic_id'];
-
-        $title = $myts->htmlSpecialChars($arr['subject']);
-        if ('text' !== $options[0] && !empty($options[5])) {
-            $title = xoops_substr($title, 0, $options[5]);
-        }
-        $topic['title']   = $title;
-        $topic['post_id'] = $arr['post_id'];
-        $topic['time']    = newbbFormatTimestamp($arr['post_time']);
-        if (!empty($author_name[$arr['uid']])) {
-            $topic_poster = $author_name[$arr['uid']];
-        } else {
-            $topic_poster = $myts->htmlSpecialChars($arr['poster_name'] ?: $GLOBALS['xoopsConfig']['anonymous']);
-        }
-        $topic['topic_poster'] = $topic_poster;
-
-        if ('text' === $options[0]) {
-            $post_text = $myts->displayTarea($arr['post_text'], $arr['dohtml'], $arr['dosmiley'], $arr['doxcode'], 1, $arr['dobr']);
-            if (!empty($options[5])) {
-                $post_text = xoops_substr(strip_tags($post_text), 0, $options[5]);
-            }
-            $topic['post_text'] = $post_text;
-        }
-        // START irmtfan remove hardcoded html in URLs
-        $seo_url       = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewtopic.php?post_id=' . $topic['post_id'];
-        $seo_forum_url = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewforum.php?forum=' . $topic['forum_id'];
-        // END irmtfan remove hardcoded html in URLs
-        if (!empty($newbbConfig['do_rewrite'])) {
-            $topic['seo_url']       = seo_urls($seo_url);
-            $topic['seo_forum_url'] = seo_urls($seo_forum_url);
-        } else {
-            $topic['seo_url']       = $seo_url;
-            $topic['seo_forum_url'] = $seo_forum_url;
-        }
-
-        $block['topics'][] = $topic;
-        unset($topic);
-    }
-    // START irmtfan remove hardcoded html in URLs
-    $seo_top_allforums          = XOOPS_URL . '/' . SEO_MODULE_NAME;
-    $block['seo_top_allforums'] = !empty($newbbConfig['do_rewrite']) ? seo_urls($seo_top_allforums) : $seo_top_allforums;
-    $seo_top_allforums          = XOOPS_URL . '/' . SEO_MODULE_NAME . '/list.topic.php';
-    $block['seo_top_alltopics'] = !empty($newbbConfig['do_rewrite']) ? seo_urls($seo_top_allforums) : $seo_top_allforums;
-    $seo_top_allforums          = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewpost.php';
-    $block['seo_top_allposts']  = !empty($newbbConfig['do_rewrite']) ? seo_urls($seo_top_allforums) : $seo_top_allforums;
-    // END irmtfan remove hardcoded html in URLs
-
-    $block['indexNav'] = (int)$options[4];
-
-    return $block;
+	global $accessForums;
+	global $newbbConfig;
+
+	require_once  dirname(__DIR__) . '/include/functions.time.php';
+	$myts          = \MyTextSanitizer::getInstance();
+	$block         = [];
+	$i             = 0;
+	$order         = '';
+	$extraCriteria = '';
+	$time_criteria = null;
+	if (!empty($options[2])) {
+		$time_criteria = time() - newbbGetSinceTime($options[2]);
+		$extraCriteria = ' AND p.post_time>' . $time_criteria;
+	}
+
+	switch ($options[0]) {
+		case 'text':
+			if (!empty($newbbConfig['enable_karma'])) {
+				$extraCriteria .= ' AND p.post_karma = 0';
+			}
+			if (!empty($newbbConfig['allow_require_reply'])) {
+				$extraCriteria .= ' AND p.require_reply = 0';
+			}
+		// no break
+		default:
+			$order = 'p.post_id';
+			break;
+	}
+
+	if (!isset($accessForums)) {
+		/** var Newbb\PermissionHandler $permHandler */
+		$permHandler = Newbb\Helper::getInstance()->getHandler('Permission');
+		if (!$accessForums = $permHandler->getForums()) {
+			return $block;
+		}
+	}
+
+	$newbbConfig = newbbLoadConfig();
+	if (!empty($newbbConfig['do_rewrite'])) {
+		require_once $GLOBALS['xoops']->path('modules/newbb/seo_url.php');
+	} else {
+		if (!defined('SEO_MODULE_NAME')) {
+			define('SEO_MODULE_NAME', 'modules/newbb');
+		}
+	}
+
+	if (!empty($options[6])) {
+		$myallowedForums = array_filter(array_slice($options, 6), 'b_newbb_array_filter'); // get allowed forums
+		$allowedForums   = array_intersect($myallowedForums, $accessForums);
+	} else {
+		$allowedForums = $accessForums;
+	}
+	if (empty($allowedForums)) {
+		return $block;
+	}
+
+	$forumCriteria   = ' AND p.forum_id IN (' . implode(',', $allowedForums) . ')';
+	$approveCriteria = ' AND p.approved = 1';
+
+	$query = 'SELECT';
+	$query .= '    p.post_id, p.subject, p.post_time, p.icon, p.uid, p.poster_name,';
+	if ('text' === $options[0]) {
+		$query .= '    pt.dohtml, pt.dosmiley, pt.doxcode, pt.dobr, pt.post_text,';
+	}
+	$query .= '    f.forum_id, f.forum_name' . '    FROM ' . $GLOBALS['xoopsDB']->prefix('newbb_posts') . ' AS p ' . '    LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('newbb_forums') . ' AS f ON f.forum_id=p.forum_id';
+	if ('text' === $options[0]) {
+		$query .= '    LEFT JOIN ' . $GLOBALS['xoopsDB']->prefix('newbb_posts_text') . ' AS pt ON pt.post_id=p.post_id';
+	}
+	$query .= '    WHERE 1=1 ' . $forumCriteria . $approveCriteria . $extraCriteria . ' ORDER BY ' . $order . ' DESC';
+
+	$result = $GLOBALS['xoopsDB']->query($query, $options[1], 0);
+	if (!$result) {
+		//xoops_error($GLOBALS['xoopsDB']->error());
+		return $block;
+	}
+	$block['disp_mode'] = ('text' === $options[0]) ? 3 : $options[3]; // 0 - full view; 1 - compact view; 2 - lite view;
+	$rows               = [];
+	$author             = [];
+	while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
+		$rows[]              = $row;
+		$author[$row['uid']] = 1;
+	}
+	if (count($rows) < 1) {
+		return $block;
+	}
+	require_once  dirname(__DIR__) . '/include/functions.user.php';
+	$author_name = newbbGetUnameFromIds(array_keys($author), $newbbConfig['show_realname'], true);
+
+	foreach ($rows as $arr) {
+		//if ($arr['icon'] && is_file($GLOBALS['xoops']->path('images/subject/' . $arr['icon']))) {
+		if (!empty($arr['icon'])) {
+			$last_post_icon = '<img src="' . XOOPS_URL . '/images/subject/' . htmlspecialchars($arr['icon'], ENT_QUOTES | ENT_HTML5) . '" alt="" />';
+		} else {
+			$last_post_icon = '<img src="' . XOOPS_URL . '/images/subject/icon1.gif" alt="" />';
+		}
+		//$topic['jump_post'] = "<a href='" . XOOPS_URL . "/modules/newbb/viewtopic.php?post_id=" . $arr['post_id'] ."#forumpost" . $arr['post_id'] . "'>" . $last_post_icon . '</a>';
+		$topic               = [];
+		$topic['forum_id']   = $arr['forum_id'];
+		$topic['forum_name'] = $myts->htmlSpecialChars($arr['forum_name']);
+		//$topic['id'] = $arr['topic_id'];
+
+		$title = $myts->htmlSpecialChars($arr['subject']);
+		if ('text' !== $options[0] && !empty($options[5])) {
+			$title = xoops_substr($title, 0, $options[5]);
+		}
+		$topic['title']   = $title;
+		$topic['post_id'] = $arr['post_id'];
+		$topic['time']    = newbbFormatTimestamp($arr['post_time']);
+		if (!empty($author_name[$arr['uid']])) {
+			$topic_poster = $author_name[$arr['uid']];
+		} else {
+			$topic_poster = $myts->htmlSpecialChars($arr['poster_name'] ?: $GLOBALS['xoopsConfig']['anonymous']);
+		}
+		$topic['topic_poster'] = $topic_poster;
+
+		if ('text' === $options[0]) {
+			$post_text = $myts->displayTarea($arr['post_text'], $arr['dohtml'], $arr['dosmiley'], $arr['doxcode'], 1, $arr['dobr']);
+			if (!empty($options[5])) {
+				$post_text = xoops_substr(strip_tags($post_text), 0, $options[5]);
+			}
+			$topic['post_text'] = $post_text;
+		}
+		// START irmtfan remove hardcoded html in URLs
+		$seo_url       = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewtopic.php?post_id=' . $topic['post_id'];
+		$seo_forum_url = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewforum.php?forum=' . $topic['forum_id'];
+		// END irmtfan remove hardcoded html in URLs
+		if (!empty($newbbConfig['do_rewrite'])) {
+			$topic['seo_url']       = seo_urls($seo_url);
+			$topic['seo_forum_url'] = seo_urls($seo_forum_url);
+		} else {
+			$topic['seo_url']       = $seo_url;
+			$topic['seo_forum_url'] = $seo_forum_url;
+		}
+
+		$block['topics'][] = $topic;
+		unset($topic);
+	}
+	// START irmtfan remove hardcoded html in URLs
+	$seo_top_allforums          = XOOPS_URL . '/' . SEO_MODULE_NAME;
+	$block['seo_top_allforums'] = !empty($newbbConfig['do_rewrite']) ? seo_urls($seo_top_allforums) : $seo_top_allforums;
+	$seo_top_allforums          = XOOPS_URL . '/' . SEO_MODULE_NAME . '/list.topic.php';
+	$block['seo_top_alltopics'] = !empty($newbbConfig['do_rewrite']) ? seo_urls($seo_top_allforums) : $seo_top_allforums;
+	$seo_top_allforums          = XOOPS_URL . '/' . SEO_MODULE_NAME . '/viewpost.php';
+	$block['seo_top_allposts']  = !empty($newbbConfig['do_rewrite']) ? seo_urls($seo_top_allforums) : $seo_top_allforums;
+	// END irmtfan remove hardcoded html in URLs
+
+	$block['indexNav'] = (int)$options[4];
+
+	return $block;
 }
 
 // options[0] - Citeria valid: post(by default), topic, digest, sticky
@@ -572,95 +572,95 @@ 
  */
 function b_newbb_author_show($options)
 {
-    global $accessForums;
-    global $newbbConfig;
-    $myts  = \MyTextSanitizer::getInstance();
-    $block = [];
-    //    $i              = 0;
-    $type          = 'topic';
-    $order         = 'count';
-    $extraCriteria = '';
-    $time_criteria = null;
-    if (!empty($options[2])) {
-        require_once  dirname(__DIR__) . '/include/functions.time.php';
-        $time_criteria = time() - newbbGetSinceTime($options[2]);
-        $extraCriteria = ' AND topic_time > ' . $time_criteria;
-    }
-    switch ($options[0]) {
-        case 'topic':
-            break;
-        case 'digest':
-            $extraCriteria = ' AND topic_digest = 1';
-            if (null !== $time_criteria) {
-                $extraCriteria .= ' AND digest_time > ' . $time_criteria;
-            }
-            break;
-        case 'sticky':
-            $extraCriteria .= ' AND topic_sticky = 1';
-            break;
-        case 'post':
-        default:
-            $type = 'post';
-            if (null !== $time_criteria) {
-                $extraCriteria = ' AND post_time > ' . $time_criteria;
-            }
-            break;
-    }
-
-    if (!isset($accessForums)) {
-        /** var Newbb\PermissionHandler $permHandler */
-        $permHandler = Newbb\Helper::getInstance()->getHandler('Permission');
-        if (!$accessForums = $permHandler->getForums()) {
-            return $block;
-        }
-    }
-
-    if (!empty($options[5])) {
-        $myallowedForums = array_filter(array_slice($options, 5), 'b_newbb_array_filter'); // get allowed forums
-        $allowedForums   = array_intersect($myallowedForums, $accessForums);
-    } else {
-        $allowedForums = $accessForums;
-    }
-    if (empty($allowedForums)) {
-        return false;
-    }
-
-    if ('topic' === $type) {
-        $forumCriteria   = ' AND forum_id IN (' . implode(',', $allowedForums) . ')';
-        $approveCriteria = ' AND approved = 1';
-        $query           = 'SELECT DISTINCT topic_poster AS author, COUNT(*) AS count
+	global $accessForums;
+	global $newbbConfig;
+	$myts  = \MyTextSanitizer::getInstance();
+	$block = [];
+	//    $i              = 0;
+	$type          = 'topic';
+	$order         = 'count';
+	$extraCriteria = '';
+	$time_criteria = null;
+	if (!empty($options[2])) {
+		require_once  dirname(__DIR__) . '/include/functions.time.php';
+		$time_criteria = time() - newbbGetSinceTime($options[2]);
+		$extraCriteria = ' AND topic_time > ' . $time_criteria;
+	}
+	switch ($options[0]) {
+		case 'topic':
+			break;
+		case 'digest':
+			$extraCriteria = ' AND topic_digest = 1';
+			if (null !== $time_criteria) {
+				$extraCriteria .= ' AND digest_time > ' . $time_criteria;
+			}
+			break;
+		case 'sticky':
+			$extraCriteria .= ' AND topic_sticky = 1';
+			break;
+		case 'post':
+		default:
+			$type = 'post';
+			if (null !== $time_criteria) {
+				$extraCriteria = ' AND post_time > ' . $time_criteria;
+			}
+			break;
+	}
+
+	if (!isset($accessForums)) {
+		/** var Newbb\PermissionHandler $permHandler */
+		$permHandler = Newbb\Helper::getInstance()->getHandler('Permission');
+		if (!$accessForums = $permHandler->getForums()) {
+			return $block;
+		}
+	}
+
+	if (!empty($options[5])) {
+		$myallowedForums = array_filter(array_slice($options, 5), 'b_newbb_array_filter'); // get allowed forums
+		$allowedForums   = array_intersect($myallowedForums, $accessForums);
+	} else {
+		$allowedForums = $accessForums;
+	}
+	if (empty($allowedForums)) {
+		return false;
+	}
+
+	if ('topic' === $type) {
+		$forumCriteria   = ' AND forum_id IN (' . implode(',', $allowedForums) . ')';
+		$approveCriteria = ' AND approved = 1';
+		$query           = 'SELECT DISTINCT topic_poster AS author, COUNT(*) AS count
                     FROM ' . $GLOBALS['xoopsDB']->prefix('newbb_topics') . '
                     WHERE topic_poster>0 ' . $forumCriteria . $approveCriteria . $extraCriteria . ' GROUP BY topic_poster ORDER BY ' . $order . ' DESC';
-    } else {
-        $forumCriteria   = ' AND forum_id IN (' . implode(',', $allowedForums) . ')';
-        $approveCriteria = ' AND approved = 1';
-        $query           = 'SELECT DISTINCT uid AS author, COUNT(*) AS count
+	} else {
+		$forumCriteria   = ' AND forum_id IN (' . implode(',', $allowedForums) . ')';
+		$approveCriteria = ' AND approved = 1';
+		$query           = 'SELECT DISTINCT uid AS author, COUNT(*) AS count
                     FROM ' . $GLOBALS['xoopsDB']->prefix('newbb_posts') . '
                     WHERE uid > 0 ' . $forumCriteria . $approveCriteria . $extraCriteria . ' GROUP BY uid ORDER BY ' . $order . ' DESC';
-    }
-
-    $result = $GLOBALS['xoopsDB']->query($query, $options[1], 0);
-    if (!$result) {
-        //xoops_error($GLOBALS['xoopsDB']->error());
-        return $block;
-    }
-    $author = [];
-    while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
-        $author[$row['author']]['count'] = $row['count'];
-    }
-    if (count($author) < 1) {
-        return $block;
-    }
-    require_once  dirname(__DIR__) . '/include/functions.user.php';
-    $author_name = newbbGetUnameFromIds(array_keys($author), $newbbConfig['show_realname']);
-    foreach (array_keys($author) as $uid) {
-        $author[$uid]['name'] = $myts->htmlSpecialChars($author_name[$uid]);
-    }
-    $block['authors']   =& $author;
-    $block['disp_mode'] = $options[3]; // 0 - full view; 1 - lite view;
-    $block['indexNav']  = (int)$options[4];
-
-    return $block;
+	}
+
+	$result = $GLOBALS['xoopsDB']->query($query, $options[1], 0);
+	if (!$result) {
+		//xoops_error($GLOBALS['xoopsDB']->error());
+		return $block;
+	}
+	$author = [];
+	while (false !== ($row = $GLOBALS['xoopsDB']->fetchArray($result))) {
+		$author[$row['author']]['count'] = $row['count'];
+	}
+	if (count($author) < 1) {
+		return $block;
+	}
+	require_once  dirname(__DIR__) . '/include/functions.user.php';
+	$author_name = newbbGetUnameFromIds(array_keys($author), $newbbConfig['show_realname']);
+	foreach (array_keys($author) as $uid) {
+		$author[$uid]['name'] = $myts->htmlSpecialChars($author_name[$uid]);
+	}
+	$block['authors']   =& $author;
+	$block['disp_mode'] = $options[3]; // 0 - full view; 1 - lite view;
+	$block['indexNav']  = (int)$options[4];
+
+	return $block;
 }
 
 /**
@@ -669,58 +669,58 @@ 
  */
 function b_newbb_edit($options)
 {
-    require_once  dirname(__DIR__) . '/include/functions.forum.php';
-
-    $form = _MB_NEWBB_CRITERIA . "<select name='options[0]'>";
-    $form .= "<option value='time'";
-    if ('time' === $options[0]) {
-        $form .= " selected='selected' ";
-    }
-    $form .= '>' . _MB_NEWBB_CRITERIA_TIME . '</option>';
-    $form .= '</select>';
-    $form .= '<br>' . _MB_NEWBB_DISPLAY . "<input type='text' name='options[1]' value='" . $options[1] . "' />";
-    $form .= '<br>' . _MB_NEWBB_TIME . "<input type='text' name='options[2]' value='" . $options[2] . "' />";
-    $form .= '<br>&nbsp;&nbsp;&nbsp;&nbsp;<small>' . _MB_NEWBB_TIME_DESC . '</small>';
-    $form .= '<br>' . _MB_NEWBB_DISPLAYMODE . "<input type='radio' name='options[3]' value='0'";
-    if (0 == $options[3]) {
-        $form .= ' checked';
-    }
-    $form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_FULL . "<input type='radio' name='options[3]' value='1'";
-    if (1 == $options[3]) {
-        $form .= ' checked';
-    }
-    $form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_COMPACT . "<input type='radio' name='options[3]' value='2'";
-    if (2 == $options[3]) {
-        $form .= ' checked';
-    }
-    $form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_LITE;
-
-    $form .= '<br>' . _MB_NEWBB_INDEXNAV . '<input type="radio" name="options[4]" value="1"';
-    if (1 == $options[4]) {
-        $form .= ' checked';
-    }
-    $form .= ' />' . _YES . '<input type="radio" name="options[4]" value="0"';
-    if (0 == $options[4]) {
-        $form .= ' checked';
-    }
-    $form .= ' />' . _NO;
-
-    $form .= '<br>' . _MB_NEWBB_TITLE_LENGTH . "<input type='text' name='options[5]' value='" . $options[5] . "' />";
-
-    $form .= '<br><br>' . _MB_NEWBB_FORUMLIST;
-
-    $optionsForum = array_filter(array_slice($options, 6), 'b_newbb_array_filter'); // get allowed forums
-    $isAll        = (0 === count($optionsForum) || empty($optionsForum[0]));
-    $form         .= '<br>&nbsp;&nbsp;<select name="options[]" multiple="multiple">';
-    $form         .= '<option value="0" ';
-    if ($isAll) {
-        $form .= ' selected';
-    }
-    $form .= '>' . _ALL . '</option>';
-    $form .= newbbForumSelectBox($optionsForum);
-    $form .= '</select><br>';
-
-    return $form;
+	require_once  dirname(__DIR__) . '/include/functions.forum.php';
+
+	$form = _MB_NEWBB_CRITERIA . "<select name='options[0]'>";
+	$form .= "<option value='time'";
+	if ('time' === $options[0]) {
+		$form .= " selected='selected' ";
+	}
+	$form .= '>' . _MB_NEWBB_CRITERIA_TIME . '</option>';
+	$form .= '</select>';
+	$form .= '<br>' . _MB_NEWBB_DISPLAY . "<input type='text' name='options[1]' value='" . $options[1] . "' />";
+	$form .= '<br>' . _MB_NEWBB_TIME . "<input type='text' name='options[2]' value='" . $options[2] . "' />";
+	$form .= '<br>&nbsp;&nbsp;&nbsp;&nbsp;<small>' . _MB_NEWBB_TIME_DESC . '</small>';
+	$form .= '<br>' . _MB_NEWBB_DISPLAYMODE . "<input type='radio' name='options[3]' value='0'";
+	if (0 == $options[3]) {
+		$form .= ' checked';
+	}
+	$form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_FULL . "<input type='radio' name='options[3]' value='1'";
+	if (1 == $options[3]) {
+		$form .= ' checked';
+	}
+	$form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_COMPACT . "<input type='radio' name='options[3]' value='2'";
+	if (2 == $options[3]) {
+		$form .= ' checked';
+	}
+	$form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_LITE;
+
+	$form .= '<br>' . _MB_NEWBB_INDEXNAV . '<input type="radio" name="options[4]" value="1"';
+	if (1 == $options[4]) {
+		$form .= ' checked';
+	}
+	$form .= ' />' . _YES . '<input type="radio" name="options[4]" value="0"';
+	if (0 == $options[4]) {
+		$form .= ' checked';
+	}
+	$form .= ' />' . _NO;
+
+	$form .= '<br>' . _MB_NEWBB_TITLE_LENGTH . "<input type='text' name='options[5]' value='" . $options[5] . "' />";
+
+	$form .= '<br><br>' . _MB_NEWBB_FORUMLIST;
+
+	$optionsForum = array_filter(array_slice($options, 6), 'b_newbb_array_filter'); // get allowed forums
+	$isAll        = (0 === count($optionsForum) || empty($optionsForum[0]));
+	$form         .= '<br>&nbsp;&nbsp;<select name="options[]" multiple="multiple">';
+	$form         .= '<option value="0" ';
+	if ($isAll) {
+		$form .= ' selected';
+	}
+	$form .= '>' . _ALL . '</option>';
+	$form .= newbbForumSelectBox($optionsForum);
+	$form .= '</select><br>';
+
+	return $form;
 }
 
 /**
@@ -729,78 +729,78 @@ 
  */
 function b_newbb_topic_edit($options)
 {
-    require_once  dirname(__DIR__) . '/include/functions.forum.php';
-    $form = _MB_NEWBB_CRITERIA . "<select name='options[0]'>";
-    $form .= "<option value='time'";
-    if ('time' === $options[0]) {
-        $form .= " selected='selected' ";
-    }
-    $form .= '>' . _MB_NEWBB_CRITERIA_TIME . '</option>';
-    $form .= "<option value='views'";
-    if ('views' === $options[0]) {
-        $form .= " selected='selected' ";
-    }
-    $form .= '>' . _MB_NEWBB_CRITERIA_VIEWS . '</option>';
-    $form .= "<option value='replies'";
-    if ('replies' === $options[0]) {
-        $form .= " selected='selected' ";
-    }
-    $form .= '>' . _MB_NEWBB_CRITERIA_REPLIES . '</option>';
-    $form .= "<option value='digest'";
-    if ('digest' === $options[0]) {
-        $form .= " selected='selected' ";
-    }
-    $form .= '>' . _MB_NEWBB_CRITERIA_DIGEST . '</option>';
-    $form .= "<option value='sticky'";
-    if ('sticky' === $options[0]) {
-        $form .= " selected='selected' ";
-    }
-    $form .= '>' . _MB_NEWBB_CRITERIA_STICKY . '</option>';
-    $form .= '</select>';
-    $form .= '<br>' . _MB_NEWBB_DISPLAY . "<input type='text' name='options[1]' value='" . $options[1] . "' />";
-    $form .= '<br>' . _MB_NEWBB_TIME . "<input type='text' name='options[2]' value='" . $options[2] . "' />";
-    $form .= '<br>&nbsp;&nbsp;&nbsp;&nbsp;<small>' . _MB_NEWBB_TIME_DESC . '</small>';
-    $form .= '<br>' . _MB_NEWBB_DISPLAYMODE . "<input type='radio' name='options[3]' value='0'";
-    if (0 == $options[3]) {
-        $form .= ' checked';
-    }
-    $form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_FULL . "<input type='radio' name='options[3]' value='1'";
-    if (1 == $options[3]) {
-        $form .= ' checked';
-    }
-    $form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_COMPACT . "<input type='radio' name='options[3]' value='2'";
-    if (2 == $options[3]) {
-        $form .= ' checked';
-    }
-    $form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_LITE;
-
-    $form .= '<br>' . _MB_NEWBB_INDEXNAV . '<input type="radio" name="options[4]" value="1"';
-    if (1 == $options[4]) {
-        $form .= ' checked';
-    }
-    $form .= ' />' . _YES . '<input type="radio" name="options[4]" value="0"';
-    if (0 == $options[4]) {
-        $form .= ' checked';
-    }
-    $form .= ' />' . _NO;
-
-    $form .= '<br>' . _MB_NEWBB_TITLE_LENGTH . "<input type='text' name='options[5]' value='" . $options[5] . "' />";
-
-    $form .= '<br><br>' . _MB_NEWBB_FORUMLIST;
-
-    $optionsForum = array_filter(array_slice($options, 6), 'b_newbb_array_filter'); // get allowed forums
-
-    $isAll = (0 === count($optionsForum) || empty($optionsForum[0]));
-    $form  .= '<br>&nbsp;&nbsp;<select name="options[]" multiple="multiple">';
-    $form  .= '<option value="0" ';
-    if ($isAll) {
-        $form .= ' selected="selected"';
-    }
-    $form .= '>' . _ALL . '</option>';
-    $form .= newbbForumSelectBox($optionsForum);
-    $form .= '</select><br>';
-
-    return $form;
+	require_once  dirname(__DIR__) . '/include/functions.forum.php';
+	$form = _MB_NEWBB_CRITERIA . "<select name='options[0]'>";
+	$form .= "<option value='time'";
+	if ('time' === $options[0]) {
+		$form .= " selected='selected' ";
+	}
+	$form .= '>' . _MB_NEWBB_CRITERIA_TIME . '</option>';
+	$form .= "<option value='views'";
+	if ('views' === $options[0]) {
+		$form .= " selected='selected' ";
+	}
+	$form .= '>' . _MB_NEWBB_CRITERIA_VIEWS . '</option>';
+	$form .= "<option value='replies'";
+	if ('replies' === $options[0]) {
+		$form .= " selected='selected' ";
+	}
+	$form .= '>' . _MB_NEWBB_CRITERIA_REPLIES . '</option>';
+	$form .= "<option value='digest'";
+	if ('digest' === $options[0]) {
+		$form .= " selected='selected' ";
+	}
+	$form .= '>' . _MB_NEWBB_CRITERIA_DIGEST . '</option>';
+	$form .= "<option value='sticky'";
+	if ('sticky' === $options[0]) {
+		$form .= " selected='selected' ";
+	}
+	$form .= '>' . _MB_NEWBB_CRITERIA_STICKY . '</option>';
+	$form .= '</select>';
+	$form .= '<br>' . _MB_NEWBB_DISPLAY . "<input type='text' name='options[1]' value='" . $options[1] . "' />";
+	$form .= '<br>' . _MB_NEWBB_TIME . "<input type='text' name='options[2]' value='" . $options[2] . "' />";
+	$form .= '<br>&nbsp;&nbsp;&nbsp;&nbsp;<small>' . _MB_NEWBB_TIME_DESC . '</small>';
+	$form .= '<br>' . _MB_NEWBB_DISPLAYMODE . "<input type='radio' name='options[3]' value='0'";
+	if (0 == $options[3]) {
+		$form .= ' checked';
+	}
+	$form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_FULL . "<input type='radio' name='options[3]' value='1'";
+	if (1 == $options[3]) {
+		$form .= ' checked';
+	}
+	$form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_COMPACT . "<input type='radio' name='options[3]' value='2'";
+	if (2 == $options[3]) {
+		$form .= ' checked';
+	}
+	$form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_LITE;
+
+	$form .= '<br>' . _MB_NEWBB_INDEXNAV . '<input type="radio" name="options[4]" value="1"';
+	if (1 == $options[4]) {
+		$form .= ' checked';
+	}
+	$form .= ' />' . _YES . '<input type="radio" name="options[4]" value="0"';
+	if (0 == $options[4]) {
+		$form .= ' checked';
+	}
+	$form .= ' />' . _NO;
+
+	$form .= '<br>' . _MB_NEWBB_TITLE_LENGTH . "<input type='text' name='options[5]' value='" . $options[5] . "' />";
+
+	$form .= '<br><br>' . _MB_NEWBB_FORUMLIST;
+
+	$optionsForum = array_filter(array_slice($options, 6), 'b_newbb_array_filter'); // get allowed forums
+
+	$isAll = (0 === count($optionsForum) || empty($optionsForum[0]));
+	$form  .= '<br>&nbsp;&nbsp;<select name="options[]" multiple="multiple">';
+	$form  .= '<option value="0" ';
+	if ($isAll) {
+		$form .= ' selected="selected"';
+	}
+	$form .= '>' . _ALL . '</option>';
+	$form .= newbbForumSelectBox($optionsForum);
+	$form .= '</select><br>';
+
+	return $form;
 }
 
 /**
@@ -809,62 +809,62 @@ 
  */
 function b_newbb_post_edit($options)
 {
-    require_once  dirname(__DIR__) . '/include/functions.forum.php';
-    $form = _MB_NEWBB_CRITERIA . "<select name='options[0]'>";
-    $form .= "<option value='title'";
-    if ('title' === $options[0]) {
-        $form .= " selected='selected' ";
-    }
-    $form .= '>' . _MB_NEWBB_CRITERIA_TITLE . '</option>';
-    $form .= "<option value='text'";
-    if ('text' === $options[0]) {
-        $form .= " selected='selected' ";
-    }
-    $form .= '>' . _MB_NEWBB_CRITERIA_TEXT . '</option>';
-    $form .= '</select>';
-    $form .= '<br>' . _MB_NEWBB_DISPLAY . "<input type='text' name='options[1]' value='" . $options[1] . "' />";
-    $form .= '<br>' . _MB_NEWBB_TIME . "<input type='text' name='options[2]' value='" . $options[2] . "' />";
-    $form .= '<br>&nbsp;&nbsp;&nbsp;&nbsp;<small>' . _MB_NEWBB_TIME_DESC . '</small>';
-    $form .= '<br>' . _MB_NEWBB_DISPLAYMODE . "<input type='radio' name='options[3]' value='0'";
-    if (0 == $options[3]) {
-        $form .= ' checked';
-    }
-    $form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_FULL . "<input type='radio' name='options[3]' value='1'";
-    if (1 == $options[3]) {
-        $form .= ' checked';
-    }
-    $form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_COMPACT . "<input type='radio' name='options[3]' value='2'";
-    if (2 == $options[3]) {
-        $form .= ' checked';
-    }
-    $form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_LITE;
-
-    $form .= '<br>' . _MB_NEWBB_INDEXNAV . '<input type="radio" name="options[4]" value="1"';
-    if (1 == $options[4]) {
-        $form .= ' checked';
-    }
-    $form .= ' />' . _YES . '<input type="radio" name="options[4]" value="0"';
-    if (0 == $options[4]) {
-        $form .= ' checked';
-    }
-    $form .= ' />' . _NO;
-
-    $form .= '<br>' . _MB_NEWBB_TITLE_LENGTH . "<input type='text' name='options[5]' value='" . $options[5] . "' />";
-
-    $form .= '<br><br>' . _MB_NEWBB_FORUMLIST;
-
-    $optionsForum = array_filter(array_slice($options, 6), 'b_newbb_array_filter'); // get allowed forums
-    $isAll        = (0 === count($optionsForum) || empty($optionsForum[0]));
-    $form         .= '<br>&nbsp;&nbsp;<select name="options[]" multiple="multiple">';
-    $form         .= '<option value="0" ';
-    if ($isAll) {
-        $form .= ' selected="selected"';
-    }
-    $form .= '>' . _ALL . '</option>';
-    $form .= newbbForumSelectBox($optionsForum);
-    $form .= '</select><br>';
-
-    return $form;
+	require_once  dirname(__DIR__) . '/include/functions.forum.php';
+	$form = _MB_NEWBB_CRITERIA . "<select name='options[0]'>";
+	$form .= "<option value='title'";
+	if ('title' === $options[0]) {
+		$form .= " selected='selected' ";
+	}
+	$form .= '>' . _MB_NEWBB_CRITERIA_TITLE . '</option>';
+	$form .= "<option value='text'";
+	if ('text' === $options[0]) {
+		$form .= " selected='selected' ";
+	}
+	$form .= '>' . _MB_NEWBB_CRITERIA_TEXT . '</option>';
+	$form .= '</select>';
+	$form .= '<br>' . _MB_NEWBB_DISPLAY . "<input type='text' name='options[1]' value='" . $options[1] . "' />";
+	$form .= '<br>' . _MB_NEWBB_TIME . "<input type='text' name='options[2]' value='" . $options[2] . "' />";
+	$form .= '<br>&nbsp;&nbsp;&nbsp;&nbsp;<small>' . _MB_NEWBB_TIME_DESC . '</small>';
+	$form .= '<br>' . _MB_NEWBB_DISPLAYMODE . "<input type='radio' name='options[3]' value='0'";
+	if (0 == $options[3]) {
+		$form .= ' checked';
+	}
+	$form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_FULL . "<input type='radio' name='options[3]' value='1'";
+	if (1 == $options[3]) {
+		$form .= ' checked';
+	}
+	$form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_COMPACT . "<input type='radio' name='options[3]' value='2'";
+	if (2 == $options[3]) {
+		$form .= ' checked';
+	}
+	$form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_LITE;
+
+	$form .= '<br>' . _MB_NEWBB_INDEXNAV . '<input type="radio" name="options[4]" value="1"';
+	if (1 == $options[4]) {
+		$form .= ' checked';
+	}
+	$form .= ' />' . _YES . '<input type="radio" name="options[4]" value="0"';
+	if (0 == $options[4]) {
+		$form .= ' checked';
+	}
+	$form .= ' />' . _NO;
+
+	$form .= '<br>' . _MB_NEWBB_TITLE_LENGTH . "<input type='text' name='options[5]' value='" . $options[5] . "' />";
+
+	$form .= '<br><br>' . _MB_NEWBB_FORUMLIST;
+
+	$optionsForum = array_filter(array_slice($options, 6), 'b_newbb_array_filter'); // get allowed forums
+	$isAll        = (0 === count($optionsForum) || empty($optionsForum[0]));
+	$form         .= '<br>&nbsp;&nbsp;<select name="options[]" multiple="multiple">';
+	$form         .= '<option value="0" ';
+	if ($isAll) {
+		$form .= ' selected="selected"';
+	}
+	$form .= '>' . _ALL . '</option>';
+	$form .= newbbForumSelectBox($optionsForum);
+	$form .= '</select><br>';
+
+	return $form;
 }
 
 /**
@@ -873,66 +873,66 @@ 
  */
 function b_newbb_author_edit($options)
 {
-    require_once  dirname(__DIR__) . '/include/functions.forum.php';
-    $form = _MB_NEWBB_CRITERIA . "<select name='options[0]'>";
-    $form .= "<option value='post'";
-    if ('post' === $options[0]) {
-        $form .= " selected='selected' ";
-    }
-    $form .= '>' . _MB_NEWBB_CRITERIA_POST . '</option>';
-    $form .= "<option value='topic'";
-    if ('topic' === $options[0]) {
-        $form .= " selected='selected' ";
-    }
-    $form .= '>' . _MB_NEWBB_CRITERIA_TOPIC . '</option>';
-    $form .= "<option value='digest'";
-    if ('digest' === $options[0]) {
-        $form .= " selected='selected' ";
-    }
-    $form .= '>' . _MB_NEWBB_CRITERIA_DIGESTS . '</option>';
-    $form .= "<option value='sticky'";
-    if ('sticky' === $options[0]) {
-        $form .= " selected='selected' ";
-    }
-    $form .= '>' . _MB_NEWBB_CRITERIA_STICKYS . '</option>';
-    $form .= '</select>';
-    $form .= '<br>' . _MB_NEWBB_DISPLAY . "<input type='text' name='options[1]' value='" . $options[1] . "' />";
-    $form .= '<br>' . _MB_NEWBB_TIME . "<input type='text' name='options[2]' value='" . $options[2] . "' />";
-    $form .= '<br>&nbsp;&nbsp;&nbsp;&nbsp;<small>' . _MB_NEWBB_TIME_DESC . '</small>';
-    $form .= '<br>' . _MB_NEWBB_DISPLAYMODE . "<input type='radio' name='options[3]' value='0'";
-    if (0 == $options[3]) {
-        $form .= ' checked';
-    }
-    $form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_COMPACT . "<input type='radio' name='options[3]' value='1'";
-    if (1 == $options[3]) {
-        $form .= ' checked';
-    }
-    $form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_LITE;
-
-    $form .= '<br>' . _MB_NEWBB_INDEXNAV . '<input type="radio" name="options[4]" value="1"';
-    if (1 == $options[4]) {
-        $form .= ' checked';
-    }
-    $form .= ' />' . _YES . '<input type="radio" name="options[4]" value="0"';
-    if (0 == $options[4]) {
-        $form .= ' checked';
-    }
-    $form .= ' />' . _NO;
-
-    $form .= '<br><br>' . _MB_NEWBB_FORUMLIST;
-
-    $optionsForum = array_filter(array_slice($options, 5), 'b_newbb_array_filter'); // get allowed forums
-    $isAll        = (0 === count($optionsForum) || empty($optionsForum[0]));
-    $form         .= '<br>&nbsp;&nbsp;<select name="options[]" multiple="multiple">';
-    $form         .= '<option value="0" ';
-    if ($isAll) {
-        $form .= ' selected="selected"';
-    }
-    $form .= '>' . _ALL . '</option>';
-    $form .= newbbForumSelectBox($optionsForum);
-    $form .= '</select><br>';
-
-    return $form;
+	require_once  dirname(__DIR__) . '/include/functions.forum.php';
+	$form = _MB_NEWBB_CRITERIA . "<select name='options[0]'>";
+	$form .= "<option value='post'";
+	if ('post' === $options[0]) {
+		$form .= " selected='selected' ";
+	}
+	$form .= '>' . _MB_NEWBB_CRITERIA_POST . '</option>';
+	$form .= "<option value='topic'";
+	if ('topic' === $options[0]) {
+		$form .= " selected='selected' ";
+	}
+	$form .= '>' . _MB_NEWBB_CRITERIA_TOPIC . '</option>';
+	$form .= "<option value='digest'";
+	if ('digest' === $options[0]) {
+		$form .= " selected='selected' ";
+	}
+	$form .= '>' . _MB_NEWBB_CRITERIA_DIGESTS . '</option>';
+	$form .= "<option value='sticky'";
+	if ('sticky' === $options[0]) {
+		$form .= " selected='selected' ";
+	}
+	$form .= '>' . _MB_NEWBB_CRITERIA_STICKYS . '</option>';
+	$form .= '</select>';
+	$form .= '<br>' . _MB_NEWBB_DISPLAY . "<input type='text' name='options[1]' value='" . $options[1] . "' />";
+	$form .= '<br>' . _MB_NEWBB_TIME . "<input type='text' name='options[2]' value='" . $options[2] . "' />";
+	$form .= '<br>&nbsp;&nbsp;&nbsp;&nbsp;<small>' . _MB_NEWBB_TIME_DESC . '</small>';
+	$form .= '<br>' . _MB_NEWBB_DISPLAYMODE . "<input type='radio' name='options[3]' value='0'";
+	if (0 == $options[3]) {
+		$form .= ' checked';
+	}
+	$form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_COMPACT . "<input type='radio' name='options[3]' value='1'";
+	if (1 == $options[3]) {
+		$form .= ' checked';
+	}
+	$form .= ' />&nbsp;' . _MB_NEWBB_DISPLAYMODE_LITE;
+
+	$form .= '<br>' . _MB_NEWBB_INDEXNAV . '<input type="radio" name="options[4]" value="1"';
+	if (1 == $options[4]) {
+		$form .= ' checked';
+	}
+	$form .= ' />' . _YES . '<input type="radio" name="options[4]" value="0"';
+	if (0 == $options[4]) {
+		$form .= ' checked';
+	}
+	$form .= ' />' . _NO;
+
+	$form .= '<br><br>' . _MB_NEWBB_FORUMLIST;
+
+	$optionsForum = array_filter(array_slice($options, 5), 'b_newbb_array_filter'); // get allowed forums
+	$isAll        = (0 === count($optionsForum) || empty($optionsForum[0]));
+	$form         .= '<br>&nbsp;&nbsp;<select name="options[]" multiple="multiple">';
+	$form         .= '<option value="0" ';
+	if ($isAll) {
+		$form .= ' selected="selected"';
+	}
+	$form .= '>' . _ALL . '</option>';
+	$form .= newbbForumSelectBox($optionsForum);
+	$form .= '</select><br>';
+
+	return $form;
 }
 
 /**
@@ -941,22 +941,22 @@ 
  */
 function b_newbb_custom($options)
 {
-    // if no newbb module block set, we have to include the language file
-    if (is_readable($GLOBALS['xoops']->path('modules/newbb/language/' . $GLOBALS['xoopsConfig']['language'] . '/blocks.php'))) {
-        require_once $GLOBALS['xoops']->path('modules/newbb/language/' . $GLOBALS['xoopsConfig']['language'] . '/blocks.php');
-    } else {
-        require_once $GLOBALS['xoops']->path('modules/newbb/language/english/blocks.php');
-    }
-
-    $options = explode('|', $options);
-    $block   = b_newbb_show($options);
-    if (count($block['topics']) < 1) {
-        return false;
-    }
-
-    $tpl = new \XoopsTpl();
-    $tpl->assign('block', $block);
-    $tpl->display('db:newbb_block.tpl');
+	// if no newbb module block set, we have to include the language file
+	if (is_readable($GLOBALS['xoops']->path('modules/newbb/language/' . $GLOBALS['xoopsConfig']['language'] . '/blocks.php'))) {
+		require_once $GLOBALS['xoops']->path('modules/newbb/language/' . $GLOBALS['xoopsConfig']['language'] . '/blocks.php');
+	} else {
+		require_once $GLOBALS['xoops']->path('modules/newbb/language/english/blocks.php');
+	}
+
+	$options = explode('|', $options);
+	$block   = b_newbb_show($options);
+	if (count($block['topics']) < 1) {
+		return false;
+	}
+
+	$tpl = new \XoopsTpl();
+	$tpl->assign('block', $block);
+	$tpl->display('db:newbb_block.tpl');
 }
 
 /**
@@ -965,20 +965,20 @@ 
  */
 function b_newbb_custom_topic($options)
 {
-    /** @var \XoopsModules\Newbb\Helper $helper */
-    $helper = \XoopsModules\Newbb\Helper::getInstance();
-    // if no newbb module block set, we have to include the language file
-    $helper->loadLanguage('blocks');
-
-    $options = explode('|', $options);
-    $block   = b_newbb_topic_show($options);
-    if (count($block['topics']) < 1) {
-        return false;
-    }
-
-    $tpl = new \XoopsTpl();
-    $tpl->assign('block', $block);
-    $tpl->display('db:newbb_block_topic.tpl');
+	/** @var \XoopsModules\Newbb\Helper $helper */
+	$helper = \XoopsModules\Newbb\Helper::getInstance();
+	// if no newbb module block set, we have to include the language file
+	$helper->loadLanguage('blocks');
+
+	$options = explode('|', $options);
+	$block   = b_newbb_topic_show($options);
+	if (count($block['topics']) < 1) {
+		return false;
+	}
+
+	$tpl = new \XoopsTpl();
+	$tpl->assign('block', $block);
+	$tpl->display('db:newbb_block_topic.tpl');
 }
 
 /**
@@ -987,20 +987,20 @@ 
  */
 function b_newbb_custom_post($options)
 {
-    /** @var \XoopsModules\Newbb\Helper $helper */
-    $helper = \XoopsModules\Newbb\Helper::getInstance();
-    // if no newbb module block set, we have to include the language file
-    $helper->loadLanguage('blocks');
-
-    $options = explode('|', $options);
-    $block   = b_newbb_post_show($options);
-    if (count($block['topics']) < 1) {
-        return false;
-    }
-
-    $tpl = new \XoopsTpl();
-    $tpl->assign('block', $block);
-    $tpl->display('db:newbb_block_post.tpl');
+	/** @var \XoopsModules\Newbb\Helper $helper */
+	$helper = \XoopsModules\Newbb\Helper::getInstance();
+	// if no newbb module block set, we have to include the language file
+	$helper->loadLanguage('blocks');
+
+	$options = explode('|', $options);
+	$block   = b_newbb_post_show($options);
+	if (count($block['topics']) < 1) {
+		return false;
+	}
+
+	$tpl = new \XoopsTpl();
+	$tpl->assign('block', $block);
+	$tpl->display('db:newbb_block_post.tpl');
 }
 
 /**
@@ -1009,20 +1009,20 @@ 
  */
 function b_newbb_custom_author($options)
 {
-    /** @var \XoopsModules\Newbb\Helper $helper */
-    $helper = \XoopsModules\Newbb\Helper::getInstance();
-    // if no newbb module block set, we have to include the language file
-    $helper->loadLanguage('blocks');
-
-    $options = explode('|', $options);
-    $block   = b_newbb_author_show($options);
-    if (count($block['authors']) < 1) {
-        return false;
-    }
-
-    $tpl = new \XoopsTpl();
-    $tpl->assign('block', $block);
-    $tpl->display('db:newbb_block_author.tpl');
+	/** @var \XoopsModules\Newbb\Helper $helper */
+	$helper = \XoopsModules\Newbb\Helper::getInstance();
+	// if no newbb module block set, we have to include the language file
+	$helper->loadLanguage('blocks');
+
+	$options = explode('|', $options);
+	$block   = b_newbb_author_show($options);
+	if (count($block['authors']) < 1) {
+		return false;
+	}
+
+	$tpl = new \XoopsTpl();
+	$tpl->assign('block', $block);
+	$tpl->display('db:newbb_block_author.tpl');
 }
 
 // irmtfan add local stylesheet and js footer.php

blocks/newbb_block_tag.php

Indentation +20 added lines, -20 removed lines

@@ -20,12 +20,12 @@ 
  */
 function newbb_tag_block_cloud_show($options)
 {
-    if (!@require_once $GLOBALS['xoops']->path('modules/tag/blocks/block.php')) {
-        return null;
-    }
-    $block_content = tag_block_cloud_show($options, 'newbb');
+	if (!@require_once $GLOBALS['xoops']->path('modules/tag/blocks/block.php')) {
+		return null;
+	}
+	$block_content = tag_block_cloud_show($options, 'newbb');
 
-    return $block_content;
+	return $block_content;
 }
 
 /**
@@ -34,12 +34,12 @@ 
  */
 function newbb_tag_block_cloud_edit($options)
 {
-    if (!@require_once $GLOBALS['xoops']->path('modules/tag/blocks/block.php')) {
-        return null;
-    }
-    $form = tag_block_cloud_edit($options);
+	if (!@require_once $GLOBALS['xoops']->path('modules/tag/blocks/block.php')) {
+		return null;
+	}
+	$form = tag_block_cloud_edit($options);
 
-    return $form;
+	return $form;
 }
 
 /**#@+
@@ -49,12 +49,12 @@ 
  */
 function newbb_tag_block_top_show($options)
 {
-    if (!@require_once $GLOBALS['xoops']->path('modules/tag/blocks/block.php')) {
-        return null;
-    }
-    $block_content = tag_block_top_show($options, 'newbb');
+	if (!@require_once $GLOBALS['xoops']->path('modules/tag/blocks/block.php')) {
+		return null;
+	}
+	$block_content = tag_block_top_show($options, 'newbb');
 
-    return $block_content;
+	return $block_content;
 }
 
 /**
@@ -63,10 +63,10 @@ 
  */
 function newbb_tag_block_top_edit($options)
 {
-    if (!@require_once $GLOBALS['xoops']->path('modules/tag/blocks/block.php')) {
-        return null;
-    }
-    $form = tag_block_top_edit($options);
+	if (!@require_once $GLOBALS['xoops']->path('modules/tag/blocks/block.php')) {
+		return null;
+	}
+	$form = tag_block_top_edit($options);
 
-    return $form;
+	return $form;
 }

report.php

Indentation +69 added lines, -69 removed lines

@@ -35,22 +35,22 @@ 
 
 $GPC = '_GET';
 if (Request::getString('submit', '', 'POST')) {
-    $GPC = '_POST';
+	$GPC = '_POST';
 }
 
 foreach (['post_id', 'order', 'forum', 'topic_id'] as $getint) {
-    ${$getint} = (int)(@${$GPC}[$getint]);
+	${$getint} = (int)(@${$GPC}[$getint]);
 }
 $viewmode = (isset(${$GPC}['viewmode']) && 'flat' !== ${$GPC}['viewmode']) ? 'thread' : 'flat';
 
 if (empty($post_id)) {
-    redirect_header(XOOPS_URL . '/index.php', 2, _MD_NEWBB_ERRORPOST);
+	redirect_header(XOOPS_URL . '/index.php', 2, _MD_NEWBB_ERRORPOST);
 }
 
 if ($GLOBALS['xoopsModuleConfig']['wol_enabled']) {
-    //    /** @var Newbb\OnlineHandler $onlineHandler */
-    //    $onlineHandler = Newbb\Helper::getInstance()->getHandler('Online');
-    $onlineHandler->init($forum);
+	//    /** @var Newbb\OnlineHandler $onlineHandler */
+	//    $onlineHandler = Newbb\Helper::getInstance()->getHandler('Online');
+	$onlineHandler->init($forum);
 }
 
 $myts = \MyTextSanitizer::getInstance();
@@ -60,68 +60,68 @@ 
 require_once $GLOBALS['xoops']->path('class/xoopsformloader.php');
 
 if (Request::hasVar('submit', 'POST')) {
-    $error_message = '';
-    if (!is_object($GLOBALS['xoopsUser'])) {
-        xoops_load('xoopscaptcha');
-        $xoopsCaptcha = \XoopsCaptcha::getInstance();
-        if (!$xoopsCaptcha->verify()) {
-            $captcha_invalid = true;
-            $error_message   = $xoopsCaptcha->getMessage();
-        }
-    }
-    if ('' !== $error_message) {
-        xoops_error($error_message);
-    } else {
-        //        $reportHandler = Newbb\Helper::getInstance()->getHandler('Report');
-        $report = $reportHandler->create();
-        $report->setVar('report_text', Request::getString('report_text', '', 'POST'));
-        $report->setVar('post_id', Request::getInt('post_id', 0, 'POST'));
-        $report->setVar('report_time', time());
-        $report->setVar('reporter_uid', is_object($GLOBALS['xoopsUser']) ? $GLOBALS['xoopsUser']->getVar('uid') : 0);
-        $report->setVar('reporter_ip', \Xmf\IPAddress::fromRequest()->asReadable());
-        $report->setVar('report_result', 0);
-        $report->setVar('report_memo', '');
-
-        if ($report_id = $reportHandler->insert($report)) {
-            //            $forumHandler = Newbb\Helper::getInstance()->getHandler('Forum');
-            if (empty($forum)) {
-            }
-            $forumObject = $forumHandler->get($forum);
-
-            if (is_object($forumObject)) {
-                $mods   = $forumObject->getVar('forum_moderator');
-                $emails = [];
-                /** @var \XoopsMemberHandler $memberHandler */
-                $memberHandler = xoops_getHandler('member');
-                foreach ($mods as $mod) {
-                    $thisUser = $memberHandler->getUser($mod);
-                    if (is_object($thisUser)) {
-                        $emails[] = $thisUser->getVar('email');
-                        unset($thisUser);
-                    }
-                }
-                $xoopsMailer = xoops_getMailer();
-                $xoopsMailer->reset();
-                $xoopsMailer->setTemplateDir();
-                $xoopsMailer->useMail();
-                $xoopsMailer->setTemplate('forum_report.tpl');
-                $xoopsMailer->setToEmails($emails);
-                $xoopsMailer->assign('MESSAGE', Request::getString('report_text', '', 'POST'));
-                $xoopsMailer->setSubject(_MD_NEWBB_REPORTSUBJECT);
-                $xoopsMailer->send();
-            }
-            $message = _MD_NEWBB_REPORTED;
-        } else {
-            $message = _MD_NEWBB_REPORT_ERROR;
-        }
-        redirect_header("viewtopic.php?forum=$forum&topic_id=$topic_id&post_id=$post_id&order=$order&viewmode=$viewmode", 2, $message);
-    }
+	$error_message = '';
+	if (!is_object($GLOBALS['xoopsUser'])) {
+		xoops_load('xoopscaptcha');
+		$xoopsCaptcha = \XoopsCaptcha::getInstance();
+		if (!$xoopsCaptcha->verify()) {
+			$captcha_invalid = true;
+			$error_message   = $xoopsCaptcha->getMessage();
+		}
+	}
+	if ('' !== $error_message) {
+		xoops_error($error_message);
+	} else {
+		//        $reportHandler = Newbb\Helper::getInstance()->getHandler('Report');
+		$report = $reportHandler->create();
+		$report->setVar('report_text', Request::getString('report_text', '', 'POST'));
+		$report->setVar('post_id', Request::getInt('post_id', 0, 'POST'));
+		$report->setVar('report_time', time());
+		$report->setVar('reporter_uid', is_object($GLOBALS['xoopsUser']) ? $GLOBALS['xoopsUser']->getVar('uid') : 0);
+		$report->setVar('reporter_ip', \Xmf\IPAddress::fromRequest()->asReadable());
+		$report->setVar('report_result', 0);
+		$report->setVar('report_memo', '');
+
+		if ($report_id = $reportHandler->insert($report)) {
+			//            $forumHandler = Newbb\Helper::getInstance()->getHandler('Forum');
+			if (empty($forum)) {
+			}
+			$forumObject = $forumHandler->get($forum);
+
+			if (is_object($forumObject)) {
+				$mods   = $forumObject->getVar('forum_moderator');
+				$emails = [];
+				/** @var \XoopsMemberHandler $memberHandler */
+				$memberHandler = xoops_getHandler('member');
+				foreach ($mods as $mod) {
+					$thisUser = $memberHandler->getUser($mod);
+					if (is_object($thisUser)) {
+						$emails[] = $thisUser->getVar('email');
+						unset($thisUser);
+					}
+				}
+				$xoopsMailer = xoops_getMailer();
+				$xoopsMailer->reset();
+				$xoopsMailer->setTemplateDir();
+				$xoopsMailer->useMail();
+				$xoopsMailer->setTemplate('forum_report.tpl');
+				$xoopsMailer->setToEmails($emails);
+				$xoopsMailer->assign('MESSAGE', Request::getString('report_text', '', 'POST'));
+				$xoopsMailer->setSubject(_MD_NEWBB_REPORTSUBJECT);
+				$xoopsMailer->send();
+			}
+			$message = _MD_NEWBB_REPORTED;
+		} else {
+			$message = _MD_NEWBB_REPORT_ERROR;
+		}
+		redirect_header("viewtopic.php?forum=$forum&topic_id=$topic_id&post_id=$post_id&order=$order&viewmode=$viewmode", 2, $message);
+	}
 }
 
 $report_form = new \XoopsThemeForm('', 'reportform', 'report.php');
 $report_form->addElement(new \XoopsFormText(_MD_NEWBB_REPORT_TEXT, 'report_text', 80, 255, Request::getString('report_text', '', 'POST')), true);
 if (!is_object($GLOBALS['xoopsUser'])) {
-    $report_form->addElement(new \XoopsFormCaptcha());
+	$report_form->addElement(new \XoopsFormCaptcha());
 }
 
 //$postHandler = Newbb\Helper::getInstance()->getHandler('Post');
@@ -147,19 +147,19 @@ 
 
 $r_subject = $postObject->getVar('subject', 'E');
 if ($GLOBALS['xoopsModuleConfig']['enable_karma'] && $postObject->getVar('post_karma') > 0) {
-    $r_message = sprintf(_MD_NEWBB_KARMA_REQUIREMENT, '***', $postObject->getVar('post_karma')) . '</div>';
+	$r_message = sprintf(_MD_NEWBB_KARMA_REQUIREMENT, '***', $postObject->getVar('post_karma')) . '</div>';
 } elseif ($GLOBALS['xoopsModuleConfig']['allow_require_reply'] && $postObject->getVar('require_reply')) {
-    $r_message = _MD_NEWBB_REPLY_REQUIREMENT;
+	$r_message = _MD_NEWBB_REPLY_REQUIREMENT;
 } else {
-    $r_message = $postObject->getVar('post_text');
+	$r_message = $postObject->getVar('post_text');
 }
 
 $r_date = formatTimestamp($postObject->getVar('post_time'));
 if ($postObject->getVar('uid')) {
-    $r_name = newbbGetUnameFromId($postObject->getVar('uid'), $GLOBALS['xoopsModuleConfig']['show_realname']);
+	$r_name = newbbGetUnameFromId($postObject->getVar('uid'), $GLOBALS['xoopsModuleConfig']['show_realname']);
 } else {
-    $poster_name = $postObject->getVar('poster_name');
-    $r_name      = empty($poster_name) ? $GLOBALS['xoopsConfig']['anonymous'] : $myts->htmlSpecialChars($poster_name);
+	$poster_name = $postObject->getVar('poster_name');
+	$r_name      = empty($poster_name) ? $GLOBALS['xoopsConfig']['anonymous'] : $myts->htmlSpecialChars($poster_name);
 }
 $r_content = _MD_NEWBB_SUBJECTC . ' ' . $r_subject . '<br>';
 $r_content .= _MD_NEWBB_BY . ' ' . $r_name . ' ' . _MD_NEWBB_ON . ' ' . $r_date . '<br><br>';

update_type.php

Indentation +29 added lines, -29 removed lines

@@ -11,15 +11,15 @@ 
 require_once __DIR__ . '/header.php';
 
 if (!is_object($GLOBALS['xoopsUser']) || !$GLOBALS['xoopsUser']->isAdmin()) {
-    exit(_NOPERM);
+	exit(_NOPERM);
 }
 
 if ($xoopsModule->getVar('version') >= 401) {
-    exit('Version not valid');
+	exit('Version not valid');
 }
 
 if (empty($GLOBALS['xoopsModuleConfig']['subject_prefix'])) {
-    exit('No need for update');
+	exit('No need for update');
 }
 
 $GLOBALS['xoopsDB']->queryF('DROP TABLE ' . $GLOBALS['xoopsDB']->prefix('newbb_type_tmp'));
@@ -35,7 +35,7 @@ 
           PRIMARY KEY              (`type_id`)
         ) ENGINE=MyISAM;
     ")) {
-    exit('Can not create tmp table for `bb_type_tmp`');
+	exit('Can not create tmp table for `bb_type_tmp`');
 }
 
 if (!$GLOBALS['xoopsDB']->queryF('
@@ -50,8 +50,8 @@ 
           KEY `type_order`        (`type_order`)
         ) ENGINE=MyISAM;
     ")) {
-    $GLOBALS['xoopsDB']->queryF('DROP TABLE ' . $GLOBALS['xoopsDB']->prefix('newbb_type_tmp'));
-    exit('Can not create tmp table for `bb_type_forum_tmp`');
+	$GLOBALS['xoopsDB']->queryF('DROP TABLE ' . $GLOBALS['xoopsDB']->prefix('newbb_type_tmp'));
+	exit('Can not create tmp table for `bb_type_forum_tmp`');
 }
 
 //$typeHandler = Newbb\Helper::getInstance()->getHandler('Type');
@@ -59,43 +59,43 @@ 
 $types       = [];
 $order       = 1;
 foreach ($subjectpres as $subjectpre) {
-    if (preg_match("/<[^#]*color=[\"'](#[^'\"\s]*)[^>]>[\[]?([^<\]]*)[\]]?/is", $subjectpre, $matches)) {
-        if (!$GLOBALS['xoopsDB']->queryF('
+	if (preg_match("/<[^#]*color=[\"'](#[^'\"\s]*)[^>]>[\[]?([^<\]]*)[\]]?/is", $subjectpre, $matches)) {
+		if (!$GLOBALS['xoopsDB']->queryF('
                 INSERT INTO ' . $GLOBALS['xoopsDB']->prefix('newbb_type_tmp') . '
                     (`type_name`, `type_color`)
                 VALUES
                     (' . $GLOBALS['xoopsDB']->quoteString($matches[2]) . ', ' . $GLOBALS['xoopsDB']->quoteString($matches[1]) . ')
             ')) {
-            xoops_error("Can not add type of `{$matches[2]}`");
-            continue;
-        }
-        $types[$GLOBALS['xoopsDB']->getInsertId()] = $order++;
-    }
+			xoops_error("Can not add type of `{$matches[2]}`");
+			continue;
+		}
+		$types[$GLOBALS['xoopsDB']->getInsertId()] = $order++;
+	}
 }
 if (0 === count($types)) {
-    $GLOBALS['xoopsDB']->queryF('DROP TABLE ' . $GLOBALS['xoopsDB']->prefix('newbb_type_tmp'));
-    $GLOBALS['xoopsDB']->queryF('DROP TABLE ' . $GLOBALS['xoopsDB']->prefix('newbb_type_forum_tmp'));
-    exit('No type item created');
+	$GLOBALS['xoopsDB']->queryF('DROP TABLE ' . $GLOBALS['xoopsDB']->prefix('newbb_type_tmp'));
+	$GLOBALS['xoopsDB']->queryF('DROP TABLE ' . $GLOBALS['xoopsDB']->prefix('newbb_type_forum_tmp'));
+	exit('No type item created');
 }
 
 ///** @var Newbb\ForumHandler $forumHandler */
 //$forumHandler = Newbb\Helper::getInstance()->getHandler('Forum');
 if ($forums_type = $forumHandler->getIds(new \Criteria('allow_subject_prefix', 1))) {
-    foreach ($forums_type as $forum_id) {
-        $type_query = [];
-        foreach ($types as $key => $order) {
-            $type_query[] = "({$key}, {$forum_id}, {$order})";
-        }
+	foreach ($forums_type as $forum_id) {
+		$type_query = [];
+		foreach ($types as $key => $order) {
+			$type_query[] = "({$key}, {$forum_id}, {$order})";
+		}
 
-        $sql = 'INSERT INTO ' . $GLOBALS['xoopsDB']->prefix('newbb_type_forum_tmp') . ' (type_id, forum_id, type_order) ' . ' VALUES ' . implode(', ', $type_query);
-        if (false === ($result = $GLOBALS['xoopsDB']->queryF($sql))) {
-            xoops_error($GLOBALS['xoopsDB']->error());
-        }
-    }
+		$sql = 'INSERT INTO ' . $GLOBALS['xoopsDB']->prefix('newbb_type_forum_tmp') . ' (type_id, forum_id, type_order) ' . ' VALUES ' . implode(', ', $type_query);
+		if (false === ($result = $GLOBALS['xoopsDB']->queryF($sql))) {
+			xoops_error($GLOBALS['xoopsDB']->error());
+		}
+	}
 } else {
-    $GLOBALS['xoopsDB']->queryF('DROP TABLE ' . $GLOBALS['xoopsDB']->prefix('newbb_type_tmp'));
-    $GLOBALS['xoopsDB']->queryF('DROP TABLE ' . $GLOBALS['xoopsDB']->prefix('newbb_type_forum_tmp'));
-    exit('No type item to update');
+	$GLOBALS['xoopsDB']->queryF('DROP TABLE ' . $GLOBALS['xoopsDB']->prefix('newbb_type_tmp'));
+	$GLOBALS['xoopsDB']->queryF('DROP TABLE ' . $GLOBALS['xoopsDB']->prefix('newbb_type_forum_tmp'));
+	exit('No type item to update');
 }
 
 die('update succeeded');

list.topic.php

Indentation +9 added lines, -9 removed lines

@@ -20,7 +20,7 @@ 
 require_once $GLOBALS['xoops']->path('header.php');
 // irmtfan new method
 if (!empty($GLOBALS['xoopsModuleConfig']['rss_enable'])) {
-    $xoopsTpl->assign('xoops_module_header', '
+	$xoopsTpl->assign('xoops_module_header', '
     <link rel="alternate" type="application/xml+rss" title="' . $xoopsModule->getVar('name') . '" href="' . XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname', 'n') . '/rss.php" />
     ' . @$xoopsTpl->get_template_vars('xoops_module_header'));
 }
@@ -35,7 +35,7 @@ 
 $topic_renderer->userlevel = $GLOBALS['xoopsUserIsAdmin'] ? 2 : is_object($GLOBALS['xoopsUser']);
 // irmtfan if list topic block is in the page then force to parse
 if (defined('LIST_TOPIC_DEFINED')) {
-    $topic_renderer->force = true; // force against static vars
+	$topic_renderer->force = true; // force against static vars
 }
 
 $topic_renderer->is_multiple = true;
@@ -52,14 +52,14 @@ 
 //$isAdmin = $GLOBALS["xoopsUserIsAdmin"];
 /* Only admin has access to admin mode */
 if ($topic_renderer->userlevel < 2) { // irmtfan use userlevel
-    $mode = 0;
+	$mode = 0;
 }
 
 if ($GLOBALS['xoopsModuleConfig']['wol_enabled']) {
-    //    /** @var Newbb\OnlineHandler $onlineHandler */
-    //    $onlineHandler = Newbb\Helper::getInstance()->getHandler('Online');
-    $onlineHandler->init();
-    $onlineHandler->render($xoopsTpl);
+	//    /** @var Newbb\OnlineHandler $onlineHandler */
+	//    $onlineHandler = Newbb\Helper::getInstance()->getHandler('Online');
+	$onlineHandler->init();
+	$onlineHandler->render($xoopsTpl);
 }
 
 $topic_renderer->buildHeaders($xoopsTpl);
@@ -87,8 +87,8 @@ 
 $xoopsTpl->assign('newpost_link', 'viewpost.php?status=new');
 
 if (!empty($GLOBALS['xoopsModuleConfig']['show_jump'])) {
-    require_once __DIR__ . '/include/functions.forum.php';
-    $xoopsTpl->assign('forum_jumpbox', newbbMakeJumpbox());
+	require_once __DIR__ . '/include/functions.forum.php';
+	$xoopsTpl->assign('forum_jumpbox', newbbMakeJumpbox());
 }
 $xoopsTpl->assign('menumode', $menumode);
 $xoopsTpl->assign('menumode_other', $menumode_other);