X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=org.argeo.security.core%2Fsrc%2Forg%2Fargeo%2Fosgi%2Fuseradmin%2FLdifAuthorization.java;h=e06c42e1ffb213c501ce23b2bc2edd90498b74c0;hb=8260f4470f514ea347ca53f5b4dfc632c4a4de66;hp=845fefd13da483a6ee88c3c48690c0f45062ad0d;hpb=25071ab6bcb2df1fa4057c2c04137f2d606772e7;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.security.core/src/org/argeo/osgi/useradmin/LdifAuthorization.java b/org.argeo.security.core/src/org/argeo/osgi/useradmin/LdifAuthorization.java
index 845fefd13..e06c42e1f 100644
--- a/org.argeo.security.core/src/org/argeo/osgi/useradmin/LdifAuthorization.java
+++ b/org.argeo.security.core/src/org/argeo/osgi/useradmin/LdifAuthorization.java
@@ -9,11 +9,13 @@ import org.osgi.service.useradmin.Authorization;
 import org.osgi.service.useradmin.Role;
 import org.osgi.service.useradmin.User;
 
-public class LdifAuthorization implements Authorization, LdapNames {
+/** Basic authorization. */
+class LdifAuthorization implements Authorization {
 	private final String name;
 	private final String displayName;
 	private final List<String> allRoles;
 
+	@SuppressWarnings("unchecked")
 	public LdifAuthorization(User user, List<Role> allRoles) {
 		if (user == null) {
 			this.name = null;
@@ -21,16 +23,16 @@ public class LdifAuthorization implements Authorization, LdapNames {
 		} else {
 			this.name = user.getName();
 			Dictionary<String, Object> props = user.getProperties();
-			Object displayName = props.get(LDAP_DISPLAY_NAME);
+			Object displayName = props.get(LdifName.displayName);
 			if (displayName == null)
-				displayName = props.get(LDAP_CN);
+				displayName = props.get(LdifName.cn);
 			if (displayName == null)
-				displayName = props.get(LDAP_UID);
+				displayName = props.get(LdifName.uid);
 			if (displayName == null)
 				displayName = user.getName();
 			if (displayName == null)
-				throw new UserDirectoryException(
-						"Cannot set display name for " + user);
+				throw new UserDirectoryException("Cannot set display name for "
+						+ user);
 			this.displayName = displayName.toString();
 		}
 		// roles