[BUGFIX] Set current report as selected in ext:reports
authorGeorg Ringer <georg.ringer@gmail.com>
Sat, 22 Sep 2012 16:15:47 +0000 (18:15 +0200)
committerSusanne Moog <typo3@susannemoog.de>
Sat, 22 Sep 2012 17:17:03 +0000 (19:17 +0200)
there can be usecases when not the correct option is set to
selected="selected".

example:
select "installed service"
switch to a different module
switch back to reports module
content shows "installed services" but select box is wrong

Change-Id: I9166ab8489a246a30673c7936214a2d8e0495fbc
Resolves: #41258
Releases: 6.0
Reviewed-on: http://review.typo3.org/14919
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
typo3/sysext/reports/Classes/ViewHelpers/ActionMenuItemViewHelper.php

index 0d9b488..e9e3bc3 100644 (file)
@@ -56,6 +56,8 @@ class ActionMenuItemViewHelper extends \TYPO3\CMS\Fluid\Core\ViewHelper\Abstract
                $currentArguments = $currentRequest->getArguments();
                unset($currentArguments['action']);
                unset($currentArguments['controller']);
+               unset($currentArguments['redirect']);
+
                if ($action === $currentAction && $controller === $currentController && $currentArguments === $arguments) {
                        $this->tag->addAttribute('selected', 'selected');
                }