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=7600883989eb33f3287eedb87880671dcbf6d401;hb=d249537c17dc045b106bb661cf74e8c98d8b2a48;hp=4c1e8f4cbe47f0dba2ff84051b3c3264a2eb93b6;hpb=86140b8db15a11cfd942892eface6a4f90329a41;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 4c1e8f4cb..760088398 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
@@ -26,9 +26,9 @@ import org.argeo.ArgeoException;
 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.security.ui.admin.SecurityAdminPlugin;
 import org.argeo.security.ui.admin.internal.UiAdminUtils;
-import org.argeo.security.ui.admin.internal.UserAdminConstants;
 import org.argeo.security.ui.admin.internal.UserAdminWrapper;
 import org.eclipse.core.commands.AbstractHandler;
 import org.eclipse.core.commands.ExecutionEvent;
@@ -69,25 +69,10 @@ public class NewUser extends AbstractHandler {
 		NewUserWizard newUserWizard = new NewUserWizard();
 		WizardDialog dialog = new WizardDialog(
 				HandlerUtil.getActiveShell(event), newUserWizard);
-
 		dialog.open();
-
-		// // Force refresh until the listener are implemented
-		// if (Window.OK == dialog.open())
-		// forceRefresh(event);
 		return null;
 	}
 
-	// private void forceRefresh(ExecutionEvent event) {
-	// IWorkbenchWindow iww = HandlerUtil.getActiveWorkbenchWindow(event);
-	// if (iww == null)
-	// return;
-	// IWorkbenchPage activePage = iww.getActivePage();
-	// IWorkbenchPart part = activePage.getActivePart();
-	// if (part instanceof UsersView)
-	// ((UsersView) part).refresh();
-	// }
-
 	private class NewUserWizard extends Wizard {
 
 		// pages
@@ -125,20 +110,20 @@ public class NewUser extends AbstractHandler {
 
 				String lastNameStr = lastNameTxt.getText();
 				if (UiAdminUtils.notNull(lastNameStr))
-					props.put(UserAdminConstants.KEY_LASTNAME, lastNameStr);
+					props.put(LdifName.sn.name(), lastNameStr);
 
 				String firstNameStr = firstNameTxt.getText();
 				if (UiAdminUtils.notNull(firstNameStr))
-					props.put(UserAdminConstants.KEY_FIRSTNAME, firstNameStr);
+					props.put(LdifName.givenname.name(), firstNameStr);
 
 				String cn = UiAdminUtils
 						.getDefaultCn(firstNameStr, lastNameStr);
 				if (UiAdminUtils.notNull(cn))
-					props.put(UserAdminConstants.KEY_CN, cn);
+					props.put(LdifName.cn.name(), cn);
 
 				String mailStr = primaryMailTxt.getText();
 				if (UiAdminUtils.notNull(mailStr))
-					props.put(UserAdminConstants.KEY_MAIL, mailStr);
+					props.put(LdifName.mail.name(), mailStr);
 
 				char[] password = mainUserInfo.getPassword();
 				user.getCredentials().put(null, password);