Remove 32 bits SWT.
[lgpl/argeo-commons.git] / org.argeo.cms.ui.workbench / src / org / argeo / cms / ui / workbench / internal / useradmin / parts / GroupMainPage.java
index 0b16466adac9962f0e2a8f9b0c212b938658ece8..6987a217d807c06450aa927e36817dda7ce0a44d 100644 (file)
  */
 package org.argeo.cms.ui.workbench.internal.useradmin.parts;
 
+import static org.argeo.cms.util.UserAdminUtils.setProperty;
+import static org.argeo.naming.LdapAttrs.businessCategory;
+import static org.argeo.naming.LdapAttrs.description;
+import static org.argeo.node.NodeInstance.WORKGROUP;
+
 import java.util.ArrayList;
 import java.util.Iterator;
 import java.util.List;
@@ -38,7 +43,6 @@ import org.argeo.cms.ui.workbench.internal.useradmin.providers.CommonNameLP;
 import org.argeo.cms.ui.workbench.internal.useradmin.providers.MailLP;
 import org.argeo.cms.ui.workbench.internal.useradmin.providers.RoleIconLP;
 import org.argeo.cms.ui.workbench.internal.useradmin.providers.UserFilter;
-import org.argeo.cms.ui.workbench.internal.useradmin.providers.UserNameLP;
 import org.argeo.cms.ui.workbench.internal.useradmin.providers.UserTableDefaultDClickListener;
 import org.argeo.cms.util.CmsUtils;
 import org.argeo.cms.util.UserAdminUtils;
@@ -65,6 +69,7 @@ import org.eclipse.swt.dnd.Transfer;
 import org.eclipse.swt.dnd.TransferData;
 import org.eclipse.swt.events.DisposeEvent;
 import org.eclipse.swt.events.DisposeListener;
+import org.eclipse.swt.events.ModifyListener;
 import org.eclipse.swt.events.SelectionAdapter;
 import org.eclipse.swt.events.SelectionEvent;
 import org.eclipse.swt.graphics.Cursor;
