X-Git-Url: https://git.argeo.org/?a=blobdiff_plain;ds=sidebyside;f=org.argeo.util%2Fsrc%2Forg%2Fargeo%2Fosgi%2Fuseradmin%2FAggregatingUserAdmin.java;h=79d2bd3cbc2f21142f1801a1df5c5fbc307c2095;hb=336930c69f0cd3e1242e518479624c6366541275;hp=ac641de97cf6a128b5c2781f9c053cba8e98a7d7;hpb=cceead8a279e8630f63cc9e8213bdcdca39955a5;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.util/src/org/argeo/osgi/useradmin/AggregatingUserAdmin.java b/org.argeo.util/src/org/argeo/osgi/useradmin/AggregatingUserAdmin.java
index ac641de97..79d2bd3cb 100644
--- a/org.argeo.util/src/org/argeo/osgi/useradmin/AggregatingUserAdmin.java
+++ b/org.argeo.util/src/org/argeo/osgi/useradmin/AggregatingUserAdmin.java
@@ -1,6 +1,6 @@
 package org.argeo.osgi.useradmin;
 
-import static org.argeo.osgi.useradmin.AbstractUserDirectory.toLdapName;
+import static org.argeo.osgi.useradmin.DirectoryUserAdmin.toLdapName;
 
 import java.util.ArrayList;
 import java.util.Arrays;
