Can't access config settings for non-admin roles

Created on 27 February 2023, over 1 year ago
Updated 28 February 2023, over 1 year ago

Problem/Motivation

No direct menu link under Configuration for admin permission to work.

Steps to reproduce

I assigned the "Administer Geoblock" to a non-admin user. Since there is no direct menu link under /admin/config, this role has no access to administer the module.

Proposed resolution

There should be a direct menu link under Configuration.

πŸ› Bug report
Status

Closed: won't fix

Version

1.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States JCL324 Portland, OR

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @JCL324
  • Assigned to elber
  • πŸ‡§πŸ‡·Brazil elber Brazil

    Hi I will try to work on it.

  • Issue was unassigned.
  • Status changed to Postponed over 1 year ago
  • πŸ‡§πŸ‡·Brazil elber Brazil

    Hi this needs to be postponed because when I try to acess the config form using the route /admin/config/geoblock I have this error: "Error: Class "League\ISO3166\ISO3166" not found in Drupal\geoblock\Form\SettingsForm->getCountryCodeOptions() (line 163 of modules/contrib/geoblock/src/Form/SettingsForm.php)."

    I wil open another issue to fix the error.

  • Status changed to Closed: won't fix over 1 year ago
  • πŸ‡ΊπŸ‡ΈUnited States clayfreeman Paragould, AR

    This project uses a very strict interpretation of Drupal's backward compatibility policy β†’ which prevents us from leveraging Drupal's administration menu out of the box. If you wish to add an item to Drupal's administration menu, you can still do so manually by navigating to Structure > Menus > Administration > Add link and using the following path: /admin/config/geoblock.

  • πŸ‡ΊπŸ‡ΈUnited States JCL324 Portland, OR

    @clayfreeman, please do tell me how this violates the policy. Contrib modules add links all the time! It could easily go under admin/config/development or admin/config/system.

  • πŸ‡ΊπŸ‡ΈUnited States clayfreeman Paragould, AR

    Defining a menu link in the administration menu would require this module to establish a potentially disruptive, implicit dependency on Drupal's current menu structure -- in this case, the plugin ID of a given parent menu item provided by Drupal core. While the backward compatibility policy does state that "plugin IDs [...] can be relied upon," I've chosen to avoid this implicit dependency.

    This module instead leverages the configure key in its *.info.yml, which adds a "Configure" link to the module's description on the "Extend" page. Should Drupal choose to forgo this concept in the future, there is no potentially disruptive dependency relationship to worry about.

    Reliability is a primary priority for this module. I may be paranoid in making this decision, but given that unintended side effects could result in an inaccessible site, I want to reduce the incidence of these types of scenarios as much as possible.

    All that being said, end users who need a menu link can still add one manually.

Production build 0.69.0 2024