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

Timestamp:

09/29/2017 04:47:56 PM (9 years ago)

Author:

coffee2code

Message:

Browse Happy API: Add support for Android Browser.

• Also fixes a misidentification of certain user-agents for Chrome on Android as being Mobile Safari.
• Also fixes misidentifications in a few existing unit tests which were actually Android Browser user-agents and not Mobile Safari or Safari.

File:

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

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

@@ -91 +91 @@
             $data['name'] = 'Microsoft Edge';
         } elseif ( $key = array_search( 'Mobile Safari', $result['name'] ) ) {
-            $data['name'] = 'Mobile Safari';
+            if ( $key2 = array_search( 'Chrome', $result['name'] ) ) {
+                $data['name'] = 'Chrome';
+                $version = $result['version'][ $key2 ];
+            } elseif ( 'Android' === $data['platform'] ) {
+                $data['name'] = 'Android Browser';
+            } else {
+                $data['name'] = 'Mobile Safari';
+            }
         // } elseif ( ( 'Android' == $data['platform'] && !($key = 0) ) || $key = array_search( 'Chrome', $result['name'] ) ) {
         } elseif ( $key = array_search( 'RockMelt', $result['name'] ) ) {
@@ -103 +110 @@
             $data['name'] = 'Kindle';
         } elseif ( $key = array_search( 'Safari', $result['name'] ) ) {
-            $data['name'] = 'Safari';
+            if ( 'Android' === $data['platform'] ) {
+                $data['name'] = 'Android Browser';
+            } else {
+                $data['name'] = 'Safari';
+            }
         } else {
             $key = 0;