Rework valid transitions to respect group scoped system

Created on 29 July 2024, 8 months ago

Problem/Motivation

Lets say, you have 3 states (draft, published, unpublished). Globally, you want role X to use all the three states. But in your group, you want the role to use 2 states e.g. (published, unpublished). Although, the group scoped system allows you do to this, the current implementation of gcontent_moderation doesn't support it...

This is because, in release 2.0.x and further, the module still checks for global transition access even when a group has been identified. My guess for having this in group 1 is the fact that users loss their permissions (granted through the "advanced outsider permissions") when joining a group.

In group 2, new concept for group roles and permission calculation has been introduced. See #3204083: Rework group roles into a scoped system and adjust permission calculation .
This allows us to create scoped group roles and to grant different permissions for each (insider and outsider) roles.

Proposed resolution

When group has been identified, do not check the global transition access BUT check for group access only...

Remaining tasks

Create MR

User interface changes

No

API changes

No

Data model changes

No

Note

Whoever want to use the MR, should explicitly set the transition permissions for both outsider and insider roles....

Feature request
Status

Active

Version

2.0

Component

Code

Created by

🇧🇪Belgium msnassar

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

Merge Requests

Comments & Activities

Production build 0.71.5 2024