From e86a21bd83a0ae6cadfe9c919582951f306d3b64 Mon Sep 17 00:00:00 2001 From: alecpl <alec@alec.pl> Date: Fri, 06 Jan 2012 05:55:07 -0500 Subject: [PATCH] - Fix typo in timezone handling, more exception catching --- program/include/rcube_browser.php | 37 +++++++++++++++++++------------------ 1 files changed, 19 insertions(+), 18 deletions(-) diff --git a/program/include/rcube_browser.php b/program/include/rcube_browser.php index 23ef615..859f367 100644 --- a/program/include/rcube_browser.php +++ b/program/include/rcube_browser.php @@ -4,8 +4,8 @@ +-----------------------------------------------------------------------+ | program/include/rcube_browser.php | | | - | This file is part of the RoundCube Webmail client | - | Copyright (C) 2007-2009, RoundCube Dev. - Switzerland | + | This file is part of the Roundcube Webmail client | + | Copyright (C) 2007-2009, The Roundcube Dev Team | | Licensed under the GNU GPL | | | | PURPOSE: | @@ -30,25 +30,25 @@ { function __construct() { - $HTTP_USER_AGENT = $_SERVER['HTTP_USER_AGENT']; + $HTTP_USER_AGENT = strtolower($_SERVER['HTTP_USER_AGENT']); $this->ver = 0; - $this->win = stristr($HTTP_USER_AGENT, 'win'); - $this->mac = stristr($HTTP_USER_AGENT, 'mac'); - $this->linux = stristr($HTTP_USER_AGENT, 'linux'); - $this->unix = stristr($HTTP_USER_AGENT, 'unix'); + $this->win = strpos($HTTP_USER_AGENT, 'win') != false; + $this->mac = strpos($HTTP_USER_AGENT, 'mac') != false; + $this->linux = strpos($HTTP_USER_AGENT, 'linux') != false; + $this->unix = strpos($HTTP_USER_AGENT, 'unix') != false; - $this->opera = stristr($HTTP_USER_AGENT, 'opera'); - $this->ns4 = stristr($HTTP_USER_AGENT, 'mozilla/4') && !stristr($HTTP_USER_AGENT, 'msie'); - $this->ns = ($this->ns4 || stristr($HTTP_USER_AGENT, 'netscape')); - $this->ie = stristr($HTTP_USER_AGENT, 'compatible; msie') && !$this->opera; - $this->mz = stristr($HTTP_USER_AGENT, 'mozilla/5'); - $this->chrome = stristr($HTTP_USER_AGENT, 'chrome'); - $this->khtml = stristr($HTTP_USER_AGENT, 'khtml'); - $this->safari = !$this->chrome && ($this->khtml || stristr($HTTP_USER_AGENT, 'safari')); + $this->opera = strpos($HTTP_USER_AGENT, 'opera') !== false; + $this->ns4 = strpos($HTTP_USER_AGENT, 'mozilla/4') !== false && strpos($HTTP_USER_AGENT, 'msie') === false; + $this->ns = ($this->ns4 || strpos($HTTP_USER_AGENT, 'netscape') !== false); + $this->ie = !$this->opera && strpos($HTTP_USER_AGENT, 'compatible; msie') !== false; + $this->mz = !$this->ie && strpos($HTTP_USER_AGENT, 'mozilla/5') !== false; + $this->chrome = strpos($HTTP_USER_AGENT, 'chrome') !== false; + $this->khtml = strpos($HTTP_USER_AGENT, 'khtml') !== false; + $this->safari = !$this->chrome && ($this->khtml || strpos($HTTP_USER_AGENT, 'safari') !== false); if ($this->ns || $this->chrome) { - $test = preg_match('/(mozilla|chrome)\/([0-9.]+)/i', $HTTP_USER_AGENT, $regs); + $test = preg_match('/(mozilla|chrome)\/([0-9.]+)/', $HTTP_USER_AGENT, $regs); $this->ver = $test ? (float)$regs[2] : 0; } else if ($this->mz) { @@ -56,11 +56,11 @@ $this->ver = $test ? (float)$regs[1] : 0; } else if ($this->ie || $this->opera) { - $test = preg_match('/(msie|opera) ([0-9.]+)/i', $HTTP_USER_AGENT, $regs); + $test = preg_match('/(msie|opera) ([0-9.]+)/', $HTTP_USER_AGENT, $regs); $this->ver = $test ? (float)$regs[2] : 0; } - if (preg_match('/ ([a-z]{2})-([a-z]{2})/i', $HTTP_USER_AGENT, $regs)) + if (preg_match('/ ([a-z]{2})-([a-z]{2})/', $HTTP_USER_AGENT, $regs)) $this->lang = $regs[1]; else $this->lang = 'en'; @@ -68,6 +68,7 @@ $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->imgdata = !$this->ie; } } -- Gitblit v1.9.1