Changeset 5988 for sites/trunk/api.wordpress.org/public_html/core/browse-happy/1.0/parse.php

Timestamp:

10/02/2017 07:39:32 PM (6 years ago)

Author:

coffee2code

Message:

Browse Happy API: Improve indentification of BlackBerry, Nokia, and Sony Ericsson devices generically as mobile devices if platform not otherwise known.

File:

• sites/trunk/api.wordpress.org/public_html/core/browse-happy/1.0/parse.php (modified) (3 diffs)

sites/trunk/api.wordpress.org/public_html/core/browse-happy/1.0/parse.php

@@ -38 +38 @@
         'mobile'          => false,
     );
+    $mobile_device = '';
 
     if ( preg_match(
@@ -54 +55 @@
             $data['platform'] = 'Android';
         }
-    } elseif ( 'Windows Phone' === $data['platform'] ) {
+    }
+    // Normalize Windows Phone OS name when "OS" is omitted.
+    elseif ( 'Windows Phone' === $data['platform'] ) {
         $data['platform'] = 'Windows Phone OS';
     }
-
-    if ( in_array( $data['platform'], array( 'Android', 'Fire OS', 'iPad', 'iPhone', 'PlayBook', 'RIM Tablet OS', 'Windows Phone OS' ) ) ) {
+    // Generically detect some mobile devices.
+    elseif (
+        ! $data['platform']
+    &&
+        preg_match( '/BlackBerry|Nokia|SonyEricsson/', $user_agent, $matches )
+    ) {
+        $data['platform'] = 'Mobile';
+        $mobile_device    = $matches[0];
+    }
+
+    // Flag known mobile platforms as mobile.
+    if ( in_array( $data['platform'], array( 'Android', 'Fire OS', 'iPad', 'iPhone', 'Mobile', 'PlayBook', 'RIM Tablet OS', 'Windows Phone OS' ) ) ) {
         $data['mobile'] = true;
     }
@@ -146 +159 @@
             $data['name'] = 'unknown';
             $result['version'][ $key ] = '';
+            $version = '';
         }
         $data['version'] = $result['version'][ $key ];