X-Git-Url: https://git.argeo.org/?a=blobdiff_plain;ds=sidebyside;f=security%2Feclipse%2Fplugins%2Forg.argeo.security.ui%2Fsrc%2Fmain%2Fjava%2Forg%2Fargeo%2Fsecurity%2Fui%2Feditors%2FArgeoUserEditor.java;fp=security%2Feclipse%2Fplugins%2Forg.argeo.security.ui%2Fsrc%2Fmain%2Fjava%2Forg%2Fargeo%2Fsecurity%2Fui%2Feditors%2FArgeoUserEditor.java;h=7c0213704a299a6785c7d3695597de491eddee60;hb=2663dfc4d32c474f8fc7407d24bdcf0fa50ce08a;hp=70afce67762652c34663f49e431cb4ee65a96f15;hpb=814f2e01731aca304d826d070922d51fd84efa29;p=lgpl%2Fargeo-commons.git

diff --git a/security/eclipse/plugins/org.argeo.security.ui/src/main/java/org/argeo/security/ui/editors/ArgeoUserEditor.java b/security/eclipse/plugins/org.argeo.security.ui/src/main/java/org/argeo/security/ui/editors/ArgeoUserEditor.java
index 70afce677..7c0213704 100644
--- a/security/eclipse/plugins/org.argeo.security.ui/src/main/java/org/argeo/security/ui/editors/ArgeoUserEditor.java
+++ b/security/eclipse/plugins/org.argeo.security.ui/src/main/java/org/argeo/security/ui/editors/ArgeoUserEditor.java
@@ -5,6 +5,8 @@ import org.apache.commons.logging.LogFactory;
 import org.argeo.ArgeoException;
 import org.argeo.security.ArgeoSecurityService;
 import org.argeo.security.ArgeoUser;
+import org.argeo.security.SimpleArgeoUser;
+import org.argeo.security.nature.SimpleUserNature;
 import org.eclipse.core.runtime.IProgressMonitor;
 import org.eclipse.ui.IEditorInput;
 import org.eclipse.ui.IEditorSite;
@@ -25,9 +27,14 @@ public class ArgeoUserEditor extends FormEditor {
 		super.init(site, input);
 		String username = ((ArgeoUserEditorInput) getEditorInput())
 				.getUsername();
-		user = securityService.getSecurityDao().getUser(username);
-		this.setPartProperty("name", username);
-		setPartName(username);
+		if (username == null) {// new
+			user = new SimpleArgeoUser();
+			user.getUserNatures().put(SimpleUserNature.TYPE,
+					new SimpleUserNature());
+		} else
+			user = securityService.getSecurityDao().getUser(username);
+		this.setPartProperty("name", username != null ? username : "<new user>");
+		setPartName(username != null ? username : "<new user>");
 	}
 
 	protected void addPages() {
@@ -41,16 +48,23 @@ public class ArgeoUserEditor extends FormEditor {
 
 	@Override
 	public void doSave(IProgressMonitor monitor) {
-		if (log.isDebugEnabled())
-			log.debug("doSave called");
-		// for (int i = 0; i < getPageCount(); i++) {
-		// IEditorPart editor = getEditor(i);
-		// if (editor != null)
-		// editor.doSave(monitor);
-		// }
+		// list pages
+		// TODO: make it more generic
 		findPage(DefaultUserMainPage.ID).doSave(monitor);
 
-		securityService.updateUser(user);
+		if (securityService.getSecurityDao().userExists(user.getUsername()))
+			securityService.updateUser(user);
+		else {
+			try {
+				// FIXME: make it cleaner
+				((SimpleArgeoUser)user).setPassword(user.getUsername());
+				securityService.newUser(user);
+				setPartName(user.getUsername());
+			} catch (Exception e) {
+				// TODO Auto-generated catch block
+				e.printStackTrace();
+			}
+		}
 		firePropertyChange(PROP_DIRTY);
 	}