Show errors when ?board= and/or ?topic= are given invalid values

Signed-off-by: Jon Stovell <jonstovell@gmail.com>

Sources/Forum.php

@@ -704,9 +704,18 @@ protected static function findAction(?string $action): string|callable|false
 	{
 		// If no action was supplied, is there an implied action?
 		if (empty($action)) {
-			// Action and board are both empty... BoardIndex!
-			if (empty(Board::$info->id) && empty(Topic::$topic_id)) {
-				// ... unless someone else wants to do something different.
+			// We check $_GET['topic'] and $_GET['board'] because the implied
+			// action depends on what was in the URL, not on whatever values we
+			// calculated for Board::$info->id and Topic::$info->id. This means
+			// that if $_GET['topic'] or $_GET['board'] are set to something
+			// invalid, an error will be shown. This is the correct and intended
+			// behaviour.
+			if (isset($_GET['topic'])) {
+				$action = 'display';
+			} elseif (isset($_GET['board'])) {
+				$action = 'messageindex';
+			} else {
+				// Does a mod want to change the default action?
 				if (!empty(Config::$modSettings['integrate_default_action'])) {
 					$default_action = explode(',', Config::$modSettings['integrate_default_action'])[0];
 
@@ -715,14 +724,6 @@ protected static function findAction(?string $action): string|callable|false
 
 				$action = 'boardindex';
 			}
-			// Topic is empty, and action is empty.... MessageIndex!
-			elseif (empty(Topic::$topic_id)) {
-				$action = 'messageindex';
-			}
-			// Board is not empty... topic is not empty... action is empty... Display!
-			else {
-				$action = 'display';
-			}
 		}
 
 		// Still no valid action?