This module breaks the menu system with used with Menu Admin Per Menu

Created on 4 November 2019, almost 5 years ago
Updated 26 April 2023, over 1 year ago

I am finding that Menu Link module breaks the menu system when Menu Admin Per Menu when it has been patched as per this issue

🐛 Bug report
Status

Active

Version

2.0

Component

Code

Created by

🇬🇧United Kingdom mrpauldriver

Live updates comments and jobs are added and updated live.
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.

  • 🇺🇸United States pookmish

    Reopening because when updating the menu_link_weight module, it requires this module. Then it breaks the access to the menu fields on the node form. Since this module provides no permissions, it is breaking the intended functionality of "Menu Admin Per Menu" module.

    I don't want to provide the "Administer menus and menu links" permissions because that grants access to menus they shouldn't have access to.

  • Open on Drupal.org →
    Core: 10.0.7 + Environment: PHP 8.1 & MySQL 8
    last update over 1 year ago
    Waiting for branch to pass
  • @pookmish opened merge request.
  • Status changed to Needs review over 1 year ago
  • 🇫🇮Finland iSampo

    Confirming that the MR does fix the menu link field being inaccessible.

    Attached is a patch with the current MR changes, for static composer patch.

  • 🇫🇮Finland iSampo

    Actually with some further testing the Menu Admin Per Menu perms aren't fully respected, as all of the available menu permissions are being added into the array.

    So, if the user has perms by MAPM to only administer the "Main" menu, but the Menu field configuration allows placing items into "Footer" and "User menu" as well, the latter ones will be shown to the user too. So this does fix the issue with the access to the field being FALSE, but the individual menus inside the field aren't being limited.

Production build 0.71.5 2024