Hi Guys,
Ok, you've pursuaded me that it is a usability issue that needs to be addressed in the UI. My resistance was mainly based on I do not want to change the way security is enforced and I do not want to update role permissions in the database during upgrades, but after thinking about it I beleive I can achieve the needed changes in the UI without doing either of those things.
Just to correct something you said, checking the Administrators box doesn't really "lock the page down" to Administrators only, because you can still check the box for other user roles to allow them rights in addition to Administrators. It really just locks out Content Administrators from editing.
Acutally that is not quite right, but it only makes it more clear that it is not easy to understand and needs improvement for usability.
The way it works is if a page or feature instance permission is set to Administrators and no other roles then only administrators have access. Yes it locks out Content Administrators but it locks out everyone else too.
If no roles are checked then Administrators and Content Administrators have access.
If any roles other than Administrators are checked (even if Administrators is also checked) then, access is available to Administrators, Content Administrators, and the other selected roles.
Basically Administrators and Content Administrators are not subject to role permissions, but the special case is designed in order to make it possible to have some content that is only available to Administrators. So in that case Content Administrators are subject to that special case rule where if Administrators is the one and only allowed role then only Administrators get access.
So I think from the UI we will need 2 radio buttons above the roles checkboxes and I will remove Administrators from the checkbox list. The radio buttons will be like:
1. Only Administrators are allowed
2. Administrators, Content Administrators, and roles selected below are allowed.
If the first radio button is chosen then I will set the role to Administrators only on postback, else it will be based on the checkbox list.
Best,
Joe