X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=org.argeo.security.ui.admin%2Fsrc%2Forg%2Fargeo%2Fsecurity%2Fui%2Fadmin%2Finternal%2Fcommands%2FNewGroup.java;h=b83412d1081fa6a4076e4d59fab3079d1c172d45;hb=1c75ba2f683c41d0e081970758d1fb5b2e91689f;hp=37bae611ffa7c11cb753bd77e65b4df85bf28046;hpb=331be8c500d2477aa1476b9f93ff56302c830322;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/commands/NewGroup.java b/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/commands/NewGroup.java
index 37bae611f..b83412d10 100644
--- a/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/commands/NewGroup.java
+++ b/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/commands/NewGroup.java
@@ -64,10 +64,10 @@ public class NewGroup extends AbstractHandler {
 
 	private class NewGroupWizard extends Wizard {
 
-		// pages
+		// Pages
 		private MainGroupInfoWizardPage mainGroupInfo;
 
-		// End user fields
+		// UI fields
 		private Text dNameTxt, commonNameTxt, descriptionTxt;
 		private Combo baseDnCmb;
 
@@ -88,8 +88,9 @@ public class NewGroup extends AbstractHandler {
 			String commonName = commonNameTxt.getText();
 			try {
 				userAdminWrapper.beginTransactionIfNeeded();
+				String dn = getDn(commonName);
 				Group group = (Group) userAdminWrapper.getUserAdmin()
-						.createRole(getDn(commonName), Role.GROUP);
+						.createRole(dn, Role.GROUP);
 				Dictionary props = group.getProperties();
 				String descStr = descriptionTxt.getText();
 				if (EclipseUiUtils.notEmpty(descStr))
@@ -133,7 +134,6 @@ public class NewGroup extends AbstractHandler {
 						"Common name");
 				commonNameTxt.addFocusListener(this);
 
-				
 				Label descLbl = new Label(bodyCmp, SWT.LEAD);
 				descLbl.setText("Description");
 				descLbl.setLayoutData(new GridData(SWT.RIGHT, SWT.TOP, false,
@@ -171,7 +171,7 @@ public class NewGroup extends AbstractHandler {
 			public void focusGained(FocusEvent event) {
 			}
 
-			/** @return error message or null if complete */
+			/** @return the error message or null if complete */
 			protected String checkComplete() {
 				String name = commonNameTxt.getText();