diff options
author | Aleksander Machniak <alec@alec.pl> | 2014-10-18 15:27:31 +0200 |
---|---|---|
committer | Aleksander Machniak <alec@alec.pl> | 2014-10-18 15:27:31 +0200 |
commit | e2251db1d5b536f61fe7db57b96a9601465bd139 (patch) | |
tree | b8cbd74b83933b9bcde7c4efc6b813fa6ba6afca /plugins/new_user_identity | |
parent | 111af7f077d50bff1ae95185a0d486c852f54b88 (diff) |
CS fixes after PR merge
Diffstat (limited to 'plugins/new_user_identity')
-rw-r--r-- | plugins/new_user_identity/new_user_identity.php | 29 |
1 files changed, 10 insertions, 19 deletions
diff --git a/plugins/new_user_identity/new_user_identity.php b/plugins/new_user_identity/new_user_identity.php index bc0658a4b..4dbca3aab 100644 --- a/plugins/new_user_identity/new_user_identity.php +++ b/plugins/new_user_identity/new_user_identity.php @@ -36,42 +36,33 @@ class new_user_identity extends rcube_plugin { if ($this->init_ldap($args['host'])) { $results = $this->ldap->search('*', $args['user'], true); + if (count($results->records) == 1) { $user_name = is_array($results->records[0]['name']) ? $results->records[0]['name'][0] : $results->records[0]['name']; $user_email = is_array($results->records[0]['email']) ? $results->records[0]['email'][0] : $results->records[0]['email']; - $args['user_name'] = $user_name; + $args['user_name'] = $user_name; + $args['email_list'] = array(); + if (!$args['user_email'] && strpos($user_email, '@')) { $args['user_email'] = rcube_utils::idn_to_ascii($user_email); } - $args['email_list'] = array(); - $i=0; - $keys = array_keys( $results[0] ); - - foreach ( $keys as $key ) { - if (!preg_match( '/^email($|:)/', $key )) { + foreach (array_keys($results[0]) as $key) { + if (!preg_match('/^email($|:)/', $key)) { continue; } - if (is_array($results->records[0][$key])) { - foreach($results->records[0][$key] as $alias) { - if(strpos($alias, '@')) { - $args['email_list'][$i++] = rcube_idn_to_ascii($alias); - } - } - } - else { - $alias = $results->records[0][$key]; + foreach ((array) $results->records[0][$key] as $alias) { if (strpos($alias, '@')) { - $args['email_list'][$i++] = rcube_idn_to_ascii($alias); + $args['email_list'][] = rcube_utils::idn_to_ascii($alias); } - - } + } } } } + return $args; } |