Get rid of array_walk() when simple foreach() can be used

9 years ago · 8cc6b18bcd
parent 70422cd7a2
commit 8cc6b18bcd
1 changed files with 4 additions and 7 deletions
--- a/program/lib/Roundcube/rcube_ldap.php
+++ b/program/lib/Roundcube/rcube_ldap.php
@ -834,15 +834,12 @@ class rcube_ldap extends rcube_addressbook
            }
            else {
                // map address book fields into ldap attributes
-                $me         = $this;
                $attributes = array();
-                $fields     = (array) $fields;
-
-                array_walk($fields, function($field) use ($me, &$attributes) {
-                    if ($me->coltypes[$field] && ($attrs = (array)$me->coltypes[$field]['attributes'])) {
-                        $attributes = array_merge($attributes, $attrs);
+                foreach ((array) $fields as $field) {
+                    if ($this->coltypes[$field] && ($attrs = $this->coltypes[$field]['attributes'])) {
+                        $attributes = array_merge($attributes, (array) $attrs);
                    }
-                });
+                }
            }

            // compose a full-text-like search filter