@@ -30,9 +30,9 @@ public class AggregatingUserAdmin implements UserAdmin {
 	private final LdapName tokensBaseDn;
 
 	// DAOs
-	private AbstractUserDirectory systemRoles = null;
-	private AbstractUserDirectory tokens = null;
-	private Map<LdapName, AbstractUserDirectory> businessRoles = new HashMap<LdapName, AbstractUserDirectory>();
+	private DirectoryUserAdmin systemRoles = null;
+	private DirectoryUserAdmin tokens = null;
+	private Map<LdapName, DirectoryUserAdmin> businessRoles = new HashMap<LdapName, DirectoryUserAdmin>();
 
 	// TODO rather use an empty constructor and an init method
 	public AggregatingUserAdmin(String systemRolesBaseDn, String tokensBaseDn) {
@@ -91,7 +91,7 @@ public class AggregatingUserAdmin implements UserAdmin {
 		if (user == null) {// anonymous
 			return systemRoles.getAuthorization(null);
 		}
-		AbstractUserDirectory userReferentialOfThisUser = findUserAdmin(user.getName());
+		DirectoryUserAdmin userReferentialOfThisUser = findUserAdmin(user.getName());
 		Authorization rawAuthorization = userReferentialOfThisUser.getAuthorization(user);
 		String usernameToUse;
 		String displayNameToUse;
@@ -113,11 +113,11 @@ public class AggregatingUserAdmin implements UserAdmin {
 		}
 
 		// gather roles from other referentials
-		final AbstractUserDirectory userAdminToUse;// possibly scoped when authenticating
+		final DirectoryUserAdmin userAdminToUse;// possibly scoped when authenticating
 		if (user instanceof DirectoryUser) {
 			userAdminToUse = userReferentialOfThisUser;
 		} else if (user instanceof AuthenticatingUser) {
-			userAdminToUse = userReferentialOfThisUser.scope(user);
+			userAdminToUse = (DirectoryUserAdmin) userReferentialOfThisUser.scope(user);
 		} else {
 			throw new IllegalArgumentException("Unsupported user type " + user.getClass());
 		}
@@ -157,9 +157,9 @@ public class AggregatingUserAdmin implements UserAdmin {
 	// USER ADMIN AGGREGATOR
 	//
 	protected void addUserDirectory(UserDirectory ud) {
-		if (!(ud instanceof AbstractUserDirectory))
-			throw new IllegalArgumentException("Only " + AbstractUserDirectory.class.getName() + " is supported");
-		AbstractUserDirectory userDirectory = (AbstractUserDirectory) ud;
+		if (!(ud instanceof DirectoryUserAdmin))
+			throw new IllegalArgumentException("Only " + DirectoryUserAdmin.class.getName() + " is supported");
+		DirectoryUserAdmin userDirectory = (DirectoryUserAdmin) ud;
 		String basePath = userDirectory.getContext();
 		if (isSystemRolesBaseDn(basePath)) {
 			this.systemRoles = userDirectory;
@@ -181,7 +181,7 @@ public class AggregatingUserAdmin implements UserAdmin {
 	protected void postAdd(UserDirectory userDirectory) {
 	}
 
-	private AbstractUserDirectory findUserAdmin(String name) {
+	private DirectoryUserAdmin findUserAdmin(String name) {
 		try {
 			return findUserAdmin(new LdapName(name));
 		} catch (InvalidNameException e) {
@@ -189,14 +189,14 @@ public class AggregatingUserAdmin implements UserAdmin {
 		}
 	}
 
-	private AbstractUserDirectory findUserAdmin(LdapName name) {
+	private DirectoryUserAdmin findUserAdmin(LdapName name) {
 		if (name.startsWith(systemRolesBaseDn))
 			return systemRoles;
 		if (tokensBaseDn != null && name.startsWith(tokensBaseDn))
 			return tokens;
-		List<AbstractUserDirectory> res = new ArrayList<>(1);
+		List<DirectoryUserAdmin> res = new ArrayList<>(1);
 		userDirectories: for (LdapName baseDn : businessRoles.keySet()) {
-			AbstractUserDirectory userDirectory = businessRoles.get(baseDn);
+			DirectoryUserAdmin userDirectory = businessRoles.get(baseDn);
 			if (name.startsWith(baseDn)) {
 				if (userDirectory.isDisabled())
 					continue userDirectories;
@@ -238,9 +238,13 @@ public class AggregatingUserAdmin implements UserAdmin {
 //		return res;
 //	}
 
-	public void destroy() {
+	public void start() {
+
+	}
+
+	public void stop() {
 		for (LdapName name : businessRoles.keySet()) {
-			AbstractUserDirectory userDirectory = businessRoles.get(name);
+			DirectoryUserAdmin userDirectory = businessRoles.get(name);
 			destroy(userDirectory);
 		}
 		businessRoles.clear();
@@ -249,18 +253,26 @@ public class AggregatingUserAdmin implements UserAdmin {
 		systemRoles = null;
 	}
 
-	private void destroy(AbstractUserDirectory userDirectory) {
+	private void destroy(DirectoryUserAdmin userDirectory) {
 		preDestroy(userDirectory);
 		userDirectory.destroy();
 	}
 
+//	protected void removeUserDirectory(UserDirectory userDirectory) {
+//		LdapName baseDn = toLdapName(userDirectory.getContext());
+//		businessRoles.remove(baseDn);
+//		if (userDirectory instanceof DirectoryUserAdmin)
+//			destroy((DirectoryUserAdmin) userDirectory);
+//	}
+
+	@Deprecated
 	protected void removeUserDirectory(String basePath) {
 		if (isSystemRolesBaseDn(basePath))
 			throw new IllegalArgumentException("System roles cannot be removed ");
 		LdapName baseDn = toLdapName(basePath);
 		if (!businessRoles.containsKey(baseDn))
 			throw new IllegalStateException("No user directory registered for " + baseDn);
-		AbstractUserDirectory userDirectory = businessRoles.remove(baseDn);
+		DirectoryUserAdmin userDirectory = businessRoles.remove(baseDn);
 		destroy(userDirectory);
 	}
 
@@ -274,6 +286,8 @@ public class AggregatingUserAdmin implements UserAdmin {
 	public Set<UserDirectory> getUserDirectories() {
 		TreeSet<UserDirectory> res = new TreeSet<>((o1, o2) -> o1.getContext().compareTo(o2.getContext()));
 		res.addAll(businessRoles.values());
+		res.add(systemRoles);
 		return res;
 	}
+
 }