BUGFIX Only show "Roles" tab in admin/security if user has APPLY_ROLES permissions (fixes #5258)

git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/cms/branches/2.4@101719 467b73ca-7a2a-4603-9d3b-597d59a354a9
This commit is contained in:
Ingo Schommer 2010-03-25 22:44:33 +00:00 committed by Sam Minnee
parent fb16cc63f6
commit e507f9488b

View File

@ -81,20 +81,21 @@ class SecurityAdmin extends LeftAndMain implements PermissionProvider {
) )
) )
); );
if(Permission::check('APPLY_ROLES')) {
$fields->addFieldToTab( $fields->addFieldToTab(
'Root.Roles', 'Root.Roles',
new LiteralField( new LiteralField(
'RolesAddEditLink', 'RolesAddEditLink',
sprintf( sprintf(
'<p class="add-role"><a href="%s">%s</a></p>', '<p class="add-role"><a href="%s">%s</a></p>',
$this->Link('show/root'), $this->Link('show/root'),
// TODO This should include #Root_Roles to switch directly to the tab, // TODO This should include #Root_Roles to switch directly to the tab,
// but tabstrip.js doesn't display tabs when directly adressed through a URL pragma // but tabstrip.js doesn't display tabs when directly adressed through a URL pragma
_t('Group.RolesAddEditLink', 'Add/edit roles') _t('Group.RolesAddEditLink', 'Add/edit roles')
)
) )
) );
); }
} }
$actions = new FieldSet( $actions = new FieldSet(