From 3046dc949039a847a41f838d887afc63a72c2b78 Mon Sep 17 00:00:00 2001
From: jensp <jensp@8810af33-2d31-482b-a856-94f89814c4df>
Date: Fri, 2 Mar 2018 10:01:17 +0000
Subject: [PATCH] CCM NG: Improvments for permissions management including an
 authoring step for managing the permissions for a single item

git-svn-id: https://svn.libreccm.org/ccm/ccm_ng@5322 8810af33-2d31-482b-a856-94f89814c4df
---
 .../ui/permissions/CMSPermissionsPane.java    |  36 ++-
 .../permissions/CMSPermissionsTableModel.java |  35 +-
 .../ui/permissions/ItemPermissionsStep.java   | 303 +++++++++---------
 .../contentsection/ContentSectionConfig.java  |   2 +-
 .../com/arsdigita/ui/admin/AdminServlet.java  |   8 +-
 .../arsdigita/ui/admin/WorkflowAdminTab.java  |  38 ---
 .../usersgroupsroles/groups/GroupsTable.java  |   4 +-
 7 files changed, 215 insertions(+), 211 deletions(-)
 delete mode 100644 ccm-core/src/main/java/com/arsdigita/ui/admin/WorkflowAdminTab.java

diff --git a/ccm-cms/src/main/java/com/arsdigita/cms/ui/permissions/CMSPermissionsPane.java b/ccm-cms/src/main/java/com/arsdigita/cms/ui/permissions/CMSPermissionsPane.java
index f239d797c..b140a2a6d 100755
--- a/ccm-cms/src/main/java/com/arsdigita/cms/ui/permissions/CMSPermissionsPane.java
+++ b/ccm-cms/src/main/java/com/arsdigita/cms/ui/permissions/CMSPermissionsPane.java
@@ -143,6 +143,26 @@ public class CMSPermissionsPane extends SimpleContainer implements Resettable,
         this.selectionModel = selectionModel;
         this.privilegeNameMap = privilegeNameMap;
     }
