Permalink
Browse files

MINOR Removed "only advanced users" notice in Group->getCMSFields() -…

… this field is now sufficiently useable for all admins with access to admin/security without knowing about permission codes (from r100770)

git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/trunk@105646 467b73ca-7a2a-4603-9d3b-597d59a354a9
  • Loading branch information...
1 parent 5c96533 commit 787a7d90a1931d151790689863864878f3d819d5 @chillu chillu committed May 25, 2010
Showing with 3 additions and 11 deletions.
  1. +3 −11 security/Group.php
View
@@ -68,19 +68,9 @@ public function getCMSFields() {
),
$permissionsTab = new Tab(_t('SecurityAdmin.PERMISSIONS', 'Permissions'),
- new LiteralField(
- "",
- "<p>" .
- _t('SecurityAdmin.ADVANCEDONLY',
- "This section is for advanced users only.
- See <a href=\"http://doc.silverstripe.com/doku.php?id=permissions:codes\" target=\"_blank\">this page</a>
- for more information."
- ) .
- "</p>"
- ),
new PermissionCheckboxSetField(
'Permissions',
- singleton('Permission')->i18n_plural_name(),
+ false,
'Permission',
'GroupID',
$this
@@ -121,6 +111,8 @@ public function getCMSFields() {
$fields->removeFieldFromTab('Root', 'IP Addresses');
}
+ // Only show the "Roles" tab if permissions are granted to edit them,
+ // and at least one role exists
if(Permission::check('APPLY_ROLES') && DataObject::get('PermissionRole')) {
$fields->addFieldToTab('Root.' . _t('SecurityAdmin.ROLES', 'Roles'),
new LiteralField(

0 comments on commit 787a7d9

Please sign in to comment.