Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Access Handler class/annotation to Metadata Display Entities #402

Open
DiegoPino opened this issue Jan 10, 2024 · 0 comments
Open

Add Access Handler class/annotation to Metadata Display Entities #402

DiegoPino opened this issue Jan 10, 2024 · 0 comments
Assignees
Labels
Custom Entities Drupal 10 Upgrade economy enhancement New feature or request Twig Twig template processing
Milestone

Comments

@DiegoPino
Copy link
Member

What?

We are right now inheriting the default access handler for generic Content Entities. There is nothing generic about Metadata Displays and we should provide for better future-proof compatibility/needs a custom Access handler class. A good example is \Drupal\node\NodeAccessControlHandler

This also allows us to implement more fine grained permissions for revisions and unrelated, maybe also handle the strange use case that a metadata display was created by a user, and then the user is deleted.

thx

@DiegoPino DiegoPino added enhancement New feature or request Twig Twig template processing Custom Entities Drupal 10 Upgrade economy labels Jan 10, 2024
@DiegoPino DiegoPino added this to the 1.4.0 milestone Jan 10, 2024
@DiegoPino DiegoPino self-assigned this Jan 10, 2024
@DiegoPino DiegoPino modified the milestones: 1.4.0, 1.5.0 Aug 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Custom Entities Drupal 10 Upgrade economy enhancement New feature or request Twig Twig template processing
Projects
None yet
Development

No branches or pull requests

1 participant