Remove dead branch in RequestHandler::redirect()
authorTim Düsterhus <duesterhus@woltlab.com>
Fri, 20 May 2022 12:36:06 +0000 (14:36 +0200)
committerTim Düsterhus <duesterhus@woltlab.com>
Fri, 20 May 2022 12:36:06 +0000 (14:36 +0200)
The `redirect()` method is only called in a single location where `$controller`
will also be set to a string.

wcfsetup/install/files/lib/system/request/RequestHandler.class.php

index 2ee37d02b3341c5d89bfa0570fd0b0c27a490159..109f9824658f915a7b16a1d7122914b0300e66de 100644 (file)
@@ -241,15 +241,11 @@ final class RequestHandler extends SingletonFactory
      * Redirects to the actual URL, e.g. controller has been aliased or mistyped (boardlist instead of board-list).
      *
      * @param string[] $routeData
-     * @param string $application
-     * @param string $controller
      */
-    protected function redirect(array $routeData, $application, $controller = null)
+    protected function redirect(array $routeData, string $application, string $controller)
     {
         $routeData['application'] = $application;
-        if ($controller !== null) {
-            $routeData['controller'] = $controller;
-        }
+        $routeData['controller'] = $controller;
 
         // append the remaining query parameters
         foreach ($_GET as $key => $value) {