X-Git-Url: https://git.argeo.org/?a=blobdiff_plain;f=security%2Fplugins%2Forg.argeo.security.ui.admin%2Fsrc%2Fmain%2Fjava%2Forg%2Fargeo%2Fsecurity%2Fui%2Fadmin%2Fcommands%2FRefreshUsersList.java;fp=security%2Fplugins%2Forg.argeo.security.ui.admin%2Fsrc%2Fmain%2Fjava%2Forg%2Fargeo%2Fsecurity%2Fui%2Fadmin%2Fcommands%2FRefreshUsersList.java;h=71ec21c445de98291a9da7836edbe7538316de10;hb=61320e5902070e49d32379aab06faaa1544835ab;hp=c40d4ab5db7b5cd75ceb0ff8ca4dbc6197d073d3;hpb=2862f0241c2f32b2113639b5d0cc2a878c56aadd;p=lgpl%2Fargeo-commons.git

diff --git a/security/plugins/org.argeo.security.ui.admin/src/main/java/org/argeo/security/ui/admin/commands/RefreshUsersList.java b/security/plugins/org.argeo.security.ui.admin/src/main/java/org/argeo/security/ui/admin/commands/RefreshUsersList.java
index c40d4ab5d..71ec21c44 100644
--- a/security/plugins/org.argeo.security.ui.admin/src/main/java/org/argeo/security/ui/admin/commands/RefreshUsersList.java
+++ b/security/plugins/org.argeo.security.ui.admin/src/main/java/org/argeo/security/ui/admin/commands/RefreshUsersList.java
@@ -19,6 +19,7 @@ import java.util.Set;
 
 import javax.jcr.Node;
 import javax.jcr.NodeIterator;
+import javax.jcr.Repository;
 import javax.jcr.RepositoryException;
 import javax.jcr.Session;
 import javax.jcr.query.Query;
@@ -40,11 +41,13 @@ import org.eclipse.ui.handlers.HandlerUtil;
  */
 public class RefreshUsersList extends AbstractHandler {
 	private UserAdminService userAdminService;
-	private Session session;
+	private Repository repository;
 
 	public Object execute(ExecutionEvent event) throws ExecutionException {
 		Set<String> users = userAdminService.listUsers();
+		Session session = null;
 		try {
+			session = repository.login();
 			Query query = session
 					.getWorkspace()
 					.getQueryManager()
@@ -63,6 +66,8 @@ public class RefreshUsersList extends AbstractHandler {
 		} catch (RepositoryException e) {
 			JcrUtils.discardQuietly(session);
 			throw new ArgeoException("Cannot list users", e);
+		} finally {
+			JcrUtils.logoutQuietly(session);
 		}
 
 		userAdminService.synchronize();
@@ -77,8 +82,8 @@ public class RefreshUsersList extends AbstractHandler {
 		this.userAdminService = userAdminService;
 	}
 
-	public void setSession(Session session) {
-		this.session = session;
+	public void setRepository(Repository repository) {
+		this.repository = repository;
 	}
 
 }
\ No newline at end of file