Problem/Motivation

We use this module in combination with menu_admin_per_menu. We use the menu_admin_per_menu module to simplify the UI for site editors; they can add/reorder/remove menu items, but cannot change menu configuration.

Menu Item Extra adds a "Clear related data" link to menu forms (to make uninstalling easy). We've found this link to be confusing for site editors.

Proposed resolution

Only display the "Clear related data" link to users with the "administer menu" permission.

Remaining tasks

(reviews needed, tests to be written or run, documentation to be written, etc.)

User interface changes

The "Clear related data" link is only displayed to users with the "administer menu" permission.

API changes

None

Data model changes

None

CommentFileSizeAuthor
#2 3037983-2.patch521 bytescasey

Comments

casey created an issue. See original summary.

casey’s picture

casey commented
Status: Active » Needs review
StatusFileSize
new 3037983-2.patch521 bytes
ozin’s picture

ozin
Primary language Ukrainian
commented

Hi @casey

I think will be better to fix permission for the route `entity.menu.clear_extra_data` in the route file, what do you think?

voleger’s picture

voleger
he/his
Primary language Ukrainian
Location Ukraine, Rivne
commented
Issue tags: +DrupalCamp Kyiv 2019
voleger’s picture

voleger
he/his
Primary language Ukrainian
Location Ukraine, Rivne
commented
Issue tags: -DrupalCamp Kyiv 2019 +dckyiv2019

  • ozin committed 3b3f77c on 8.x-2.x authored by casey 
    Issue #3037983 by casey: Add permission to Clear related data
ozin’s picture

ozin
Primary language Ukrainian
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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