- Issue created by @jonathanshaw
- πΊπΈUnited States john.oltman
This appears to be the last unassigned issue that is part of π± [META] Access refactoring Active . I'll take it and hopefully we can finish that meta issue off.
- Merge request !96#3494643: Integrate edit state check into access handler β (Merged) created by john.oltman
- π¬π§United Kingdom jonathanshaw Stroud, UK
I wonder whether the operation should be called 'update state' to align with the 'update' operation. Not that it matters at all.
I think probably the RegistrationAccesscontrolHandler should call out to the host handler with an 'edit registrations state' operation in case the host has an opinion as well. I like doing this almost always (even with no *host permission from this module) as it makes it easier for sophisticated access control per host using the group module and other custom use cases.
- πΊπΈUnited States john.oltman
For now I'm going to leave the operation as "edit state" to match the permission - and because I'm a little worried about someone confusing it with the update operation - so having it be distinct might be a positive.
That sounds like a good idea on the host access - will give that a shot.
- πΊπΈUnited States john.oltman
Added draft change record: https://www.drupal.org/node/3500382 β
-
john.oltman β
committed 72ce0363 on 3.3.x
#3494643: Integrate edit state check into access handler
-
john.oltman β
committed 72ce0363 on 3.3.x
Automatically closed - issue fixed for 2 weeks with no activity.