Improve user admin conf

[lgpl/argeo-commons.git] / org.argeo.security.core / src / org / argeo / osgi / useradmin / UserAdminConf.java

diff --git a/org.argeo.security.core/src/org/argeo/osgi/useradmin/UserAdminConf.java b/org.argeo.security.core/src/org/argeo/osgi/useradmin/UserAdminConf.java
index 2ed7edd98e1dd55aba1703bf8341bce095fc96c3..e909e7aa3d3196f627fe6b3b86a55abe25b1714f 100644 (file)
--- a/org.argeo.security.core/src/org/argeo/osgi/useradmin/UserAdminConf.java
+++ b/org.argeo.security.core/src/org/argeo/osgi/useradmin/UserAdminConf.java
@@ -16,10 +16,10 @@ import javax.naming.Context;
 
 /** Properties used to configure user admins. */
 public enum UserAdminConf {
-       /** Base DN */
+       /** Base DN (cannot be configured externally) */
        baseDn("dc=example,dc=com"),
 
-       /** URI of the underlying resource */
+       /** URI of the underlying resource (cannot be configured externally) */
        uri("ldap://localhost:10389"),
 
        /** User objectClass */
@@ -52,7 +52,7 @@ public enum UserAdminConf {
 
        /** For use as Java property. */
        public String property() {
-               return getPrefix() + name();
+               return getPrefix() + '.' + name();
        }
 
        public String getPrefix() {
@@ -75,7 +75,7 @@ public enum UserAdminConf {
        }
 
        public static UserAdminConf local(String property) {
-               return UserAdminConf.valueOf(property.substring(PREFIX.length()));
+               return UserAdminConf.valueOf(property.substring(PREFIX.length() + 1));
        }
 
        /** Hides host and credentials. */