Contrib.social

Allow term merge access when user has administer taxonomy permission

Open on Drupal.org →
Created on 9 March 2023, almost 2 years ago
Updated 21 September 2023, over 1 year ago

Problem/Motivation

Currently `MergeAccess` checks for specific vocabulary term edit permissions, which is fine becaus eit allows for fine grained control in addition to the the 'merge terms' permission.

However, when this access check denies access to users who only have the 'administer taxonomy' access (which allows editting of all terms in all vocabularies), and is a common permission for higher roles.

Steps to reproduce

Create a user with permissions: "merge terms" and "administer taxonomy".
Verify that you do not see the Merge tab on any vocabulary overview.

Proposed resolution

Add an `orIF` for 'administer taxonomy' in the access check.

Remaining tasks

Patch. Review. Test.

User interface changes

None.

API changes

None.

Data model changes

None.

✨ Feature request
Status

Fixed

Version

2.0

Component

Code

Created by

🇳🇱Netherlands askibinski

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

Comments & Activities

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024