From 9d78c68cbf5ce86f02e8fa16814293cdfd204432 Mon Sep 17 00:00:00 2001 From: Aleksander Machniak <alec@alec.pl> Date: Wed, 29 Jul 2015 14:38:21 -0400 Subject: [PATCH] Fix so imap folder attribute comparisons are case-insensitive (#1490466) + make in_array_nocase() much faster for ASCII strings --- program/lib/Roundcube/bootstrap.php | 106 ++++++++++++++++++----------------------------------- 1 files changed, 36 insertions(+), 70 deletions(-) diff --git a/program/lib/Roundcube/bootstrap.php b/program/lib/Roundcube/bootstrap.php index fe9c389..2e5d9e6 100644 --- a/program/lib/Roundcube/bootstrap.php +++ b/program/lib/Roundcube/bootstrap.php @@ -1,9 +1,9 @@ <?php -/* +/** +-----------------------------------------------------------------------+ | This file is part of the Roundcube PHP suite | - | Copyright (C) 2005-2014, The Roundcube Dev Team | + | Copyright (C) 2005-2015, The Roundcube Dev Team | | | | Licensed under the GNU General Public License version 3 or | | any later version with exceptions for skins & plugins. | @@ -16,7 +16,6 @@ | Author: Aleksander Machniak <alec@alec.pl> | +-----------------------------------------------------------------------+ */ - /** * Roundcube Framework Initialization @@ -54,7 +53,7 @@ } // framework constants -define('RCUBE_VERSION', '1.1-git'); +define('RCUBE_VERSION', '1.2-git'); define('RCUBE_CHARSET', 'UTF-8'); if (!defined('RCUBE_LIB_DIR')) { @@ -78,9 +77,11 @@ } // set internal encoding for mbstring extension -if (extension_loaded('mbstring')) { +if (function_exists('mb_internal_encoding')) { mb_internal_encoding(RCUBE_CHARSET); - @mb_regex_encoding(RCUBE_CHARSET); +} +if (function_exists('mb_regex_encoding')) { + mb_regex_encoding(RCUBE_CHARSET); } // make sure the Roundcube lib directory is in the include_path @@ -97,30 +98,40 @@ spl_autoload_register('rcube_autoload'); // set PEAR error handling (will also load the PEAR main class) -PEAR::setErrorHandling(PEAR_ERROR_CALLBACK, 'rcube_pear_error'); - +if (class_exists('PEAR')) { + @PEAR::setErrorHandling(PEAR_ERROR_CALLBACK, 'rcube_pear_error'); +} /** - * Similar function as in_array() but case-insensitive + * Similar function as in_array() but case-insensitive with multibyte support. * - * @param string $needle Needle value - * @param array $heystack Array to search in + * @param string $needle Needle value + * @param array $heystack Array to search in * * @return boolean True if found, False if not */ function in_array_nocase($needle, $haystack) { - $needle = mb_strtolower($needle); - foreach ((array)$haystack as $value) { - if ($needle === mb_strtolower($value)) { - return true; + // use much faster method for ascii + if (is_ascii($needle)) { + foreach ((array) $haystack as $value) { + if (strcasecmp($value, $needle) === 0) { + return true; + } + } + } + else { + $needle = mb_strtolower($needle); + foreach ((array) $haystack as $value) { + if ($needle === mb_strtolower($value)) { + return true; + } } } return false; } - /** * Parse a human readable string for a number of bytes. @@ -156,7 +167,6 @@ return floatval($bytes); } - /** * Make sure the string ends with a slash */ @@ -165,7 +175,6 @@ return unslashify($str).'/'; } - /** * Remove slashes at the end of the string */ @@ -173,7 +182,6 @@ { return preg_replace('/\/+$/', '', $str); } - /** * Returns number of seconds for a specified offset string. @@ -207,7 +215,6 @@ return $amount; } - /** * Create a unix timestamp with a specified offset from now. * @@ -220,7 +227,6 @@ { return time() + get_offset_sec($offset_str) * $factor; } - /** * Truncate string if it is longer than the allowed length. @@ -252,7 +258,6 @@ return $str; } - /** * Get all keys from array (recursive). * @@ -276,7 +281,6 @@ return $keys; } - /** * Remove all non-ascii and non-word chars except ., -, _ */ @@ -285,7 +289,6 @@ $allowed = 'a-z0-9\_\-' . (!$css_id ? '\.' : ''); return preg_replace("/[^$allowed]/i", $replace_with, $str); } - /** * Check if a string contains only ascii characters @@ -300,7 +303,6 @@ $regexp = $control_chars ? '/[^\x00-\x7F]/' : '/[^\x20-\x7E]/'; return preg_match($regexp, $str) ? false : true; } - /** * Compose a valid representation of name and e-mail address @@ -326,7 +328,6 @@ return $email; } - /** * Format e-mail address * @@ -349,7 +350,6 @@ return $email; } - /** * Fix version number so it can be used correctly in version_compare() * @@ -365,50 +365,13 @@ $version); } - -/** - * mbstring replacement functions - */ -if (!extension_loaded('mbstring')) -{ - function mb_strlen($str) - { - return strlen($str); - } - - function mb_strtolower($str) - { - return strtolower($str); - } - - function mb_strtoupper($str) - { - return strtoupper($str); - } - - function mb_substr($str, $start, $len=null) - { - return substr($str, $start, $len); - } - - function mb_strpos($haystack, $needle, $offset=0) - { - return strpos($haystack, $needle, $offset); - } - - function mb_strrpos($haystack, $needle, $offset=0) - { - return strrpos($haystack, $needle, $offset); - } -} - /** * intl replacement functions */ if (!function_exists('idn_to_utf8')) { - function idn_to_utf8($domain, $flags=null) + function idn_to_utf8($domain) { static $idn, $loaded; @@ -430,7 +393,7 @@ if (!function_exists('idn_to_ascii')) { - function idn_to_ascii($domain, $flags=null) + function idn_to_ascii($domain) { static $idn, $loaded; @@ -490,8 +453,11 @@ */ function rcube_pear_error($err) { - error_log(sprintf("%s (%s): %s", - $err->getMessage(), - $err->getCode(), - $err->getUserinfo()), 0); + $msg = sprintf("ERROR: %s (%s)", $err->getMessage(), $err->getCode()); + + if ($info = $err->getUserinfo()) { + $msg .= ': ' . $info; + } + + error_log($msg, 0); } -- Gitblit v1.9.1