- Issue created by @poker10
- Status changed to Needs review
over 1 year ago 10:31pm 30 August 2023 - last update
over 1 year ago 2,160 pass - 🇸🇰Slovakia poker10
Uploading 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. - last update
over 1 year ago 2,156 pass - last update
over 1 year ago 2,121 pass - 🇸🇰Slovakia poker10
Adding a tag for the review from another D7 maintainer before commit.
- last update
about 1 year ago 2,163 pass - last update
about 1 year ago 2,163 pass - last update
about 1 year ago 2,163 pass - last update
about 1 year ago 2,163 pass - last update
about 1 year ago 2,163 pass - last update
about 1 year ago 2,124 pass - Status changed to RTBC
about 1 year ago 10:24am 5 December 2023 - last update
about 1 year ago 2,163 pass - Status changed to Fixed
about 1 year ago 11:30am 5 December 2023 - 🇸🇰Slovakia poker10
I have drafted a CR here: https://www.drupal.org/node/3406314 → (it is similar to the D10 CR)
Automatically closed - issue fixed for 2 weeks with no activity.