plugins/kolab_auth

Jeroen van Meeuwen vanmeeuwen at kolabsys.com
Mon Aug 19 13:54:23 CEST 2013


 plugins/kolab_auth/kolab_auth.php |   12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

New commits:
commit 44021155b0b48c88878b2e0a0f7cd6ef54de5156
Author: Jeroen van Meeuwen (Kolab Systems) <vanmeeuwen at kolabsys.com>
Date:   Mon Aug 19 13:54:08 2013 +0200

    Make sure we loop over non-empty arrays only

diff --git a/plugins/kolab_auth/kolab_auth.php b/plugins/kolab_auth/kolab_auth.php
index 27c7b56..b139e32 100644
--- a/plugins/kolab_auth/kolab_auth.php
+++ b/plugins/kolab_auth/kolab_auth.php
@@ -114,12 +114,16 @@ class kolab_auth extends rcube_plugin
             return $args;
         }
 
-        foreach ($role_plugins as $role_dn => $plugins) {
-            $role_plugins[$ldap->parse_vars($role_dn)] = $plugins;
+        if (!empty($role_plugins)) {
+            foreach ($role_plugins as $role_dn => $plugins) {
+                $role_plugins[$ldap->parse_vars($role_dn)] = $plugins;
+            }
         }
 
-        foreach ($role_settings as $role_dn => $settings) {
-            $role_settings[$ldap->parse_vars($role_dn)] = $settings;
+        if (!empty($role_settings)) {
+            foreach ($role_settings as $role_dn => $settings) {
+                $role_settings[$ldap->parse_vars($role_dn)] = $settings;
+            }
         }
 
         foreach ($role_dns as $role_dn) {




More information about the commits mailing list