Aleksander Machniak
2015-10-07 a03233cebafc62ec7d88a24856b8a7b37fef4381
CS fixes
8 files modified
51 ■■■■■ changed files
program/lib/Roundcube/bootstrap.php 9 ●●●● patch | view | raw | blame | history
program/lib/Roundcube/rcube_addressbook.php 3 ●●●● patch | view | raw | blame | history
program/lib/Roundcube/rcube_browser.php 10 ●●●●● patch | view | raw | blame | history
program/lib/Roundcube/rcube_image.php patch | view | raw | blame | history
program/lib/Roundcube/rcube_imap.php 1 ●●●● patch | view | raw | blame | history
program/lib/Roundcube/rcube_mime.php 2 ●●● patch | view | raw | blame | history
program/lib/Roundcube/rcube_string_replacer.php 2 ●●●●● patch | view | raw | blame | history
program/lib/Roundcube/rcube_utils.php 24 ●●●●● patch | view | raw | blame | history
program/lib/Roundcube/bootstrap.php
@@ -254,7 +254,9 @@
        $first_part_length  = floor(($maxlength - $placeholder_length)/2);
        $second_starting_location = $length - $maxlength + $first_part_length + $placeholder_length;
        $str = mb_substr($str, 0, $first_part_length) . $placeholder . mb_substr($str, $second_starting_location);
        $prefix = mb_substr($str, 0, $first_part_length);
        $suffix = mb_substr($str, $second_starting_location);
        $str    = $prefix . $placeholder . $suffix;
    }
    return $str;
