mirror of
https://github.com/silverstripe/silverstripe-framework
synced 2024-10-22 14:05:37 +02:00
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
git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/branches/2.4@100770 467b73ca-7a2a-4603-9d3b-597d59a354a9
This commit is contained in:
parent
27f20c5948
commit
035731cd3f
@ -68,19 +68,9 @@ class Group extends DataObject {
|
|||||||
),
|
),
|
||||||
|
|
||||||
$permissionsTab = new Tab(_t('SecurityAdmin.PERMISSIONS', 'Permissions'),
|
$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(
|
new PermissionCheckboxSetField(
|
||||||
'Permissions',
|
'Permissions',
|
||||||
singleton('Permission')->i18n_plural_name(),
|
false,
|
||||||
'Permission',
|
'Permission',
|
||||||
'GroupID',
|
'GroupID',
|
||||||
$this
|
$this
|
||||||
@ -121,6 +111,8 @@ class Group extends DataObject {
|
|||||||
$fields->removeFieldFromTab('Root', 'IP Addresses');
|
$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')) {
|
if(Permission::check('APPLY_ROLES') && DataObject::get('PermissionRole')) {
|
||||||
$fields->addFieldToTab('Root.' . _t('SecurityAdmin.ROLES', 'Roles'),
|
$fields->addFieldToTab('Root.' . _t('SecurityAdmin.ROLES', 'Roles'),
|
||||||
new LiteralField(
|
new LiteralField(
|
||||||
|
Loading…
Reference in New Issue
Block a user