diff --git a/admin/code/SecurityAdmin.php b/admin/code/SecurityAdmin.php index d87831cb9..8b406a62c 100644 --- a/admin/code/SecurityAdmin.php +++ b/admin/code/SecurityAdmin.php @@ -111,7 +111,7 @@ class SecurityAdmin extends LeftAndMain implements PermissionProvider { $memberList->setPermissions(array('edit', 'delete', 'add')); $fields = new FieldList( - new TabSet( + $root = new TabSet( 'Root', new Tab('Members', singleton('Member')->i18n_plural_name(), $memberList, @@ -137,7 +137,9 @@ class SecurityAdmin extends LeftAndMain implements PermissionProvider { // necessary for tree node selection in LeftAndMain.EditForm.js new HiddenField('ID', false, 0) ); - + + $root->setTemplate('CMSTabSet'); + // Add roles editing interface if(Permission::check('APPLY_ROLES')) { $rolesCTF = new ComplexTableField( diff --git a/admin/templates/Includes/LeftAndMain_EditForm.ss b/admin/templates/Includes/LeftAndMain_EditForm.ss index 82602abaa..d1ce603c6 100644 --- a/admin/templates/Includes/LeftAndMain_EditForm.ss +++ b/admin/templates/Includes/LeftAndMain_EditForm.ss @@ -13,8 +13,8 @@
  • $Title
  • <% end_control %> - <% end_with %> + <% end_with %> <% end_if %> @@ -40,7 +40,6 @@ <% end_control %>
    -
    diff --git a/admin/templates/Includes/SecurityAdmin_Content.ss b/admin/templates/Includes/SecurityAdmin_Content.ss index b9317f895..67ef14449 100644 --- a/admin/templates/Includes/SecurityAdmin_Content.ss +++ b/admin/templates/Includes/SecurityAdmin_Content.ss @@ -1,38 +1,76 @@
    - -
    -
    -

    <% _t('SECGROUPS','Security Groups') %>

    +
    +
    +
    +

    <% _t('SECGROUPS','Security Groups') %>

    +
    + + $AddForm + +
    + + +
    + +
    + $SiteTreeAsUL +
    +
    - -
    - -
    - -
    - $AddForm - -
    - - -
    - -
    - $SiteTreeAsUL -
    - + <% with EditForm %> +
    + + <% if IncludeFormTag %> +
    + <% end_if %> + +
    + <% if Fields.hasTabset %> + <% with Fields.fieldByName('Root') %> +
    +
      + <% control Tabs %> +
    • $Title
    • + <% end_control %> +
    +
    + <% end_with %> + <% end_if %>
    -
    - -
    +
    + <% if Message %> +

    $Message

    + <% else %> + + <% end_if %> -
    - $EditForm -
    +
    + <% if Legend %>$Legend<% end_if %> + <% control Fields %> + $FieldHolder + <% end_control %> +
    +
    +
    +
    + <% if Actions %> +
    + <% control Actions %> + $Field + <% end_control %> +
    + <% end_if %> +
    + + <% if IncludeFormTag %> + + <% end_if %> +
    + <% end_with %>
    \ No newline at end of file diff --git a/forms/TabSet.php b/forms/TabSet.php index e7495863a..b722e2e61 100644 --- a/forms/TabSet.php +++ b/forms/TabSet.php @@ -153,5 +153,4 @@ class TabSet extends CompositeField { public function removeByName( $tabName, $dataFieldOnly = false ) { parent::removeByName( $tabName, $dataFieldOnly ); } -} -?> \ No newline at end of file +} \ No newline at end of file diff --git a/view/SSViewer.php b/view/SSViewer.php index 7de22b792..449de81a9 100644 --- a/view/SSViewer.php +++ b/view/SSViewer.php @@ -185,7 +185,6 @@ class SSViewer_DataPresenter extends SSViewer_Scope { * * @see http://doc.silverstripe.org/themes * @see http://doc.silverstripe.org/themes:developing - * * @package sapphire * @subpackage view