From d14a2fb8bc11df61d473d4e61822ca70dbea0ee3 Mon Sep 17 00:00:00 2001 From: Aleksander Machniak <alec@alec.pl> Date: Mon, 29 Apr 2013 02:04:15 -0400 Subject: [PATCH] Fix possible warning on some misconfig --- program/include/rcmail.php | 11 +++++------ 1 files changed, 5 insertions(+), 6 deletions(-) diff --git a/program/include/rcmail.php b/program/include/rcmail.php index 1bde403..3f6b39f 100644 --- a/program/include/rcmail.php +++ b/program/include/rcmail.php @@ -258,13 +258,13 @@ */ public function get_address_sources($writeable = false, $skip_hidden = false) { - $abook_type = strtolower($this->config->get('address_book_type')); - $ldap_config = $this->config->get('ldap_public'); + $abook_type = (string) $this->config->get('address_book_type'); + $ldap_config = (array) $this->config->get('ldap_public'); $autocomplete = (array) $this->config->get('autocomplete_addressbooks'); - $list = array(); + $list = array(); // We are using the DB address book or a plugin address book - if ($abook_type != 'ldap' && $abook_type != '') { + if (!empty($abook_type) && strtolower($abook_type) != 'ldap') { if (!isset($this->address_books['0'])) $this->address_books['0'] = new rcube_contacts($this->db, $this->get_user_id()); $list['0'] = array( @@ -277,8 +277,7 @@ ); } - if ($ldap_config) { - $ldap_config = (array) $ldap_config; + if (!empty($ldap_config)) { foreach ($ldap_config as $id => $prop) { // handle misconfiguration if (empty($prop) || !is_array($prop)) { -- Gitblit v1.9.1