Problem/Motivation

In #3249027-50: RoleAccessCheck returns Allowed if user has no role larowlan suggested that checking access should only be done by permission, not role. Therefore, deprecate \Drupal\user\Access\RoleAccessCheck should be deprecated.

. We should never be checking access by role, only by permission.

Steps to reproduce

Proposed resolution

Deprecate RoleAccessCheck

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3616218

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

quietone created an issue.

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

danielveza opened merge request !16695

danielveza’s picture

danielveza
he/him
Location Brisbane, AU
commented

Started the work for this one, ran into a bit of an issue. The Views Role plugin uses this, and I imagine is pretty actively used. Do we want to also deprecate that, or do we want to bring in a views specific version?