Changeset 660 for trunk/lib/App.inc.php

Timestamp:

Jan 28, 2019 10:51:50 PM (5 years ago)

Author:

anonymous

Message:

Reduce timezone support to simply setting defaults for user, php, and mysql (all UTC), just to ensure consistency

File:

• trunk/lib/App.inc.php (modified) (4 diffs)

trunk/lib/App.inc.php

@@ -119 +119 @@
         'sql_time_format' => '%k:%i',
 
+        // Timezone support. No codebase apps currently support switching timezones, but we explicitly set these so they're consistent.
+        'user_timezone' => 'UTC',
+        'php_timezone' => 'UTC',
+        'mysql_timezone' => 'UTC',
+
         // Use php sessions?
         'enable_session' => false,
@@ -370 +375 @@
         }
 
-        // If the 'timezone' parameter is not set, check for a tz cookie.
-        if (!$this->getParam('timezone') && isset($_COOKIE['tz']) && '' != $_COOKIE['tz']) {
-            if (preg_match('!^[A-Z_/-]{3,}$!i', $_COOKIE['tz'])) {
-                // Valid: tz cookie has a timezone name, like "UTC" or "America/Mexico_City".
-                $this->setParam(array('timezone' => $_COOKIE['tz']));
-            } else if (is_numeric($_COOKIE['tz']) && $_COOKIE['tz'] > -12 && $_COOKIE['tz'] < 14) { // https://en.wikipedia.org/wiki/List_of_UTC_time_offsets#UTC+14:00,_M%E2%80%A0
-                // tz cookie has a timezone offset, like "-6" (assume UTC).
-                $tz = timezone_name_from_abbr('', $_COOKIE['tz'] * 3600, 0);
-                if ($tz && preg_match('!^[A-Z_/-]{3,}$!i', $tz)) {
-                    // Valid.
-                    $this->setParam(array('timezone' => $tz));
-                } else {
-                    $this->logMsg(sprintf('Failed to convert UTC offset to timezone: %s', $_COOKIE['tz']), LOG_NOTICE, __FILE__, __LINE__);
-                }
-            } else {
-                $this->logMsg(sprintf('Invalid timezone cookie value: %s', $_COOKIE['tz']), LOG_NOTICE, __FILE__, __LINE__);
-            }
-        }
-        if ($this->getParam('timezone')) {
-            // Set timezone of the user.
-            if (date_default_timezone_set($this->getParam('timezone'))) {
-                $this->logMsg(sprintf('Using timezone: %s', $this->getParam('timezone')), LOG_DEBUG, __FILE__, __LINE__);
-            } else {
-                // Failed: unset the timezone parameter so it isn't used to set the database timezone.
-                $this->setParam(array('timezone' => null));
-                $this->logMsg(sprintf('Failed to set timezone: %s', $this->getParam('timezone')), LOG_NOTICE, __FILE__, __LINE__);
-            }
-        } else {
-            $this->logMsg(sprintf('Using server timezone: %s', date_default_timezone_get()), LOG_DEBUG, __FILE__, __LINE__);
+        // Server timezone used internally by PHP.
+        if ($this->getParam('php_timezone')) {
+            $this->setTimezone($this->getParam('php_timezone'));
         }
 
@@ -446 +425 @@
                 'db_debug' => $this->getParam('db_debug'),
                 'db_die_on_failure' => $this->getParam('db_die_on_failure'),
+                'timezone' => $this->getParam('mysql_timezone'),
             ));
 
@@ -1706 +1686 @@
         return $ret;
     }
+
+    /*
+    * Set timezone used internally by PHP.
+    *
+    * @access   public
+    * @param    string  $tz     Timezone, e.g., America/Mexico_City
+    * @return
+    * @author   Quinn Comendant <quinn@strangecode.com>
+    * @since    28 Jan 2019 16:38:38
+    */
+    public function setTimezone($tz)
+    {
+        $this->setTimezone();
+        // Set timezone for PHP.
+        if (date_default_timezone_set($tz)) {
+            $this->logMsg(sprintf('Using php timezone: %s', $tz), LOG_DEBUG, __FILE__, __LINE__);
+        } else {
+            // Failed!
+            $this->logMsg(sprintf('Failed to set php timezone: %s', $tz), LOG_WARNING, __FILE__, __LINE__);
+        }
+    }
 } // End.