@@ -132,28 +137,29 @@ public class GroupMainPage extends FormPage implements ArgeoNames {
        protected void appendOverviewPart(final Composite parent, final Group group) {
                FormToolkit tk = getManagedForm().getToolkit();
                Composite body = addSection(tk, parent);
-               GridLayout layout = new GridLayout(5, false);
+               // GridLayout layout = new GridLayout(5, false);
+               GridLayout layout = new GridLayout(2, false);
                body.setLayout(layout);
 
-               final Text dnTxt = createLT(body, "DN", group.getName());
-               dnTxt.setEnabled(false);
-
-               final String cn = UserAdminUtils.getProperty(group, LdapAttrs.cn.name());
-               final Text cnTxt = createLT(body, "Common Name", cn);
-               cnTxt.setEnabled(false);
-
-               final Link markAsWorkgroupLk = new Link(body, SWT.NONE);
-               markAsWorkgroupLk.setLayoutData(new GridData(SWT.FILL, SWT.CENTER, true, false));
-
-               // Label descLbl = new Label(body, SWT.LEAD);
-               // descLbl.setText("Description");
-               // descLbl.setLayoutData(new GridData(SWT.RIGHT, SWT.TOP, false,
-               // false));
-               // final Text descTxt = new Text(body, SWT.LEAD | SWT.MULTI | SWT.WRAP
-               // | SWT.BORDER);
-               // GridData gd = EclipseUiUtils.fillAll();
-               // gd.heightHint = 100;
-               // descTxt.setLayoutData(gd);
+               String cn = UserAdminUtils.getProperty(group, LdapAttrs.cn.name());
+               createReadOnlyLT(body, "Name", cn);
+               // Text dnTxt = createReadOnlyLT(body, "DN", group.getName());
+               createReadOnlyLT(body, "Domain", UserAdminUtils.getDomainName(group));
+
+               // Description
+               Label descLbl = new Label(body, SWT.LEAD);
+               descLbl.setFont(EclipseUiUtils.getBoldFont(body));
+               descLbl.setText("Description");
+               descLbl.setLayoutData(new GridData(SWT.LEAD, SWT.CENTER, false, false, 2, 1));
+               final Text descTxt = new Text(body, SWT.LEAD | SWT.MULTI | SWT.WRAP | SWT.BORDER);
+               GridData gd = EclipseUiUtils.fillAll();
+               gd.heightHint = 50;
+               gd.horizontalSpan = 2;
+               descTxt.setLayoutData(gd);
+
+               // Mark as workgroup
+               Link markAsWorkgroupLk = new Link(body, SWT.NONE);
+               markAsWorkgroupLk.setLayoutData(new GridData(SWT.FILL, SWT.CENTER, false, false, 2, 1));
 
                // create form part (controller)
                final AbstractFormPart part = new SectionPart((Section) body.getParent()) {
@@ -174,20 +180,21 @@ public class GroupMainPage extends FormPage implements ArgeoNames {
                        }
 
                        public void commit(boolean onSave) {
-                               // group.getProperties().put(LdapAttrs.description.name(),
-                               // descTxt.getText());
+                               // group.getProperties().put(LdapAttrs.description.name(), descTxt.getText());
+                               setProperty(group, description, descTxt.getText());
                                super.commit(onSave);
                        }
 
                        @Override
                        public void refresh() {
-                               dnTxt.setText(group.getName());
-                               cnTxt.setText(UserAdminUtils.getProperty(group, LdapAttrs.cn.name()));
+                               // dnTxt.setText(group.getName());
+                               // cnTxt.setText(UserAdminUtils.getProperty(group, LdapAttrs.cn.name()));
+                               descTxt.setText(UserAdminUtils.getProperty(group, LdapAttrs.description.name()));
                                Node workgroupHome = NodeUtils.getGroupHome(session, cn);
                                if (workgroupHome == null)
                                        markAsWorkgroupLk.setText("<a>Mark as workgroup</a>");
                                else
-                                       markAsWorkgroupLk.setText(cn + " is already marked as being a workgroup");
+                                       markAsWorkgroupLk.setText("Configured as workgroup");
                                parent.layout(true, true);
                                super.refresh();
                        }
@@ -207,7 +214,13 @@ public class GroupMainPage extends FormPage implements ArgeoNames {
                                                return; // already marked as workgroup, do nothing
                                        else
                                                try {
+                                                       // improve transaction management
+                                                       userAdminWrapper.beginTransactionIfNeeded();
                                                        nodeInstance.createWorkgroup(new LdapName(group.getName()));
+                                                       setProperty(group, businessCategory, WORKGROUP);
+                                                       userAdminWrapper.commitOrNotifyTransactionStateChange();
+                                                       userAdminWrapper
+                                                                       .notifyListeners(new UserAdminEvent(null, UserAdminEvent.ROLE_CHANGED, group));
                                                        part.refresh();
                                                } catch (InvalidNameException e1) {
                                                        throw new CmsException("Cannot create Workgroup for " + group.toString(), e1);
@@ -217,16 +230,16 @@ public class GroupMainPage extends FormPage implements ArgeoNames {
                        }
                });
 
-               // ModifyListener defaultListener = editor.new FormPartML(part);
-               // descTxt.addModifyListener(defaultListener);
+               ModifyListener defaultListener = editor.new FormPartML(part);
+               descTxt.addModifyListener(defaultListener);
                getManagedForm().addPart(part);
        }
 
-       /** Filtered table with members. Has drag & drop ability */
+       /** Filtered table with members. Has drag and drop ability */
        protected void appendMembersPart(Composite parent, Group group) {
-
                FormToolkit tk = getManagedForm().getToolkit();
                Section section = tk.createSection(parent, Section.TITLE_BAR);
+               section.setText("Members");
                section.setLayoutData(EclipseUiUtils.fillAll());
 
                Composite body = new Composite(section, SWT.NO_FOCUS);
@@ -235,7 +248,7 @@ public class GroupMainPage extends FormPage implements ArgeoNames {
 
                LdifUsersTable userTableViewerCmp = createMemberPart(body, group);
 
-               SectionPart part = new GroupMembersPart(section, userTableViewerCmp, group);
+               SectionPart part = new GroupMembersPart(section, userTableViewerCmp);
                getManagedForm().addPart(part);
                addRemoveAbitily(part, userTableViewerCmp.getTableViewer(), group);
        }
@@ -246,9 +259,10 @@ public class GroupMainPage extends FormPage implements ArgeoNames {
                // Define the displayed columns
                List<ColumnDefinition> columnDefs = new ArrayList<ColumnDefinition>();
                columnDefs.add(new ColumnDefinition(new RoleIconLP(), "", 0, 24));
-               columnDefs.add(new ColumnDefinition(new CommonNameLP(), "Common Name", 150));
-               columnDefs.add(new ColumnDefinition(new MailLP(), "Primary Mail", 150));
-               columnDefs.add(new ColumnDefinition(new UserNameLP(), "Distinguished Name", 240));
+               columnDefs.add(new ColumnDefinition(new CommonNameLP(), "Name", 150));
+               columnDefs.add(new ColumnDefinition(new MailLP(), "Mail", 150));
+               // columnDefs.add(new ColumnDefinition(new UserNameLP(), "Distinguished Name",
+               // 240));
 
                // Create and configure the table
                LdifUsersTable userViewerCmp = new MyUserTableViewer(parent, SWT.MULTI | SWT.H_SCROLL | SWT.V_SCROLL,
@@ -283,10 +297,12 @@ public class GroupMainPage extends FormPage implements ArgeoNames {
 
                @Override
                protected List<User> listFilteredElements(String filter) {
+                       // reload user and set it in the editor
                        Group group = (Group) editor.getDisplayedUser();
                        Role[] roles = group.getMembers();
                        List<User> users = new ArrayList<User>();
                        userFilter.setSearchText(filter);
+                       // userFilter.setShowSystemRole(true);
                        for (Role role : roles)
                                // if (role.getType() == Role.GROUP)
                                if (userFilter.select(null, null, role))
@@ -356,14 +372,14 @@ public class GroupMainPage extends FormPage implements ArgeoNames {
        // LOCAL CONTROLLERS
        private class GroupMembersPart extends SectionPart {
                private final LdifUsersTable userViewer;
-               private final Group group;
+               // private final Group group;
 
                private GroupChangeListener listener;
 
-               public GroupMembersPart(Section section, LdifUsersTable userViewer, Group group) {
+               public GroupMembersPart(Section section, LdifUsersTable userViewer) {
                        super(section);
                        this.userViewer = userViewer;
-                       this.group = group;
+                       // this.group = group;
                }
 
                @Override
@@ -381,7 +397,6 @@ public class GroupMainPage extends FormPage implements ArgeoNames {
 
                @Override
                public void refresh() {
-                       getSection().setText("Members of group " + UserAdminUtils.getProperty(group, LdapAttrs.cn.name()));
                        userViewer.refresh();
                        super.refresh();
                }
@@ -482,14 +497,27 @@ public class GroupMainPage extends FormPage implements ArgeoNames {
        }
 
        /** Creates label and text. */
-       private Text createLT(Composite parent, String label, String value) {
+       // private Text createLT(Composite parent, String label, String value) {
+       // FormToolkit toolkit = getManagedForm().getToolkit();
+       // Label lbl = toolkit.createLabel(parent, label);
+       // lbl.setLayoutData(new GridData(SWT.LEAD, SWT.CENTER, false, false));
+       // lbl.setFont(EclipseUiUtils.getBoldFont(parent));
+       // Text text = toolkit.createText(parent, value, SWT.BORDER);
+       // text.setLayoutData(new GridData(SWT.FILL, SWT.CENTER, true, false));
+       // CmsUtils.style(text, CmsWorkbenchStyles.WORKBENCH_FORM_TEXT);
+       // return text;
+       // }
+       //
+       Text createReadOnlyLT(Composite parent, String label, String value) {
                FormToolkit toolkit = getManagedForm().getToolkit();
                Label lbl = toolkit.createLabel(parent, label);
-               lbl.setLayoutData(new GridData(SWT.END, SWT.CENTER, false, false));
+               lbl.setLayoutData(new GridData(SWT.LEAD, SWT.CENTER, false, false));
                lbl.setFont(EclipseUiUtils.getBoldFont(parent));
-               Text text = toolkit.createText(parent, value, SWT.BORDER);
+               Text text = toolkit.createText(parent, value, SWT.NONE);
                text.setLayoutData(new GridData(SWT.FILL, SWT.CENTER, true, false));
+               text.setEditable(false);
                CmsUtils.style(text, CmsWorkbenchStyles.WORKBENCH_FORM_TEXT);
                return text;
        }
+
 }