Uploaded image for project: 'Magnolia Standard Templating Kit (closed)'
  1. Magnolia Standard Templating Kit (closed)
  2. MGNLSTK-1178

Start level of subnavigation not correct for sites with home not on top level

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Critical Critical
    • 2.0.11, 2.5.1
    • 2.0.10
    • None

      With MGNLSTK-949 one can now configure the start level of the sub navigation which is a very cool thing, BUT: If you do not configure a start level and you do not have your home page on the top level, the start level of the sub navigation is wrong.

      Steps to reproduce:
      1. go to demoauthor.magnolia-cms.com
      2. create a new top level node of the type redirect
      3. underneath create a second redirect node
      4. copy the whole demo-project site underneath the second redirect node
      5. open the copy of demo-project and navigate to some sup page
      => The sub navigation starts with "home" (i.e. 2 levels off)!

      The problem lies on line 87 of SiteNavigationModel where the startLevel of the sub navigation is calculated by simply adding 1 to getHorizontalLevel() instead of adding the startLevel of the horizontal navigation (as it was done in 2.0.9).

        Acceptance criteria

              jsimak Jaroslav Simak
              will Will Scheidegger
              Votes:
              1 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved:

                  Estimated:
                  Original Estimate - 1h
                  1h
                  Remaining:
                  Remaining Estimate - 1h
                  1h
                  Logged:
                  Time Spent - Not Specified
                  Not Specified