Merge pull request #5700 from owncloud/ldapWizSetDisplayNames

LDAP Wizard: do not forget to set display name attributes for user and g...
This commit is contained in:
Thomas Müller 2013-11-15 01:12:54 -08:00
commit 1e863cd959
1 changed files with 13 additions and 0 deletions

View File

@ -310,6 +310,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);
@ -323,6 +330,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');