Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
menu system
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
30 Aug 2023 at 22:26 UTC
Updated:
19 Dec 2023 at 18:09 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
poker10 commentedUploading a backport of the D10 patch. I think this should be pretty safe, because when you click "add link" in a specific menu, you want to add menu link to that specific menu, so you do not need other menus displayed. Other use-cases (like editing the menu item, or using vertical tabs in node to set a menu item) are not affected by the patch.
I had to duplicate the
assertNoOption()andassertOption()functions in tests, which is not ideal. But not sure if we are going to refactor the original ones (as proposed in the @todo) until the D7 EOL, so this is probably not a big issue and should not block this issue.Comment #3
poker10 commentedAdding a tag for the review from another D7 maintainer before commit.
Comment #4
fabianx commentedRTBC! Great fix!
Comment #6
mcdruid commentedThanks!
Comment #7
poker10 commentedI have drafted a CR here: https://www.drupal.org/node/3406314 (it is similar to the D10 CR)