[BUGFIX] Wrong overloading of AbstractController::redirect() 07/44707/2
authorFrank Nägler <frank.naegler@typo3.org>
Fri, 13 Nov 2015 15:45:35 +0000 (16:45 +0100)
committerTymoteusz Motylewski <t.motylewski@gmail.com>
Sat, 14 Nov 2015 10:39:09 +0000 (11:39 +0100)
This patch add a todo for deprecation in CMS8 and remove the usage
of the wrong overloaded method in the core.

Resolves: #71303
Releases: master
Change-Id: Id42af6b16a382d67313e66e695096669e07d4959
Reviewed-on: https://review.typo3.org/44707
Reviewed-by: Markus Sommer <markussom@posteo.de>
Tested-by: Markus Sommer <markussom@posteo.de>
Reviewed-by: Oliver Eglseder <oliver.eglseder@in2code.de>
Tested-by: Oliver Eglseder <oliver.eglseder@in2code.de>
Reviewed-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Tested-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
typo3/sysext/install/Classes/Controller/BackendModuleController.php

index 72456d6..6f98be5 100644 (file)
@@ -59,7 +59,7 @@ class BackendModuleController extends \TYPO3\CMS\Extbase\Mvc\Controller\ActionCo
     public function indexAction()
     {
         if ($this->enableFileService->checkInstallToolEnableFile()) {
-            $this->redirect('sysext/install/Start/Install.php?install[context]=backend');
+            \TYPO3\CMS\Core\Utility\HttpUtility::redirect('sysext/install/Start/Install.php?install[context]=backend');
         } else {
             $this->forward('showEnableInstallToolButton');
         }
@@ -95,6 +95,8 @@ 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
      */
     protected function redirect($uri)
     {