From 8cbebcac6aba53860a022c699a5384cbabc06a87 Mon Sep 17 00:00:00 2001 From: Aleksander Machniak Date: Thu, 30 Oct 2014 12:57:43 +0100 Subject: CS fixes and bump version number after PR merge --- plugins/new_user_identity/composer.json | 2 +- plugins/new_user_identity/new_user_identity.php | 24 +++++++++++++++--------- plugins/new_user_identity/package.xml | 4 ++-- 3 files changed, 18 insertions(+), 12 deletions(-) (limited to 'plugins') diff --git a/plugins/new_user_identity/composer.json b/plugins/new_user_identity/composer.json index 11a0af7cd..a2e9e95dc 100644 --- a/plugins/new_user_identity/composer.json +++ b/plugins/new_user_identity/composer.json @@ -3,7 +3,7 @@ "type": "roundcube-plugin", "description": "Populates a new user's default identity from LDAP on their first visit.", "license": "GNU GPLv3+", - "version": "1.0.7", + "version": "1.1", "authors": [ { "name": "Aleksander Machniak", diff --git a/plugins/new_user_identity/new_user_identity.php b/plugins/new_user_identity/new_user_identity.php index 976472210..f7d60d4fa 100644 --- a/plugins/new_user_identity/new_user_identity.php +++ b/plugins/new_user_identity/new_user_identity.php @@ -67,21 +67,27 @@ class new_user_identity extends rcube_plugin return $args; } - $identities = $this->rc->user->list_identities(); - $ldap_entry = $this->lookup_user_name(array('user' => $this->rc->user->data['username'], - 'host' => $this->rc->user->data['mail_host'])); - - foreach ($ldap_entry['email_list'] as $email) { - foreach($identities as $identity) { + $identities = $this->rc->user->list_identities(); + $ldap_entry = $this->lookup_user_name(array( + 'user' => $this->rc->user->data['username'], + 'host' => $this->rc->user->data['mail_host'], + )); + + foreach ((array) $ldap_entry['email_list'] as $email) { + foreach ($identities as $identity) { if ($identity['email'] == $email ) { continue 2; } } $plugin = $this->rc->plugins->exec_hook('identity_create', array( - 'login' => true, - 'record' => array('user_id' => $this->rc->user->ID, 'standard' => 0, - 'email' => $email, 'name' => $ldap_entry['user_name']), + 'login' => true, + 'record' => array( + 'user_id' => $this->rc->user->ID, + 'standard' => 0, + 'email' => $email, + 'name' => $ldap_entry['user_name'] + ), )); if (!$plugin['abort'] && $plugin['record']['email']) { diff --git a/plugins/new_user_identity/package.xml b/plugins/new_user_identity/package.xml index 45a3c4c8d..12409392b 100644 --- a/plugins/new_user_identity/package.xml +++ b/plugins/new_user_identity/package.xml @@ -15,9 +15,9 @@ alec@alec.pl yes - 2012-08-13 + 2014-10-30 - 1.0.7 + 1.1 1.1 -- cgit v1.2.3