- Issue created by @jonathanshaw
- πΊπΈUnited States john.oltman
I'm unclear on this, as it is possible for someone to not have access to settings, broadcast, or view registrations, and still have access to the manage registrations tab. In this case it shows a summary. So I'm not certain of the changes this issue could make. As for the operation name "broadcast" this would seem to imply the "manage settings" (does that exist?) operation would become just "settings" which doesn't seem right.
At a minimum, I think we need a list of the currently accepted operation names in a comment somewhere in RegistrationHostAccessControlHandler.
- π¬π§United Kingdom jonathanshaw Stroud, UK
it is possible for someone to not have access to settings, broadcast, or view registrations, and still have access to the manage registrations tab. In this case it shows a summary.
Yes. Perhaps you're right that for BC we should keep an operation just for this. It's slightly annoying as it seems a slightly trivial thing to have an operation for.
I lean towards thinking that the operation should be called "view registrations summary" as that is what the operation is actually about. It doesn't really allow you to "manage registrations" in any obvious sense of what it might mean to "manage" a registration; it isn't even considered when considering access to individual registrations, unlike 'view registrations', update registrations', 'administer registrations' etc.
As for the operation name "broadcast" this would seem to imply the "manage settings" (does that exist?) operation would become just "settings" which doesn't seem right.
I considered that but couldn't think of a good single word verb for "manage settings". "configure" isn't awful but has a somewhat different meaning in Drupal.
But even if we have to leave "manage settings" as two words, that doesn't mean "broadcast" isn't better than "manage broadcast" for the operation. But I accept that it might well not be worth changing.