X-Git-Url: https://git.argeo.org/?a=blobdiff_plain;f=org.argeo.util%2Fsrc%2Forg%2Fargeo%2Futil%2Fdirectory%2Fldap%2FDefaultLdapEntry.java;h=7166710f61191e594dd776714d6b7bd56f73d459;hb=e9d0731a65b0d3523906c58f987ad9610c4286b7;hp=4212c5f5576fa1cfec836695ac7f03bf9dcd5140;hpb=55d1a78150b6be0004f6bcb28703bcdd6daf55a1;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.util/src/org/argeo/util/directory/ldap/DefaultLdapEntry.java b/org.argeo.util/src/org/argeo/util/directory/ldap/DefaultLdapEntry.java
index 4212c5f55..7166710f6 100644
--- a/org.argeo.util/src/org/argeo/util/directory/ldap/DefaultLdapEntry.java
+++ b/org.argeo.util/src/org/argeo/util/directory/ldap/DefaultLdapEntry.java
@@ -35,8 +35,8 @@ public class DefaultLdapEntry implements LdapEntry {
 	private Attributes publishedAttributes;
 
 	// Temporarily expose the fields
-	protected final AttributeDictionary properties;
-	protected final AttributeDictionary credentials;
+	protected AttributeDictionary properties;
+	protected AttributeDictionary credentials;
 
 	protected DefaultLdapEntry(AbstractLdapDirectory directory, LdapName dn, Attributes attributes) {
 		Objects.requireNonNull(directory);
@@ -44,8 +44,8 @@ public class DefaultLdapEntry implements LdapEntry {
 		this.directory = directory;
 		this.dn = dn;
 		this.publishedAttributes = attributes;
-		properties = new AttributeDictionary(false);
-		credentials = new AttributeDictionary(true);
+//		properties = new AttributeDictionary(false);
+//		credentials = new AttributeDictionary(true);
 	}
 
 	@Override
@@ -54,6 +54,9 @@ public class DefaultLdapEntry implements LdapEntry {
 	}
 
 	public synchronized Attributes getAttributes() {
+		// lazy loading
+		if (publishedAttributes == null)
+			publishedAttributes = getDirectory().getDirectoryDao().doGetAttributes(dn);
 		return isEditing() ? getModifiedAttributes() : publishedAttributes;
 	}
 
@@ -103,15 +106,23 @@ public class DefaultLdapEntry implements LdapEntry {
 	public synchronized void publishAttributes(Attributes modifiedAttributes) {
 		publishedAttributes = modifiedAttributes;
 	}
-	
+
 	/*
 	 * PROPERTIES
 	 */
 	@Override
 	public Dictionary<String, Object> getProperties() {
+		if (properties == null)
+			properties = new AttributeDictionary(false);
 		return properties;
 	}
 
+	public Dictionary<String, Object> getCredentials() {
+		if (credentials == null)
+			credentials = new AttributeDictionary(false);
+		return credentials;
+	}
+
 	/*
 	 * CREDENTIALS
 	 */
@@ -153,7 +164,7 @@ public class DefaultLdapEntry implements LdapEntry {
 			return false;
 		}
 
-		Object storedValue = credentials.get(key);
+		Object storedValue = getCredentials().get(key);
 		if (storedValue == null || value == null)
 			return false;
 		if (!(value instanceof String || value instanceof byte[]))