diff options
| author | Aleksander Machniak <alec@alec.pl> | 2012-10-16 17:52:15 +0200 |
|---|---|---|
| committer | Aleksander Machniak <alec@alec.pl> | 2012-10-16 17:52:15 +0200 |
| commit | 699cb1fda3594705918c987714e7277572afb8b0 (patch) | |
| tree | c1cd8222334d4bb0a8c3fc645c5929cf2acc374f /program/include | |
| parent | 08df33b5c25577cf42d7f524eeaa367cbde886fb (diff) | |
Fix post-filtering vlv results, fixes warning "mb_strtolower() expects parameter 1 to be a string, array given"
Diffstat (limited to 'program/include')
| -rw-r--r-- | program/include/rcube_ldap.php | 39 |
1 files changed, 21 insertions, 18 deletions
diff --git a/program/include/rcube_ldap.php b/program/include/rcube_ldap.php index b9b6490fa..61a073fa3 100644 --- a/program/include/rcube_ldap.php +++ b/program/include/rcube_ldap.php @@ -771,8 +771,9 @@ class rcube_ldap extends rcube_addressbook // use VLV pseudo-search for autocompletion $rcube = rcube::get_instance(); + $list_fields = $rcube->config->get('contactlist_fields'); - if ($this->prop['vlv_search'] && $this->conn && join(',', (array)$fields) == join(',', $rcube->config->get('contactlist_fields'))) + if ($this->prop['vlv_search'] && $this->conn && join(',', (array)$fields) == join(',', $list_fields)) { // add general filter to query if (!empty($this->prop['filter']) && empty($this->filter)) @@ -800,24 +801,26 @@ class rcube_ldap extends rcube_addressbook for ($i = 0; $i < $entries['count']; $i++) { $rec = $this->_ldap2result($entries[$i]); - foreach (array('email', 'name') as $f) { - $val = mb_strtolower($rec[$f]); - switch ($mode) { - case 1: - $got = ($val == $search); - break; - case 2: - $got = ($search == substr($val, 0, strlen($search))); - break; - default: - $got = (strpos($val, $search) !== false); - break; - } + foreach ($fields as $f) { + foreach ((array)$rec[$f] as $val) { + $val = mb_strtolower($val); + switch ($mode) { + case 1: + $got = ($val == $search); + break; + case 2: + $got = ($search == substr($val, 0, strlen($search))); + break; + default: + $got = (strpos($val, $search) !== false); + break; + } - if ($got) { - $this->result->add($rec); - $this->result->count++; - break; + if ($got) { + $this->result->add($rec); + $this->result->count++; + break 2; + } } } } |