b13 / menus

TYPO3 Extension for creating fast menus in a fast fashion
GNU General Public License v2.0
54 stars 22 forks source link

State "isCurrentPage" & "isInRootline" missing in 3rd depth level and below #31

Closed rintisch closed 4 years ago

rintisch commented 4 years ago

Having a menu with depth=7 I have the problem that isCurrentPage and isInRootline are not given for pages with depth level 3 and below (so 4th, 5th, ...).

Here is a screenshot of the debug list. As you can see the first and second level do have both attributes but the third does not:

grafik

The TypoScript setup is the following:

 50 = B13\Menus\DataProcessing\TreeMenu
  50 {
    entryPoints = {$plugin.tx_extensionname.settings.mainMenuPid}
    depth = 6
    excludePages = {$plugin.tx_extensionname.settings.pageToExcludePid}
    as = mainnavigation
  }

Is this a bug or is there something that I could have made wrong?

System: TYPO3 9.5.21 EXT:b13/menus v0.2.0

rintisch commented 4 years ago

I found the bug but also saw that is was already fixed 8 months ago with this commit: https://github.com/b13/menus/commit/f7435cfb64b591f90a505796486fc8e44f4aee88

I don't understand why I got an old version. This happened the second time with a repository from B13 loaded with composer (first time it happened to me with the container extension). The version did not change with the commit which I linked above.

Might this be a reason?

Snapshot from composer.json grafik

Snapshot from PageStateMarker.php → This is not the latest version, I just made a composer update but it's not fetching the real latest version. grafik

What am I doing wrong?

rintisch commented 4 years ago

Close this issue because the problem does not exist any longer. But I now have the problem which I described in the previous comment.