Project

General

Profile

Actions

Bug #87182

closed

ActionMenuItemViewHelper issue with selected-attribute

Added by Kevin Ditscheid almost 6 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2018-12-17
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
9
PHP Version:
7.2
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

The ActionMenuItemViewHelper is not resetting the selected-attribute for the rendered items, thus all options that follow a selected menu item, will also get selected.
It also does not respect the given arguments of the menu item. It also selects menu items with the same controller => action combination, but with different arguments.


Files

Actions #1

Updated by Gerrit Code Review almost 6 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/59176

Actions #2

Updated by Gerrit Code Review almost 6 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/59176

Actions #3

Updated by Gerrit Code Review almost 6 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/59176

Actions #4

Updated by Gerrit Code Review almost 6 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/59176

Actions #5

Updated by Gerrit Code Review almost 6 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/59176

Actions #6

Updated by Gerrit Code Review almost 6 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/59176

Actions #7

Updated by Stefanos Karasavvidis over 5 years ago

to reproduce this, you can use the attached extension. It has a simple backend module (under admin tools) that uses the ActionMenuItemViewHelper for 4 different actions:
index
list1
list2 with argument 1
list2 with argument 2

Using actions index and list1 leaves the dropdown correctly selected to the action that was used.

Using action list2 leaves the dropdown always to the last action in the list. The markup that is generated has the "selected" attribute in both options.

Actions #8

Updated by Stefanos Karasavvidis over 5 years ago

corrected minor typos in test extension

Actions #9

Updated by Gerrit Code Review over 4 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/59176

Actions #10

Updated by Gerrit Code Review over 4 years ago

Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63582

Actions #11

Updated by Gerrit Code Review over 4 years ago

Patch set 2 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63582

Actions #12

Updated by Kevin Ditscheid over 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #13

Updated by Benni Mack over 4 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF