We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
RoleIds
Security\Context::getRoles() currently returns an array of (expanded) Role instances.
Security\Context::getRoles()
Role
It is used all over the place even though:
To make this a b/c change, I would suggest to introduce the additional methods:
class Context { /** * consider using {@see self::getExpandedRoleIdentifiers()} instead */ public function getRoles(): array { // ... } public function getExpandedRoleIdentifiers(): RoleIdentifiers { // ... } }
Also the RoleIds could have some static constructor like:
final readonly class RoleIds { // ... public static function forAnonymousUser(): self { return new self('Neos.Flow:Everybody', 'Neos.Flow:Anonymous'); } }
to centralize this piece of magic
The text was updated successfully, but these errors were encountered:
RoleIdentifiers
FEATURE: Introduce RoleId and RoleIds value objects
RoleId
1e61872
Resolves: #3414
bwaidelich
Successfully merging a pull request may close this issue.
Security\Context::getRoles()
currently returns an array of (expanded)Role
instances.It is used all over the place even though:
Role
instancesTo make this a b/c change, I would suggest to introduce the additional methods:
Also the
RoleIds
could have some static constructor like:to centralize this piece of magic
The text was updated successfully, but these errors were encountered: