X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=org.argeo.security.ui.admin%2Fsrc%2Forg%2Fargeo%2Fsecurity%2Fui%2Fadmin%2Finternal%2Fcommands%2FNewUser.java;h=1a85253ad9238a4e83cc0a428c93f7141d72e19f;hb=3a0d866fbeea3f78c293212f4b4fbaeba7dfe2bd;hp=6d853c91946406d3ffa63fc0c04c11c72402f4ce;hpb=25e98954db6faeec4ba9950f651e81fbea595b0c;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/commands/NewUser.java b/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/commands/NewUser.java
index 6d853c919..1a85253ad 100644
--- a/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/commands/NewUser.java
+++ b/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/commands/NewUser.java
@@ -17,18 +17,20 @@ package org.argeo.security.ui.admin.internal.commands;
 
 import java.util.Dictionary;
 import java.util.List;
+import java.util.Map;
 
 import javax.naming.InvalidNameException;
 import javax.naming.ldap.LdapName;
 import javax.naming.ldap.Rdn;
 
-import org.argeo.ArgeoException;
+import org.argeo.cms.CmsException;
+import org.argeo.cms.util.useradmin.UserAdminUtils;
 import org.argeo.eclipse.ui.EclipseUiUtils;
 import org.argeo.eclipse.ui.dialogs.ErrorFeedback;
 import org.argeo.jcr.ArgeoNames;
 import org.argeo.osgi.useradmin.LdifName;
+import org.argeo.osgi.useradmin.UserAdminConf;
 import org.argeo.security.ui.admin.SecurityAdminPlugin;
-import org.argeo.security.ui.admin.internal.UiAdminUtils;
 import org.argeo.security.ui.admin.internal.UserAdminWrapper;
 import org.eclipse.core.commands.AbstractHandler;
 import org.eclipse.core.commands.ExecutionEvent;
@@ -112,20 +114,20 @@ public class NewUser extends AbstractHandler {
 				Dictionary props = user.getProperties();
 
 				String lastNameStr = lastNameTxt.getText();
-				if (UiAdminUtils.notNull(lastNameStr))
+				if (EclipseUiUtils.notEmpty(lastNameStr))
 					props.put(LdifName.sn.name(), lastNameStr);
 
 				String firstNameStr = firstNameTxt.getText();
-				if (UiAdminUtils.notNull(firstNameStr))
+				if (EclipseUiUtils.notEmpty(firstNameStr))
 					props.put(LdifName.givenName.name(), firstNameStr);
 
-				String cn = UiAdminUtils
-						.getDefaultCn(firstNameStr, lastNameStr);
-				if (UiAdminUtils.notNull(cn))
+				String cn = UserAdminUtils.buildDefaultCn(firstNameStr,
+						lastNameStr);
+				if (EclipseUiUtils.notEmpty(cn))
 					props.put(LdifName.cn.name(), cn);
 
 				String mailStr = primaryMailTxt.getText();
-				if (UiAdminUtils.notNull(mailStr))
+				if (EclipseUiUtils.notEmpty(mailStr))
 					props.put(LdifName.mail.name(), mailStr);
 
 				char[] password = mainUserInfo.getPassword();
@@ -267,16 +269,29 @@ public class NewUser extends AbstractHandler {
 
 		}
 
+		private Map<String, String> getDns() {
+			return userAdminWrapper.getKnownBaseDns(true);
+		}
+
 		private String getDn(String uid) {
-			return "uid=" + uid + ",ou=users," + baseDnCmb.getText();
+			Map<String, String> dns = getDns();
+			String bdn = baseDnCmb.getText();
+			if (EclipseUiUtils.notEmpty(bdn)) {
+				Dictionary<String, ?> props = UserAdminConf.uriAsProperties(dns
+						.get(bdn));
+				String dn = LdifName.uid.name() + "=" + uid + ","
+						+ UserAdminConf.userBase.getValue(props) + "," + bdn;
+				return dn;
+			}
+			return null;
 		}
 
 		private void initialiseDnCmb(Combo combo) {
-			List<String> dns = userAdminWrapper.getKnownBaseDns(true);
+			Map<String, String> dns = userAdminWrapper.getKnownBaseDns(true);
 			if (dns.isEmpty())
-				throw new ArgeoException(
+				throw new CmsException(
 						"No writable base dn found. Cannot create user");
-			combo.setItems(dns.toArray(new String[0]));
+			combo.setItems(dns.keySet().toArray(new String[0]));
 			if (dns.size() == 1)
 				combo.select(0);
 		}
@@ -291,7 +306,7 @@ public class NewUser extends AbstractHandler {
 				return username + "@" + (String) rdns.get(1).getValue() + '.'
 						+ (String) rdns.get(0).getValue();
 			} catch (InvalidNameException e) {
-				throw new ArgeoException("Unable to generate mail for "
+				throw new CmsException("Unable to generate mail for "
 						+ username + " with base dn " + baseDn, e);
 			}
 		}