From 8cbebcac6aba53860a022c699a5384cbabc06a87 Mon Sep 17 00:00:00 2001 From: Aleksander Machniak <alec@alec.pl> Date: Thu, 30 Oct 2014 07:57:43 -0400 Subject: [PATCH] CS fixes and bump version number after PR merge --- plugins/new_user_identity/composer.json | 2 +- plugins/new_user_identity/package.xml | 4 ++-- plugins/new_user_identity/new_user_identity.php | 22 ++++++++++++++-------- 3 files changed, 17 insertions(+), 11 deletions(-) diff --git a/plugins/new_user_identity/composer.json b/plugins/new_user_identity/composer.json index 11a0af7..a2e9e95 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 9764722..f7d60d4 100644 --- a/plugins/new_user_identity/new_user_identity.php +++ b/plugins/new_user_identity/new_user_identity.php @@ -67,21 +67,27 @@ 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'])); + $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) { + 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 45a3c4c..1240939 100644 --- a/plugins/new_user_identity/package.xml +++ b/plugins/new_user_identity/package.xml @@ -15,9 +15,9 @@ <email>alec@alec.pl</email> <active>yes</active> </lead> - <date>2012-08-13</date> + <date>2014-10-30</date> <version> - <release>1.0.7</release> + <release>1.1</release> <api>1.1</api> </version> <stability> -- Gitblit v1.9.1