From e2251db1d5b536f61fe7db57b96a9601465bd139 Mon Sep 17 00:00:00 2001 From: Aleksander Machniak Date: Sat, 18 Oct 2014 15:27:31 +0200 Subject: CS fixes after PR merge --- plugins/new_user_identity/new_user_identity.php | 29 +++++++++---------------- 1 file changed, 10 insertions(+), 19 deletions(-) (limited to 'plugins') 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; } -- cgit v1.2.3