+    
+    public CMSPermissionsPane(
+        final Class<?> privilegesClass,
+        final CcmObjectSelectionModel<CcmObject> selectionModel) {
+        
+        final CdiUtil cdiUtil = CdiUtil.createCdiUtil();
+        final PermissionManager permissionManager = cdiUtil.findBean(
+            PermissionManager.class);
+        final List<String> privilegesFromClass = permissionManager
+            .listDefiniedPrivileges(privilegesClass);
+        
+        final Map<String, String> nameMap = new HashMap<>();
+        for(final String privilege: privilegesFromClass) {
+            nameMap.put(privilege, privilege);
+        }
+        
+        this.privileges = privilegesFromClass.toArray(new String[]{});
+        this.selectionModel = selectionModel;
+        this.privilegeNameMap = nameMap;
+    }
 
     /**
      * Overwrite this method to construct your default Permissions Pane with the
@@ -253,9 +273,10 @@ public class CMSPermissionsPane extends SimpleContainer implements Resettable,
         final CdiUtil cdiUtil = CdiUtil.createCdiUtil();
         final PermissionManager permissionManager = cdiUtil.findBean(
             PermissionManager.class);
-        final List<String> privileges = permissionManager
-            .listDefiniedPrivileges(ItemPrivileges.class);
-        final List<Label> headerLabels = privileges.stream()
+//        final List<String> privileges = permissionManager
+//            .listDefiniedPrivileges(ItemPrivileges.class);
+//        final List<Label> headerLabels = privileges.stream()
+        final List<Label> headerLabels = Arrays.stream(privileges)
             .map(privilege -> generatePrivilegeColumnHeader(privilege))
             .collect(Collectors.toList());
         headerLabels.add(0,
@@ -269,7 +290,8 @@ public class CMSPermissionsPane extends SimpleContainer implements Resettable,
                                       headerLabels.toArray());
         table.setClassAttr("dataTable");
         for (int j = 1; j < table.getColumnModel().size() - 1; j++) {
-            table.getColumn(j).setKey(privileges.get(j - 1));
+//            table.getColumn(j).setKey(privileges.get(j - 1));
+//            table.getColumn(j).setKey(privileges[j - 1]);
             table.getColumn(j).setCellRenderer(new TableCellRenderer() {
 
                 @Override
@@ -349,8 +371,10 @@ public class CMSPermissionsPane extends SimpleContainer implements Resettable,
                 final PermissionChecker permissionChecker = cdiUtil.findBean(
                     PermissionChecker.class);
                 if (columnIndex > 0 && columnIndex < lastColumnIndex) {
-                    final String privilege = table.getColumn(columnIndex)
-                        .getKey();
+//                    final String privilege = table
+//                        .getColumn(columnIndex)
+//                        .getKey();
+                    final String privilege = privileges[columnIndex - 1];
 
                     if (permissionChecker.isPermitted(privilege,
                                                       object,
diff --git a/ccm-cms/src/main/java/com/arsdigita/cms/ui/permissions/CMSPermissionsTableModel.java b/ccm-cms/src/main/java/com/arsdigita/cms/ui/permissions/CMSPermissionsTableModel.java
index dc1e6ce98..819ed7786 100644
--- a/ccm-cms/src/main/java/com/arsdigita/cms/ui/permissions/CMSPermissionsTableModel.java
+++ b/ccm-cms/src/main/java/com/arsdigita/cms/ui/permissions/CMSPermissionsTableModel.java
@@ -79,22 +79,23 @@ class CMSPermissionsTableModel implements TableModel {
     @Override
     public Object getKeyAt(final int columnIndex) {
 
-        if (columnIndex == 0) {
-            return String.format("%s-%s-role",
-                                 currentRow.getObject().getUuid(),
-                                 currentRow.getRoleName());
-        } else if (columnIndex >= currentRow.getColumns().size() - 1) {
-            return String.format("%s-%s-remove-all",
-                                 currentRow.getObject().getUuid(),
-                                 currentRow.getRoleName());
-        } else {
-            return String.format(
-                "%s-%s-%s",
-                currentRow.getObject().getUuid(),
-                currentRow.getRoleName(),
-                currentRow.getColumns().get(columnIndex - 1).getPrivilege()
-            );
-        }
+        return currentRow.getRoleName();
+        
+//        if (columnIndex == 0) {
+//            return String.format("%s-%s-role",
+//                                 currentRow.getObject().getUuid(),
+//                                 currentRow.getRoleName());
+//        } else if (columnIndex >= currentRow.getColumns().size() - 1) {
+//            return String.format("%s-%s-remove-all",
+//                                 currentRow.getObject().getUuid(),
+//                                 currentRow.getRoleName());
+//        } else {
+//            return String.format(
+//                "%s-%s-%s",
+//                currentRow.getObject().getUuid(),
+//                currentRow.getRoleName(),
+//                currentRow.getColumns().get(columnIndex - 1).getPrivilege()
+//            );
+//        }
     }
-
 }
diff --git a/ccm-cms/src/main/java/com/arsdigita/cms/ui/permissions/ItemPermissionsStep.java b/ccm-cms/src/main/java/com/arsdigita/cms/ui/permissions/ItemPermissionsStep.java
index 05e1768f4..2b36654a0 100644
--- a/ccm-cms/src/main/java/com/arsdigita/cms/ui/permissions/ItemPermissionsStep.java
+++ b/ccm-cms/src/main/java/com/arsdigita/cms/ui/permissions/ItemPermissionsStep.java
@@ -19,18 +19,10 @@
 package com.arsdigita.cms.ui.permissions;
 
 import com.arsdigita.bebop.BoxPanel;
-import com.arsdigita.bebop.Component;
-import com.arsdigita.bebop.ControlLink;
-import com.arsdigita.bebop.FormProcessException;
 import com.arsdigita.bebop.Label;
 import com.arsdigita.bebop.PageState;
-import com.arsdigita.bebop.Resettable;
-import com.arsdigita.bebop.SimpleContainer;
 import com.arsdigita.bebop.Table;
-import com.arsdigita.bebop.event.TableActionEvent;
-import com.arsdigita.bebop.event.TableActionListener;
 import com.arsdigita.bebop.parameters.StringParameter;
-import com.arsdigita.bebop.table.TableCellRenderer;
 import com.arsdigita.bebop.table.TableModel;
 import com.arsdigita.bebop.table.TableModelBuilder;
 import com.arsdigita.cms.CMS;
@@ -38,22 +30,19 @@ import com.arsdigita.cms.ItemSelectionModel;
 import com.arsdigita.cms.ui.authoring.AuthoringKitWizard;
 import com.arsdigita.cms.ui.authoring.ResettableContainer;
 import com.arsdigita.globalization.GlobalizedMessage;
+import com.arsdigita.ui.CcmObjectSelectionModel;
 import com.arsdigita.util.LockableImpl;
 
 import org.libreccm.cdi.utils.CdiUtil;
 import org.libreccm.core.CcmObject;
-import org.libreccm.core.UnexpectedErrorException;
-import org.libreccm.security.PermissionChecker;
 import org.libreccm.security.PermissionManager;
-import org.libreccm.security.Role;
-import org.libreccm.security.RoleRepository;
 import org.librecms.CmsConstants;
 import org.librecms.contentsection.privileges.ItemPrivileges;
 import org.librecms.ui.authoring.ContentItemAuthoringStep;
 
+import java.util.HashMap;
 import java.util.List;
-import java.util.Optional;
-import java.util.stream.Collectors;
+import java.util.Map;
 
 /**
  *
@@ -73,15 +62,11 @@ public class ItemPermissionsStep extends ResettableContainer {
                                final AuthoringKitWizard authoringKitWizard,
                                final StringParameter selectedLanguage) {
 
-        super("cms:permissionsStep", CMS.CMS_XML_NS);
+                super("cms:permissionsStep", CMS.CMS_XML_NS);
 
         this.itemSelectionModel = itemSelectionModel;
 
         final BoxPanel panel = new BoxPanel(BoxPanel.VERTICAL);
-        final Label header = new Label(new GlobalizedMessage(
-            "cms.ui.permissions.table.header",
-            CmsConstants.CMS_BUNDLE));
-        panel.add(header);
         super.add(panel);
 
         final CdiUtil cdiUtil = CdiUtil.createCdiUtil();
@@ -89,134 +74,166 @@ public class ItemPermissionsStep extends ResettableContainer {
             PermissionManager.class);
         final List<String> privileges = permissionManager
             .listDefiniedPrivileges(ItemPrivileges.class);
-        final List<Label> headerLabels = privileges.stream()
-            .map(privilege -> generatePrivilegeColumnHeader(privilege))
-            .collect(Collectors.toList());
-        headerLabels.add(0,
-                         new Label(new GlobalizedMessage(
-                             "cms.ui.permissions.table.role_header",
-                             CmsConstants.CMS_BUNDLE)));
-        headerLabels.add(new Label(new GlobalizedMessage(
-            "cms.ui.permissions.table.remove_all.header",
-            CmsConstants.CMS_BUNDLE)));
-        final Table table = new Table(
-            new PermissionsTableModelBuilder(),
-            headerLabels.toArray());
-        table.setClassAttr("dataTable");
-        for (int j = 1; j < table.getColumnModel().size() - 1; j++) {
-            table.getColumn(j).setKey(privileges.get(j - 1));
-            table.getColumn(j).setCellRenderer(new TableCellRenderer() {
+        final Map<String, String> privNameMap = new HashMap<>();
+        privileges.forEach(privilege -> privNameMap.put(privilege, privilege));
 
-                @Override
-                public Component getComponent(final Table table,
-                                              final PageState state,
-                                              final Object value,
-                                              final boolean isSelected,
-                                              final Object key,
-                                              final int row,
-                                              final int column) {
+        final CcmObjectSelectionModel<CcmObject> objSelectionModel
+                                                     = new CcmObjectSelectionModel<>(
+                itemSelectionModel.getStateParameter().getName());
 
-                    final ControlLink link = new ControlLink("");
-
-                    final CMSPermissionsTableColumn col
-                                                        = (CMSPermissionsTableColumn) value;
-                    if (col.isPermitted()) {
-                        link.setClassAttr("checkBoxChecked");
-                    } else {
-                        link.setClassAttr("checkBoxUnchecked");
-                    }
-
-                    return link;
-                }
-
-            });
-        }
-        table.getColumn(table.getColumnModel().size() - 1).setCellRenderer(
-            new TableCellRenderer() {
-
-            @Override
-            public Component getComponent(final Table table,
-                                          final PageState state,
-                                          final Object value,
-                                          final boolean isSelected,
-                                          final Object key,
-                                          final int row,
-                                          final int column) {
-                final ControlLink link = new ControlLink(new Label(
-                    new GlobalizedMessage(
-                        "cms.ui.permissions.table.actions.remove_all",
-                        CmsConstants.CMS_BUNDLE)));
-                link.setConfirmation(new GlobalizedMessage(
-                    "cms.ui.permissions.table.actions.remove_all.confirm",
-                    CmsConstants.CMS_BUNDLE));
-
-                return link;
-            }
-
-        });
-        table.addTableActionListener(new TableActionListener() {
-
-            @Override
-            public void cellSelected(final TableActionEvent event)
-                throws FormProcessException {
-
-                final PageState state = event.getPageState();
-                final int columnIndex = event.getColumn();
-                if (event.getRowKey() == null) {
-                    return;
-                }
-                final String roleName = (String) event.getRowKey();
-
-                final Table table = (Table) event.getSource();
-                final int columnCount = table.getColumnModel().size();
-                final int lastColumnIndex = columnCount - 1;
-
-                final CcmObject object = itemSelectionModel
-                    .getSelectedItem(state);
-                final RoleRepository roleRepo = cdiUtil.findBean(
-                    RoleRepository.class);
-                final Optional<Role> role = roleRepo.findByName(roleName);
-                if (!role.isPresent()) {
-                    throw new UnexpectedErrorException(String.format(
-                        "Role \"%s\" was not found inthe database, but was in "
-                            + "the permissions table.",
-                        roleName));
-                }
-                final PermissionChecker permissionChecker = cdiUtil.findBean(
-                    PermissionChecker.class);
-                if (columnIndex > 0 && columnIndex < lastColumnIndex) {
-                    final String privilege = table.getColumn(columnIndex)
-                        .getKey();
-
-                    if (permissionChecker.isPermitted(privilege,
-                                                      object,
-                                                      role.get())) {
-                        permissionManager.revokePrivilege(privilege,
-                                                          role.get(),
-                                                          object);
-                    } else {
-                        permissionManager.grantPrivilege(privilege,
-                                                         role.get(),
-                                                         object);
-                    }
-                } else if (columnIndex == lastColumnIndex) {
-                    final List<String> privileges = permissionManager
-                        .listDefiniedPrivileges(ItemPrivileges.class);
-                    privileges.forEach(privilege -> permissionManager
-                        .revokePrivilege(privilege, role.get(), object));
-                }
-            }
-
-            @Override
-            public void headSelected(final TableActionEvent event) {
-                //Nothing
-            }
-
-        });
-
-        panel.add(table);
+        final CMSPermissionsPane permissionsPane = new CMSPermissionsPane(
+            privileges.toArray(new String[]{}),
+            privNameMap,
+            objSelectionModel);
+        panel.add(permissionsPane);
     }
 
+//    public ItemPermissionsStep(final ItemSelectionModel itemSelectionModel,
+//                               final AuthoringKitWizard authoringKitWizard,
+//                               final StringParameter selectedLanguage) {
+//        super("cms:permissionsStep", CMS.CMS_XML_NS);
+//
+//        this.itemSelectionModel = itemSelectionModel;
+//
+//        final BoxPanel panel = new BoxPanel(BoxPanel.VERTICAL);
+//        final Label header = new Label(new GlobalizedMessage(
+//            "cms.ui.permissions.table.header",
+//            CmsConstants.CMS_BUNDLE));
+//        panel.add(header);
+//        super.add(panel);
+//
+//        final CdiUtil cdiUtil = CdiUtil.createCdiUtil();
+//        final PermissionManager permissionManager = cdiUtil.findBean(
+//            PermissionManager.class);
+//        final List<String> privileges = permissionManager
+//            .listDefiniedPrivileges(ItemPrivileges.class);
+//        final List<Label> headerLabels = privileges.stream()
+//            .map(privilege -> generatePrivilegeColumnHeader(privilege))
+//            .collect(Collectors.toList());
+//        headerLabels.add(0,
+//                         new Label(new GlobalizedMessage(
+//                             "cms.ui.permissions.table.role_header",
+//                             CmsConstants.CMS_BUNDLE)));
+//        headerLabels.add(new Label(new GlobalizedMessage(
+//            "cms.ui.permissions.table.remove_all.header",
+//            CmsConstants.CMS_BUNDLE)));
+//        final Table table = new Table(
+//            new PermissionsTableModelBuilder(),
+//            headerLabels.toArray());
+//        table.setClassAttr("dataTable");
+//        for (int j = 1; j < table.getColumnModel().size() - 1; j++) {
+//            table.getColumn(j).setKey(privileges.get(j - 1));
+//            table.getColumn(j).setCellRenderer(new TableCellRenderer() {
+//
+//                @Override
+//                public Component getComponent(final Table table,
+//                                              final PageState state,
+//                                              final Object value,
+//                                              final boolean isSelected,
+//                                              final Object key,
+//                                              final int row,
+//                                              final int column) {
+//
+//                    final ControlLink link = new ControlLink("");
+//
+//                    final CMSPermissionsTableColumn col
+//                                                        = (CMSPermissionsTableColumn) value;
+//                    if (col.isPermitted()) {
+//                        link.setClassAttr("checkBoxChecked");
+//                    } else {
+//                        link.setClassAttr("checkBoxUnchecked");
+//                    }
+//
+//                    return link;
+//                }
+//
+//            });
+//        }
+//        table.getColumn(table.getColumnModel().size() - 1).setCellRenderer(
+//            new TableCellRenderer() {
+//
+//            @Override
+//            public Component getComponent(final Table table,
+//                                          final PageState state,
+//                                          final Object value,
+//                                          final boolean isSelected,
+//                                          final Object key,
+//                                          final int row,
+//                                          final int column) {
+//                final ControlLink link = new ControlLink(new Label(
+//                    new GlobalizedMessage(
+//                        "cms.ui.permissions.table.actions.remove_all",
+//                        CmsConstants.CMS_BUNDLE)));
+//                link.setConfirmation(new GlobalizedMessage(
+//                    "cms.ui.permissions.table.actions.remove_all.confirm",
+//                    CmsConstants.CMS_BUNDLE));
+//
+//                return link;
+//            }
+//
+//        });
+//        table.addTableActionListener(new TableActionListener() {
+//
+//            @Override
+//            public void cellSelected(final TableActionEvent event)
+//                throws FormProcessException {
+//
+//                final PageState state = event.getPageState();
+//                final int columnIndex = event.getColumn();
+//                if (event.getRowKey() == null) {
+//                    return;
+//                }
+//                final String roleName = (String) event.getRowKey();
+//
+//                final Table table = (Table) event.getSource();
+//                final int columnCount = table.getColumnModel().size();
+//                final int lastColumnIndex = columnCount - 1;
+//
+//                final CcmObject object = itemSelectionModel
+//                    .getSelectedItem(state);
+//                final RoleRepository roleRepo = cdiUtil.findBean(
+//                    RoleRepository.class);
+//                final Optional<Role> role = roleRepo.findByName(roleName);
+//                if (!role.isPresent()) {
+//                    throw new UnexpectedErrorException(String.format(
+//                        "Role \"%s\" was not found inthe database, but was in "
+//                            + "the permissions table.",
+//                        roleName));
+//                }
+//                final PermissionChecker permissionChecker = cdiUtil.findBean(
+//                    PermissionChecker.class);
+//                if (columnIndex > 0 && columnIndex < lastColumnIndex) {
+//                    final String privilege = table.getColumn(columnIndex)
+//                        .getKey();
+//
+//                    if (permissionChecker.isPermitted(privilege,
+//                                                      object,
+//                                                      role.get())) {
+//                        permissionManager.revokePrivilege(privilege,
+//                                                          role.get(),
+//                                                          object);
+//                    } else {
+//                        permissionManager.grantPrivilege(privilege,
+//                                                         role.get(),
+//                                                         object);
+//                    }
+//                } else if (columnIndex == lastColumnIndex) {
+//                    final List<String> privileges = permissionManager
+//                        .listDefiniedPrivileges(ItemPrivileges.class);
+//                    privileges.forEach(privilege -> permissionManager
+//                        .revokePrivilege(privilege, role.get(), object));
+//                }
+//            }
+//
+//            @Override
+//            public void headSelected(final TableActionEvent event) {
+//                //Nothing
+//            }
+//
+//        });
+//
+//        panel.add(table);
+//    }
     private Label generatePrivilegeColumnHeader(final String privilege) {
         return new Label(new GlobalizedMessage(
             String.format("cms.ui.permissions.table.privilege.headers.%s",
diff --git a/ccm-cms/src/main/java/org/librecms/contentsection/ContentSectionConfig.java b/ccm-cms/src/main/java/org/librecms/contentsection/ContentSectionConfig.java
index f729ce14a..7141d1d62 100644
--- a/ccm-cms/src/main/java/org/librecms/contentsection/ContentSectionConfig.java
+++ b/ccm-cms/src/main/java/org/librecms/contentsection/ContentSectionConfig.java
@@ -129,7 +129,7 @@ public class ContentSectionConfig {
                 ItemCategoryStep.class.getName(),
                 ImageStep.class.getName(),
                 RelatedInfoStep.class.getName(),
-//                ItemPermissionsStep.class.getName()
+                ItemPermissionsStep.class.getName()
             });
 
     public static ContentSectionConfig getConfig() {
diff --git a/ccm-core/src/main/java/com/arsdigita/ui/admin/AdminServlet.java b/ccm-core/src/main/java/com/arsdigita/ui/admin/AdminServlet.java
index 5282b24f0..b3d3b0bb1 100644
--- a/ccm-core/src/main/java/com/arsdigita/ui/admin/AdminServlet.java
+++ b/ccm-core/src/main/java/com/arsdigita/ui/admin/AdminServlet.java
@@ -146,10 +146,10 @@ public class AdminServlet extends BaseApplicationServlet {
                                             ADMIN_BUNDLE)),
             new ConfigurationTab());
 
-        tabbedPane.addTab(
-            new Label(new GlobalizedMessage("ui.admin.tab.workflows.title",
-                                            ADMIN_BUNDLE)),
-            new WorkflowAdminTab());
+//        tabbedPane.addTab(
+//            new Label(new GlobalizedMessage("ui.admin.tab.workflows.title",
+//                                            ADMIN_BUNDLE)),
+//            new WorkflowAdminTab());
 
         tabbedPane.addTab(
             new Label(new GlobalizedMessage("ui.admin.tab.importexport.title",
diff --git a/ccm-core/src/main/java/com/arsdigita/ui/admin/WorkflowAdminTab.java b/ccm-core/src/main/java/com/arsdigita/ui/admin/WorkflowAdminTab.java
deleted file mode 100644
index 20bce0e6d..000000000
--- a/ccm-core/src/main/java/com/arsdigita/ui/admin/WorkflowAdminTab.java
+++ /dev/null
@@ -1,38 +0,0 @@
-/*
- * Copyright (C) 2016 LibreCCM Foundation.
- *
- * This library is free software; you can redistribute it and/or
- * modify it under the terms of the GNU Lesser General Public
- * License as published by the Free Software Foundation; either
- * version 2.1 of the License, or (at your option) any later version.
- *
- * This library is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- * Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public
- * License along with this library; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
- * MA 02110-1301  USA
- */
-package com.arsdigita.ui.admin;
-
-import com.arsdigita.bebop.Label;
-import com.arsdigita.toolbox.ui.LayoutPanel;
-
-/**
- *
- * @author <a href="mailto:jens.pelzetter@googlemail.com">Jens Pelzetter</a>
- */
-public class WorkflowAdminTab extends LayoutPanel {
-    
-    public WorkflowAdminTab() {
-        super();
-
-        setLeft(new Label("Workflow Admin Tab Left"));
-
-        setBody(new Label("Workflow Admin Tab Body"));
-    }
-    
-}
diff --git a/ccm-core/src/main/java/com/arsdigita/ui/admin/usersgroupsroles/groups/GroupsTable.java b/ccm-core/src/main/java/com/arsdigita/ui/admin/usersgroupsroles/groups/GroupsTable.java
index 5eb2fc7a1..a665a1c51 100644
--- a/ccm-core/src/main/java/com/arsdigita/ui/admin/usersgroupsroles/groups/GroupsTable.java
+++ b/ccm-core/src/main/java/com/arsdigita/ui/admin/usersgroupsroles/groups/GroupsTable.java
@@ -65,8 +65,8 @@ class GroupsTable extends Table {
         
         super();
 
-        setIdAttr("groupsTable");
-        setStyleAttr("width: 30em");
+        super.setIdAttr("groupsTable");
+        super.setStyleAttr("width: 30em");
 
         this.groupsTableFilter = groupsTableFilter;
         this.selectedGroupId = selectedGroupId;