Menu depth setting does not reflect the actual menu depth

Created on 16 October 2019, over 4 years ago
Updated 10 January 2024, 6 months ago

Overview of Issue
The menu depth setting in taxonomy_menu does not always reflect the actual menu depth in the menu items. It is possible to engineer two situations in which the depth is deeper than the setting in taxonomy_menu.

The menu depth setting should always be an upper limit for the menu items.

Steps to Reproduce (scenario i)
1) Create a taxonomy with a few items making sure at least one item has 3 levels.
2) Create the menu_taxonomy and set the depth to 3
3) Note that the menu has 3 levels (admin/structure/menu/manage/my-menu)
4) Now change the menu_taxonomy depth to 2

In this scenario the final menu should only have a depth of 2.

Steps to Reproduce (scenario ii)
1) Create a taxonomy with a few item making sure at least one item has 2 levels.
2) Create the menu_taxonomy and set the depth to 2
3) The menu will have 2 levels (admin/structure/menu/manage/my-menu) as follows

-- Term A
-- Term B
--- Term B_1
--- Term B_2
-- Term C

4) In the taxonomy select a term which has at least 2 levels and move this term to the child of term
5) The menu has 3 levels (admin/structure/menu/manage/my-menu) as follows

-- Term A
--- Term B
---- Term B_1
---- Term B_2
-- Term C

In this scenario the final menu should only have a depth of 2. Furthermore it's not possible to remove the 3rd level.

πŸ› Bug report
Status

Closed: outdated

Version

3.0

Component

Code

Created by

πŸ‡¬πŸ‡§United Kingdom sittard

Live updates comments and jobs are added and updated live.
  • Needs tests

    The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡ΊπŸ‡¦Ukraine Bobik

    Hi @sittard
    Is this issue actually because I can't reproduce it?

  • πŸ‡¬πŸ‡§United Kingdom sittard

    It was an issue in a project, which we are no longer maintaining. Been a while since this was reported so I would assume the issue was with a previous version of taxonomy_menu (or Drupal) and is now fixed.

    Thanks.

  • Status changed to Closed: outdated 6 months ago
  • πŸ‡ΊπŸ‡¦Ukraine Bobik

    Ok. I thing the need to close the issue.

Production build 0.69.0 2024