Problem/Motivation

Drupal core 9.3.x added the ability for permissions to have a dependencies so that roles can be updated correctly when configuration is deleted.

Proposed resolution

Add bundle dependency information to \Drupal\access_unpublished\AccessUnpublishedPermissions::permissions(). Might be able to leverage \Drupal\Core\Entity\BundlePermissionHandlerTrait to do some of this for us.

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#4 3276604-4.patch516 byteselber

Issue fork access_unpublished-3276604

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

alexpott created an issue. See original summary.

elber’s picture

elber
Primary language Portuguese, Brazil
Location Brazil
commented
Assigned: Unassigned » elber
elber’s picture

elber
Primary language Portuguese, Brazil
Location Brazil
commented

Hi @alexpott could you please be more specific? because I didn't see a method or class for to implement this dependency..

elber’s picture

elber
Primary language Portuguese, Brazil
Location Brazil
commented
Assigned: elber » Unassigned
Status: Active » Needs review
StatusFileSize
new 3276604-4.patch516 bytes

Hi @alexpott, I made my patch but I don't now if a I understood your desire. Please revise...

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented

@elber thanks for working on this - have a read of https://www.drupal.org/node/3055548 - we need to add new information to the permissions created by \Drupal\access_unpublished\AccessUnpublishedPermissions::permissions() so that Drupal can add the correct dependencies to the roles that are assigned one of these permissions.

generalredneck’s picture

generalredneck
him/his
Primary language English
Location Texas, USA 🇺🇸
commented
Status: Needs review » Needs work

This sounds important. Bumping it up

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

mably opened merge request !41

mably’s picture

mably commented
Status: Needs work » Needs review

MR created with some kernel tests.

  • mably committed 915c0980 on 8.x-1.x 
    feat: #3276604 Add dependencies to permissions

By: alexpott
By: elber...
mably’s picture

mably commented
Status: Needs review » 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.

Status: Fixed » Closed (fixed)

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