Problem/Motivation

I updated a site to Drupal 11.3, and on cache clear/taxonomy menu configuration save all of the taxonomy menu items moved out from the configured parent menu item into the root of the menu.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

karlshea created an issue. See original summary.

karlshea’s picture

Status: Active » Needs review
dark05’s picture

Hi,
I'm experiencing the exact same issue after upgrading taxonomy_menu from 3.7 to 3.8 on Drupal 11. All taxonomy menu items are placed at the root of the menu instead of under their configured parent menu item.
After investigation, the problem is in the menu_tree table: the parent column is empty for all taxonomy menu plugin links, even when a menu_parent is correctly configured.
Would it be possible to provide a patch or merge request to fix this? In the meantime I had to revert to 3.7.

deg’s picture

I am experiencing the same issue on two different sites as well.

deasly’s picture

Current Merge Request !23 works for me in 11.3.10 as well as a 10.6.9 site. I didn't try the unit test. But good to include one for a regression like this. Only issue is if you already pushed it up on a site you have to re-save your taxonomy menu entities to re-flow them where they should be on each environment. Might want to include and update hook that rebuilds the taxonomy menus for each type so that doesn't need to be done.

znerol’s picture

Status: Needs review » Reviewed & tested by the community

MR works fine for us too.

jonhattan’s picture

Priority: Normal » Critical
jonhattan’s picture

Title: Items moved out of selected parent menu item on save on Drupal 11.3 » Items moved out of selected parent menu item on 3.8 version
damien laguerre’s picture

Sometimes a single character here—“!”—makes all the difference.

Tested and working!
Thanks for the fix.

yan’s picture

I had the same problem and I can confirm that the fix in the MR solves it.

anybody’s picture

Same issue here, this is entirely broken! Any maintainer? Hard to see that critical issue being unresolved for > 6 months! ;)

damienmckenna’s picture

damienmckenna’s picture

I can !not believe we let this one slip in.

Thank you for the fix.

damienmckenna’s picture

Status: Reviewed & tested by the community » Fixed

Committed. Thanks everyone.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

vladimiraus’s picture

Thanks everyone for looking into it and quick fix. 💪

anybody’s picture

Thank YOU! Working correctly again now! (You may have to resave the affected taxonomy menu settings)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.