Improve Security

[lgpl/argeo-commons.git] / security / eclipse / plugins / org.argeo.security.ui / src / main / java / org / argeo / security / ui / editors / ArgeoUserEditor.java

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 9e258a65f8dbcf2cf471adcb2422c3efc9e7051f..70afce67762652c34663f49e431cb4ee65a96f15 100644 (file)
--- 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
@@ -1,9 +1,10 @@
 package org.argeo.security.ui.editors;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 import org.argeo.ArgeoException;
 import org.argeo.security.ArgeoSecurityService;
 import org.argeo.security.ArgeoUser;
-import org.argeo.security.UserNature;
 import org.eclipse.core.runtime.IProgressMonitor;
 import org.eclipse.ui.IEditorInput;
 import org.eclipse.ui.IEditorSite;
@@ -12,6 +13,8 @@ import org.eclipse.ui.forms.editor.FormEditor;
 
 /** Editor for an Argeo user. */
 public class ArgeoUserEditor extends FormEditor {
+       private final static Log log = LogFactory.getLog(ArgeoUserEditor.class);
+
        public final static String ID = "org.argeo.security.ui.argeoUserEditor";
 
        private ArgeoUser user;
@@ -24,12 +27,13 @@ public class ArgeoUserEditor extends FormEditor {
                                .getUsername();
                user = securityService.getSecurityDao().getUser(username);
                this.setPartProperty("name", username);
+               setPartName(username);
        }
 
        protected void addPages() {
                try {
-                       addPage(new DefaultUserMainPage(this, user));
-                               
+                       addPage(new DefaultUserMainPage(this, securityService, user));
+
                } catch (PartInitException e) {
                        throw new ArgeoException("Not able to add page ", e);
                }
@@ -37,24 +41,29 @@ public class ArgeoUserEditor extends FormEditor {
 
        @Override
        public void doSave(IProgressMonitor monitor) {
-               // TODO Auto-generated method stub
+               if (log.isDebugEnabled())
+                       log.debug("doSave called");
+               // for (int i = 0; i < getPageCount(); i++) {
+               // IEditorPart editor = getEditor(i);
+               // if (editor != null)
+               // editor.doSave(monitor);
+               // }
+               findPage(DefaultUserMainPage.ID).doSave(monitor);
 
+               securityService.updateUser(user);
+               firePropertyChange(PROP_DIRTY);
        }
 
        @Override
        public void doSaveAs() {
-               // TODO Auto-generated method stub
-
        }
 
        @Override
        public boolean isSaveAsAllowed() {
-               // TODO Auto-generated method stub
                return false;
        }
 
        public void setSecurityService(ArgeoSecurityService securityService) {
                this.securityService = securityService;
        }
-
 }