@@ -364,7 +366,8 @@
    return str_replace(
        array('-stable', '-git'),
        array('.0', '.99'),
        $version);
        $version
    );
}
/**
@@ -389,6 +392,7 @@
            catch (Exception $e) {
            }
        }
        return $domain;
    }
}
@@ -411,6 +415,7 @@
            catch (Exception $e) {
            }
        }
        return $domain;
    }
}
program/lib/Roundcube/rcube_addressbook.php
@@ -168,7 +168,6 @@
     * Set internal list page
     *
     * @param  number  Page number to list
     * @access public
     */
    function set_page($page)
    {
@@ -179,7 +178,6 @@
     * Set internal page size
     *
     * @param  number  Number of messages to display on one page
     * @access public
     */
    function set_pagesize($size)
    {
@@ -208,6 +206,7 @@
     *
     * @param array Assoziative array with data to save
     * @param boolean Attempt to fix/complete record automatically
     *
     * @return boolean True if input is valid, False if not.
     */
    public function validate(&$save_data, $autofix = false)
program/lib/Roundcube/rcube_browser.php
@@ -3,7 +3,7 @@
/**
 +-----------------------------------------------------------------------+
 | This file is part of the Roundcube Webmail client                     |
 | Copyright (C) 2007-2009, The Roundcube Dev Team                       |
 | Copyright (C) 2007-2015, The Roundcube Dev Team                       |
 |                                                                       |
 | Licensed under the GNU General Public License version 3 or            |
 | any later version with exceptions for skins & plugins.                |
@@ -54,14 +54,16 @@
            $this->ver = (float) $regs[1];
        }
        if (preg_match('/ ([a-z]{2})-([a-z]{2})/', $HTTP_USER_AGENT, $regs))
        if (preg_match('/ ([a-z]{2})-([a-z]{2})/', $HTTP_USER_AGENT, $regs)) {
            $this->lang =  $regs[1];
        else
        }
        else {
            $this->lang =  'en';
        }
        $this->dom      = $this->mz || $this->safari || ($this->ie && $this->ver>=5) || ($this->opera && $this->ver>=7);
        $this->pngalpha = $this->mz || $this->safari || ($this->ie && $this->ver>=5.5) ||
            ($this->ie && $this->ver>=5 && $this->mac) || ($this->opera && $this->ver>=7) ? true : false;
            ($this->ie && $this->ver>=5 && $this->mac) || ($this->opera && $this->ver>=7);
        $this->imgdata  = !$this->ie;
    }
}
program/lib/Roundcube/rcube_image.php
program/lib/Roundcube/rcube_imap.php
@@ -4353,7 +4353,6 @@
    /**
     * This is our own debug handler for the IMAP connection
     * @access public
     */
    public function debug_handler(&$imap, $message)
    {
program/lib/Roundcube/rcube_mime.php
@@ -290,7 +290,7 @@
    }
    /**
     * @access private
     * E-mail address list parser
     */
    private static function parse_address_list($str, $decode = true, $fallback = null)
    {
program/lib/Roundcube/rcube_string_replacer.php
@@ -59,6 +59,7 @@
     * Add a string to the internal list
     *
     * @param string String value 
     *
     * @return int Index of value for retrieval
     */
    public function add($str)
@@ -140,6 +141,7 @@
     * Callback function used to build mailto: links around e-mail strings
     *
     * @param array Matches result from preg_replace_callback
     *
     * @return int Index of saved string value
     */
    public function mailto_callback($matches)
program/lib/Roundcube/rcube_utils.php
@@ -260,9 +260,9 @@
     *
     * @return string   Field value or NULL if not available
     */
    public static function get_input_value($fname, $source, $allow_html=FALSE, $charset=NULL)
    public static function get_input_value($fname, $source, $allow_html = false, $charset = null)
    {
        $value = NULL;
        $value = null;
        if ($source == self::INPUT_GET) {
            if (isset($_GET[$fname])) {
@@ -299,7 +299,7 @@
     *
     * @return string   Parsed value
     */
    public static function parse_input_value($value, $allow_html=FALSE, $charset=NULL)
    public static function parse_input_value($value, $allow_html = false, $charset = null)
    {
        global $OUTPUT;
@@ -544,7 +544,8 @@
        }
        if (!empty($_SERVER['HTTP_X_FORWARDED_PROTO'])
            && strtolower($_SERVER['HTTP_X_FORWARDED_PROTO']) == 'https'
            && in_array($_SERVER['REMOTE_ADDR'], rcube::get_instance()->config->get('proxy_whitelist', array()))) {
            && in_array($_SERVER['REMOTE_ADDR'], rcube::get_instance()->config->get('proxy_whitelist', array()))
        ) {
            return true;
        }
        if ($port && $_SERVER['SERVER_PORT'] == $port) {
@@ -581,7 +582,8 @@
        $h = $_SESSION['storage_host'] ? $_SESSION['storage_host'] : $host;
        // %z - IMAP domain without first part, e.g. %h=imap.domain.tld, %z=domain.tld
        $z = preg_replace('/^[^\.]+\./', '', $h);
        // %s - domain name after the '@' from e-mail address provided at login screen. Returns FALSE if an invalid email is provided
        // %s - domain name after the '@' from e-mail address provided at login screen.
        //      Returns FALSE if an invalid email is provided
        if (strpos($name, '%s') !== false) {
            $user_email = self::get_input_value('_user', self::INPUT_POST);
            $user_email = self::idn_convert($user_email, true);
@@ -607,6 +609,7 @@
        if (!empty($_SERVER['HTTP_X_REAL_IP'])) {
            $remote_ip[] = 'X-Real-IP: ' . $_SERVER['HTTP_X_REAL_IP'];
        }
        // append the X-Forwarded-For header, if set
        if (!empty($_SERVER['HTTP_X_FORWARDED_FOR'])) {
            $remote_ip[] = 'X-Forwarded-For: ' . $_SERVER['HTTP_X_FORWARDED_FOR'];
@@ -704,7 +707,7 @@
     * Improved equivalent to strtotime()
     *
     * @param string $date  Date string
     * @param object DateTimeZone to use for DateTime object
     * @param DateTimeZone $timezone Timezone to use for DateTime object
     *
     * @return int Unix timestamp
     */
@@ -736,13 +739,13 @@
     * Date parsing function that turns the given value into a DateTime object
     *
     * @param string $date  Date string
     * @param object DateTimeZone to use for DateTime object
     * @param DateTimeZone $timezone Timezone to use for DateTime object
     *
     * @return object DateTime instance or false on failure
     * @return DateTime instance or false on failure
     */
    public static function anytodatetime($date, $timezone = null)
    {
        if (is_object($date) && is_a($date, 'DateTime')) {
        if ($date instanceof DateTime) {
            return $date;
        }
@@ -939,7 +942,8 @@
     *
     * @param string Haystack
     * @param string Needle
     * @return boolen True if match, False otherwise
     *
     * @return boolean True if match, False otherwise
     */
    public static function words_match($haystack, $needle)
    {