Bug #67415
closedbug with clicmenu when choosing witch option to display
100%
Description
When you personnalise the clickmenu items, you cannot use following options : moreoptions, hide, edit_access, edit_pageproperties.
In 6.2: the extra_page_cm_options extension is buggy
In 7 : the extra_page_cm_options get merged by copy paste, so bug is still present.
the array_splice didn't preseve array keys.
It's easy to fix. I'll try to provide a fix soon
here's the fix for 6.2 : line 106
replace : array_splice($menuItems, $c, 0, $localItems);
by $first = array_slice($menuItems, 0, $c, TRUE);
$last = array_slice($menuItems, $c, NULL, TRUE);
$menuItems = array_merge($first, $localItems, $last);
In 7, it need to be fixed same way, but the code can be cleaned and get rewrited to use same writing style than the rest of the extension and optimise the process.
Updated by Wouter Wolters about 9 years ago
Can you come up with a patch?
Help instructions can be found at http://wiki.typo3.org/CWT
Updated by Benni Mack about 9 years ago
- Target version changed from next-patchlevel to 7.4 (Backend)
- Sprint Focus set to On Location Sprint
Updated by Susanne Moog about 9 years ago
- Target version changed from 7.4 (Backend) to 7.5
Updated by Anja Leichsenring about 9 years ago
- Sprint Focus changed from On Location Sprint to Remote Sprint
Updated by Benni Mack almost 9 years ago
- Target version changed from 7.5 to 7 LTS
Updated by Gerrit Code Review almost 9 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 http://review.typo3.org/43880
Updated by Gerrit Code Review almost 9 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43880
Updated by Gerrit Code Review almost 9 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43880
Updated by Benni Mack almost 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 7e1883264f00aa942534b06859a3c4ae3d14e25e.
Updated by Riccardo De Contardi almost 7 years ago
- Status changed from Resolved to Closed