When importing (or synchronizing) configurations this module is causing issues, which I assume occurs due to missing configuration dependencies.

Current shortcutperrole.settings:

role:
  anonymous: default
  authenticated: default
  role1: other
  role2: other
  role3: default
  administrator: default

Expected:

role:
  anonymous: default
  authenticated: default
  role1: other
  role2: other
  role3: default
  administrator: default
dependencies:
  config:
    - shortcut.set.default
    - shortcut.set.other
  module:
    - shortcut

The expected code is based on other configs and is just to give an idea of the expected config. The actual output might differ.

Issue fork shortcutperrole-2921143

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

Neograph734 created an issue. See original summary.

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented
Version: 8.x-1.x-dev » 2.1.x-dev

anton4uk made their first commit to this issue’s fork.

anton4uk opened merge request !7

anton4uk’s picture

anton4uk commented
Status: Active » Needs review

I've investigated this issue, and the dependencies as described in the ticket cannot be implemented for simple configs like shortcutperrole. settings. You can't find any example in the core. To do that, you need to develop the whole mechanism for handling such dependencies, which doesn't make sense. It works for config entities, but not for simple configs. Anyway, I've implemented functionality to remove dependent records in config when a user role is deleted https://www.drupal.org/project/shortcutperrole/issues/3620897. The second part of cleaning dependencies I've added here. When you remove the shortcut itself, the shortcutperrole. settings will be updated as well.

mkdok’s picture

mkdok commented
Status: Needs review » Reviewed & tested by the community

LGTM

mkdok’s picture

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

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.