- Issue created by @hitfactory
- Open on Drupal.org →Core: 10.2.x + Environment: PHP 8.1 & MySQL 5.7last update
11 months ago Not currently mergeable. - Merge request !4Fix for invalid link_options data when using Menu Attributes module → (Open) created by hitfactory
- 🇳🇿New Zealand hitfactory
hitfactory → changed the visibility of the branch 3417145-menu-link-saved to hidden.
- Status changed to Needs review
11 months ago 10:14am 26 January 2024 - 🇳🇿New Zealand hitfactory
Sorry for the clutter. I couldn't work out how to delete the merge request against the wrong branch and only figured out later I could change the branch without creating a new one.
To add a bit more context, this happens because the Attributes fieldset which displays via the Menu Link Attributes module may not always be visible to the end user if they don't have the ''use menu link attributes' permission.
When a user doesn't have the ''use menu link attributes' permission, then the $menu_link_options variable passed as the first argument to array_merge_recursive in simple_megamenu_menu_link_content_form_submit will be identical to the second argument $data_simple_mega_menu.
- 🇩🇪Germany Anybody Porta Westfalica
We're running into the same issue now!
@hitfactory did you also test your MR works without the Menu Attributes module?
Code-wise I'd say it won't? - 24a9fc26 committed on 2.1.x
Issue #3417145 by anybody, hitfactory, flocondetoile: Menu link saved...
- 24a9fc26 committed on 2.1.x
- Status changed to Fixed
3 months ago 2:42pm 9 September 2024 Automatically closed - issue fixed for 2 weeks with no activity.