Bug #78031
closed
ce:menu.directory maximum level does not work properly (has patch)
0%
Description
1- Back-end page tree :
- ###
- ####
- ####
- ####
- ###
- ####
- ####
- ####
- ####
- ####
- ####
- ###
- ####
- ###
2- HTML template
<html xmlns:f="http://typo3.org/ns/TYPO3/CMS/Fluid/ViewHelpers" xmlns:ce="http://typo3.org/ns/TYPO3/CMS/FluidStyledContent/ViewHelpers" data-namespace-typo3-fluid="true">
<ce:menu.directory pageUids="{pageUids}" as="pages" levelAs="level" maximumLevel="2">
<f:if condition="{pages}">
<ul>
<f:for each="{pages}" as="page">
<li>
<f:link.page pageUid="{page.uid}">
<f:if condition="{page.nav_title}">
<f:then>
{page.nav_title}
</f:then>
<f:else>
{page.title}
</f:else>
</f:if>
</f:link.page>
<f:render partial="Menu/Directory" arguments="{pageUids: {0: page.uid}}" />
</li>
</f:for>
</ul>
</f:if>
</ce:menu.directory>
</html>
3- HTML result
<ul>
<li>
###
<ul>
<li>####</li>
<li>####</li>
<li>####</li>
</ul>
</li>
<li>###</li>
<li>###</li>
<li>###</li>
</ul>
4- Expected result
<ul>
<li>
###
<ul>
<li>####</li>
<li>####</li>
<li>####</li>
</ul>
</li>
<li>
###
<ul>
<li>####</li>
<li>####</li>
<li>####</li>
</ul>
</li>
...
...
</ul>
Files
Updated by Cyril Janody about 8 years ago
- File deleted (
DirectoryViewHelper.php.patch)
Updated by Cyril Janody about 8 years ago
- File DirectoryViewHelper.patch DirectoryViewHelper.patch added
- Assignee deleted (
Benni Mack) - PHP Version deleted (
7.0)
Updated by Cyril Janody about 8 years ago
- Subject changed from ce:menu.directory maximum level does not work properly to ce:menu.directory maximum level does not work properly (has patch)
Updated by Cyril Janody about 8 years ago
- Target version set to next-patchlevel
- Complexity set to no-brainer
Updated by Michael Oehlhof about 8 years ago
Hi Cyril,
if you register for Slack at https://forger.typo3.org/slack I can guide you on Slack https://typo3.slack.com to commit your patch by yourself if you like.
Regards
Michael
Updated by Cyril Janody about 8 years ago
Hi Michael,
Good idea, I created my account on slack. What should I do now ?
Thank you for your help.
Sincerely
Cyril
Updated by Riccardo De Contardi over 7 years ago
I think that this should be solved for 8.7-dev, as <ce:menu.directory> is no more used. Can someone confirm?
Updated by Sybille Peters over 6 years ago
Thank you for your report.
Even though it has been some time, would you consider checking if your patch idea is still up to date and upload it to our Gerrit review server?
Someone could do this for you, but I am thinking you might like the opportunity to contribute to TYPO3 yourself.
You can find a description of the TYPO3 contribution workflow here: https://docs.typo3.org/typo3cms/ContributionWorkflowGuide/
Hint: If you get stuck anywhere, ask on Slack in the #typo3-cms-coredev channel.
Also, if your issue is no longer reproducable, please let us know in the slack channel, so that this issue can be closed.
Thank you in advance!
Sybille
Updated by Riccardo De Contardi over 5 years ago
I tried to test this kind of issue on 9.5.4;
as the viewhelper ce:menu.directory> is no more present since 8.7.x, I used instead a content element of type "Sitemap of selected pages" that should be its equivalent, if I am not wrong.
I replicated the page structure in the issue description and added the following TypoScript on my setup:
tt_content.menu_sitemap_pages.dataProcessing.10.levels=2
to limit the number of levels.
The frontend structure seems correct; is it sufficient to consider this issue closed? A different test should be performed?
Updated by Benni Mack over 5 years ago
- Target version changed from next-patchlevel to Candidate for patchlevel
Updated by Stephan Großberndt almost 5 years ago
- Status changed from New to Closed
- Target version deleted (
Candidate for patchlevel)
The DirectoryViewHelper class was moved to compatibility7 in TYPO3 8.7:
Thus TYPO3 v9 does not contain the viewhelper at all, TYPO3 8.7 is security bugfixes only mode.