Ticket #4336: mythweb-iphone.diff

mythplugins/mythweb/includes/skin.php

@@ -17 +17 @@
     if (isMobileUser()) {
     // Browser is mobile but does it accept HTML?
     // @TODO Need to fail more gracefully...
-        $_SESSION['tmpl'] = 'wap';
+        $_SESSION['tmpl'] = getMobileUserAgent();
     // Make sure the skin is set to the appropriate phone-template type
         $_SESSION['skin'] = $_SESSION['tmpl'];
         define('skin', $_SESSION['skin']);

mythplugins/mythweb/includes/mobile.php

@@ -41 +41 @@
  * @return true or false
  */
 function isMobileUser() {
-  return (getScreenSize() === false) ? false : true;
+  return (getMobileUserAgent() === null) ? false : true;
 }
 
+function getMobiles() {
 
-/**
- * Return the screen size of the mobile user agent. This can be used to do very
- * detailed modifications to the web site's layout for the mobile user agents.
- * If the user is not regocnized to be a mobile device then return false. If the
- * user is a known mobile device but the screen size is unknown then return
- * an empty array. Mobile users are detected based on $_SERVER['HTTP_USER_AGENT']
- * and a list of known mobile user agent identification strings.
- *
- * @return array(width, height) or false
- */
-function getScreenSize() {
-  static $isChecked = false;
-  static $screen = false;
-
-  if (!$isChecked) {
     /*
      * The $mobiles array contains a list of known mobile phone user agent
      * identification strings. Each key of the array is a substring of the
@@ -75 +61 @@
      * an empty array or approximate dimensions.
      */
     $mobiles = array(
+                /* Apple's iPhone */
+                    'iPhone'     => array('width' => 320, 'height' => 480),
                 /* Phones using the Series 60 platform, e.g. Nokia 3650 and 6600. */
                     'Series 60'  => array('width' => 176, 'height' => 208),
                     'Series60'   => array('width' => 176, 'height' => 208),
@@ -141 +129 @@
                      //'Opera' => array('width' => 176, 'height' => 208)
              );
 
+   return $mobiles;
+}
+
+
+/**
+ * Return the mobile user agent. If the user agent is not regocnized to be
+ * a known mobile device then return null. Mobile users are detected based
+  * on $_SERVER['HTTP_USER_AGENT'] and a list of known mobile user agent
+  * identification strings.
+  *
+  * @return array(width, height) if known or null
+  */
+function getMobileUserAgent() {
+  static $isChecked = false;
+  static $userAgent = null;
+
+  $mobiles = getMobiles();
+
+  if (!$isChecked) {
+
     /* Scan through $mobiles and try to find matching user agent. */
     foreach (array_keys($mobiles) as $needle) {
       /* Check if user agent matches this screen size key. */
       if (strpos($_SERVER['HTTP_USER_AGENT'], $needle) !== false) {
-        $screen = $mobiles[$needle];
+        $userAgent = $needle;
         break;
       }
     }
 
-// If the user agent was not in our list, check if the terminal accepts WAP.
-    if ($screen === false) {
-      if (browserAcceptsMediaType(array('WAP'))) {
-        $screen = array();
-      }
+    /* If the user agent was not in our list, check media type */
+    if ($userAgent === null) {
+
+      if (browserAcceptsMediaType(array('text/html', '\*/\*')))
+        if (browserAcceptsMediaType(array('WAP')))
+            $userAgent = 'wap';
+
     }
 
     $isChecked = true;
   }
-  return $screen;
+
+  return $userAgent;
 }
 
 
@@ -182 +193 @@
 
 
 /**
+ * Return the screen size of the mobile user agent. This can be used to do very
+ * detailed modifications to the web site's layout for the mobile user agents.
+ * Meant to be called if isMobileUser() returns true.
+ *
+ * @return array(width, height) if known or null
+ */
+function getScreenSize() {
+
+  $screen = null;
+  $mobiles = getMobiles();
+  $userAgent = getMobileUserAgent();
+
+  $screen = $mobiles[$userAgent];
+
+  return $screen;
+}
+
+
+/**
  * Return the width of the screen in pixels. Meant to be called if isMobileUser()
  * returns true. See also getScreensize().
  *