diff --git a/apps/user_ldap/templates/settings.php b/apps/user_ldap/templates/settings.php index 8a879fa246..5f25570e22 100644 --- a/apps/user_ldap/templates/settings.php +++ b/apps/user_ldap/templates/settings.php @@ -5,10 +5,11 @@

- t('Leave both empty for anonymous bind for search, then bind with users credentials.');?>

+ t('Leave both empty for anonymous bind for search, then bind with users credentials.');?>

-

-

+

+

+ t('Currently the display name field needs to be the same you matched %%uid against in the filter above, because ownCloud doesn\'t distinguish between user id and user name.');?>

>

diff --git a/apps/user_ldap/user_ldap.php b/apps/user_ldap/user_ldap.php index 8557a7fde5..249def7a1c 100644 --- a/apps/user_ldap/user_ldap.php +++ b/apps/user_ldap/user_ldap.php @@ -143,8 +143,10 @@ class OC_USER_LDAP extends OC_User_Backend { else { $users = array(); foreach($entries as $row) { - if(isset($row[$this->ldap_display_name])) { - $users[] = $row[$this->ldap_display_name][0]; + // TODO ldap_get_entries() seems to lower all keys => needs review + $ldap_display_name = strtolower($this->ldap_display_name); + if(isset($row[$ldap_display_name])) { + $users[] = $row[$ldap_display_name][0]; } } // TODO language specific sorting of user names