Problem/Motivation

On the module settings page (/admin/config/system/menu_per_role), the link in the "Administrator bypass" description is broken.

Steps to reproduce

  1. Go to /admin/config/system/menu_per_role.
  2. Observe the description for "Administrator bypass".
  3. Click the "Account settings" link. It will lead to a 404 page with a malformed URL.

Proposed resolution

In the /src/Form/MenuPerRoleAdminSettings.php, update the buildForm method to point to the correct "Role settings" page and use the correct placeholder.

CommentFileSizeAuthor
#4 screenshot_menu_per_role_settings_page.png331.17 KBsnap_x

Issue fork menu_per_role-3555429

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

grask0 created an issue. See original summary.

grask0 opened merge request !19

grask0’s picture

grask0 commented
Assigned: grask0 » Unassigned
Status: Active » Needs review

Hey, I have provided the fix.

Please review.

snap_x’s picture

snap_x
Location 🇺🇦 Ukraine, Lutsk
commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new screenshot_menu_per_role_settings_page.png331.17 KB

Thanks @grask0
Your fix works fine for me

  • grask0 committed 32ebb9d9 on 8.x-1.x 
    [#3555429] fix: Broken and incorrect link in 'Administrator bypass'...
grask0’s picture

grask0 commented
Status: Reviewed & tested by the community » Fixed

Merged!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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