LDAP Wizard: do not forget to set display name attributes for user and group, makes Test Connection succeed

1 files changed, 13 insertions, 0 deletions

diff --git a/apps/user_ldap/lib/wizard.php b/apps/user_ldap/lib/wizard.php
index 9428a98e29c..a2628ed1c8f 100644
--- a/apps/user_ldap/lib/wizard.php
+++ b/apps/user_ldap/lib/wizard.php
@@ -328,6 +328,13 @@ class Wizard extends LDAPUtility {
 										   ))) {
 			return false;
 		}
+		//make sure the use display name is set
+		$displayName = $this->configuration->ldapGroupDisplayName;
+		if(empty($displayName)) {
+			$d = $this->configuration->getDefaults();
+			$this->applyFind('ldap_group_display_name',
+							 $d['ldap_group_display_name']);
+		}
 		$filter = $this->composeLdapFilter(self::LFILTER_GROUP_LIST);
 
 		$this->applyFind('ldap_group_filter', $filter);
@@ -343,6 +350,12 @@ class Wizard extends LDAPUtility {
 										   ))) {
 			return false;
 		}
+		//make sure the use display name is set
+		$displayName = $this->configuration->ldapUserDisplayName;
+		if(empty($displayName)) {
+			$d = $this->configuration->getDefaults();
+			$this->applyFind('ldap_display_name', $d['ldap_display_name']);
+		}
 		$filter = $this->composeLdapFilter(self::LFILTER_USER_LIST);
 		if(!$filter) {
 			throw new \Exception('Cannot create filter');