forum listings are now slightly happier with php 8

Author: Xkeeper0

forum.php

@@ -1,7 +1,12 @@
 <?php
+
 	require_once 'lib/function.php';
-	$id = intval($_GET['id']);
-	$user = intval($_GET['user']);
+
+	$id		= intval($_GET['id'] ?? null);
+	$user	= intval($_GET['user'] ?? null);
+	$thread	= intval($_GET['thread'] ?? null);
+	$fav	= intval($_GET['fav'] ?? null);
+	$act	= $_GET['act'] ?? null;
 
 	if ($log)
 		$postread = $sql->getresultsbykey("SELECT forum,readdate FROM forumread WHERE user=$loguserid", 'forum', 'readdate');
@@ -101,14 +106,14 @@
 	$hotcount = $sql->resultq('SELECT hotcount FROM misc',0,0);
 	if ($hotcount <= 0) $hotcount = 0xFFFF;
 
-	$ppp = (($_GET['ppp']) ? intval($_GET['ppp']) : (($log) ? $loguser['postsperpage'] : 20));
-	$ppp = max(min($ppp, 500), 1);
+	$ppp	= intval($_GET['ppp'] ?? ($loguser['postsperpage'] ?? 20));
+	$ppp	= max(min($ppp, 500), 1);
 
-	$tpp = (($_GET['tpp']) ? intval($_GET['tpp']) : (($log) ? $loguser['threadsperpage'] : 50));
-	$tpp = max(min($tpp, 500), 1);
+	$tpp	= intval($_GET['tpp'] ?? ($loguser['threadsperpage'] ?? 50));
+	$tpp	= max(min($tpp, 500), 1);
 
-	$page = intval($_GET['page']);
-    $min = $page*$tpp;
+	$page	= intval($_GET['page'] ?? 0);
+	$min	= $page * $tpp;
 
 	$newthreadbar = $forumlist = '';
 	if ($id) {
@@ -134,7 +139,7 @@
 	$forumpagelinks = '';
 	if($threadcount > $tpp){
 		$query = ($id ? "id=$id" : ($user ? "user=$user" : "fav=1"));
-		if ($_GET['tpp']) $query .= "&tpp=$tpp";
+		if ($_GET['tpp'] ?? null) $query .= "&tpp=$tpp";
 
 		$forumpagelinks = "<table width=100%><tr>
 			<td align=left class='fonts'>Pages:";
@@ -241,7 +246,7 @@
 		}
 
 		// Disabled polls
-		if ($forum['pollstyle'] == -2)
+		if (($forum['pollstyle'] ?? 0) == -2)
 			$thread['poll'] = 0;
 
 		$new          = "&nbsp;";

lib/function.php

@@ -591,6 +591,7 @@ function doforumlist($id){
 
 	$forum1= $sql->query("SELECT id,title,catid FROM forums WHERE (minpower<=$power OR minpower<=0) AND `hidden` = '0' AND `id` != '0' OR `id` = '$id' ORDER BY forder") or print mysql_error();
 	while($forum=$sql->fetch($forum1)) {
+		if (!isset($fjump[$forum['catid']])) $fjump[$forum['catid']] = "";
 		$fjump[$forum['catid']]	.="<option value=forum.php?id=$forum[id]".($forum['id']==$id?' selected':'').">$forum[title]</option>";
 	}
 
@@ -777,9 +778,12 @@ function getuserlink(&$u, $substitutions = null, $urlclass = '') {
 	if ($substitutions)
 		$fn = array_merge($fn, $substitutions);
 
-	$akafield = htmlspecialchars($u[$fn['aka']], ENT_QUOTES);
-	$alsoKnownAs = (($u[$fn['aka']] && $u[$fn['aka']] != $u[$fn['name']])
-		? " title='Also known as: {$akafield}'" : '');
+	$akafield = $alsoKnownAs = "";
+	if ($u[$fn['aka']] ?? false) {
+		$akafield = htmlspecialchars($u[$fn['aka']], ENT_QUOTES);
+		$alsoKnownAs = (($u[$fn['aka']] && $u[$fn['aka']] != $u[$fn['name']])
+			? " title='Also known as: {$akafield}'" : '');
+	}
 
 	$u[$fn['name']] = htmlspecialchars($u[$fn['name']], ENT_QUOTES);
 
@@ -934,8 +938,11 @@ function fonlineusers($id){
 	}
 	$p = ($numon ? ':' : '.');
 	$s = ($numon != 1 ? 's' : '');
+	$guests = "";
 	$numguests = $sql->resultq("SELECT count(*) AS n FROM guests WHERE date>$onlinetime AND lastforum=$id",0,0);
-	if($numguests) $guests="| $numguests guest".($numguests>1?'s':'');
+	if ($numguests) {
+		$guests = "| $numguests guest".($numguests>1?'s':'');
+	}
 	return "$numon user$s currently in $forumname$p $onlineusers $guests";
 }
 

newreply.php

@@ -240,7 +240,7 @@
 				$userlogin
 				<tr>
 					$tccell1 style='width: 150px; max-width: 150px;'><strong>Reply:</strong></td>
-					$tccell2l>$txta=message class='newposttextbox'>". htmlspecialchars($message, ENT_QUOTES) ."</TEXTAREA></td>
+					$tccell2l>$txta=message class='newposttextbox' autofocus>". htmlspecialchars($message, ENT_QUOTES) ."</TEXTAREA></td>
 				<tr>
 					$tccell1>&nbsp;</td>$tccell2l>
 					$inph=action VALUE=postreply>