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

This commit is contained in:
Arthur Schiwon 2013-11-05 13:19:32 +01:00
parent c3898061a8
commit fc040755c1
1 changed files with 13 additions and 0 deletions

View File

@ -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');