[FOLLOWUP][BUGFIX] Wrong overloading of AbstractController::redirect() 16/45216/4
authorMorton Jonuschat <m.jonuschat@mojocode.de>
Fri, 11 Dec 2015 19:47:36 +0000 (20:47 +0100)
committerMarkus Klein <markus.klein@typo3.org>
Sat, 12 Dec 2015 21:27:20 +0000 (22:27 +0100)
This patch adds the missing parameters for the redirect method so that
the method signature matches between BackendModuleController and
AbstractController. As the parameters are all optional this is a
non-breaking change.

Resolves: #71303
Resolves: #72168
Releases: master
Change-Id: I8c7d55d1a1b01e1e7cdc3859e6a99eded5008802
Reviewed-on: https://review.typo3.org/45216
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
typo3/sysext/install/Classes/Controller/BackendModuleController.php

index 6f98be5..f247058 100644 (file)
@@ -95,11 +95,22 @@ class BackendModuleController extends \TYPO3\CMS\Extbase\Mvc\Controller\ActionCo
      * Redirect to specified URI
      *
      * @param string $uri
-     * @todo: deprecate this method in CMS8, it overload the method of AbstractController in wrong way.
-     * @todo: to late to deprecate in CMS7
+     * @param string $_1 Dummy parameter to ensure compatible method signature
+     * @param string $_2 Dummy parameter to ensure compatible method signature
+     * @param array $_3 Dummy parameter to ensure compatible method signature
+     * @param int $_4 Dummy parameter to ensure compatible method signature
+     * @param int $_5 Dummy parameter to ensure compatible method signature
+     * @param int $_6 Dummy parameter to ensure compatible method signature
      */
-    protected function redirect($uri)
-    {
+    protected function redirect(
+        $uri,
+        $_1 = null,
+        $_2 = null,
+        array $_3 = null,
+        $_4 = null,
+        $_5 = 0,
+        $_6 = 303
+    ) {
         \TYPO3\CMS\Core\Utility\HttpUtility::redirect($uri);
     }
 }