Changeset 5989

Timestamp:

10/02/2017 07:44:28 PM (8 years ago)

Author:

coffee2code

Message:

Browse Happy API: Add support for BlackBerry Browser as a browser.

Location:

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

Files:

• parse.php (modified) (2 diffs)
• tests/phpunit/tests/browse-happy.php (modified) (2 diffs)

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

@@ -91 +91 @@
     // No indentifiers provided
     if ( empty( $result['name'] ) ) {
-        $data['name'] = 'unknown';
+        if ( 'BlackBerry' === $mobile_device ) {
+            $data['name'] = 'BlackBerry Browser';
+        } else {
+            $data['name'] = 'unknown';
+        }
     }
     // Opera
@@ -138 +142 @@
             } elseif ( 'Fire OS' === $data['platform'] ) {
                 $data['name'] = 'Kindle Browser';
+            } elseif ( false !== strpos( $user_agent, 'BlackBerry' ) || false !== strpos( $user_agent, 'BB10' ) ) {
+                $data['name']   = 'BlackBerry Browser';
+                $data['mobile'] = true;
+
+                if ( false !== stripos( $user_agent, 'BB10' ) ) {
+                    $result['version'][ $key ] = '';
+                    $version = '';
+                }
             } else {
                 $data['name'] = 'Mobile Safari';

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

@@ -68 +68 @@
             ],
 
+            // BlackBerry Browser
+
+            [
+                'Mozilla/5.0 (BB10; Touch) AppleWebKit/537.35+ (KHTML, like Gecko) Version/10.3.2.2876 Mobile Safari/537.35+',
+                'Mobile BlackBerry Browser',
+            ],
+            [
+                'BlackBerry; U; Blackberry 9900; en',
+                'Mobile BlackBerry Browser',
+            ],
+            [
+                'BlackBerry8520/5.0.0.592 Profile/MIDP-2.1 Configuration/CLDC-1.1 VendorID/168',
+                'Mobile BlackBerry Browser',
+            ],
+            [
+                'Mozilla/5.0 (BlackBerry; U; BlackBerry 9720; en) AppleWebKit/534.11+ (KHTML, like Gecko) Version/7.1.0.1121 Mobile Safari/534.11+',
+                'Mobile BlackBerry Browser 7.1.0.1121',
+            ],
+
             // Camino
 
@@ -514 +533 @@
                 'Mozilla/5.0 (iPhone; CPU iPhone OS 9_3_2 like Mac OS X) AppleWebKit/601.1.46 (KHTML, like Gecko) Mobile/13F69 [FBAN/FBIOS;FBAV/59.0.0.51.142;FBBV/33266808;FBRV/0;FBDV/iPhone7,1;FBMD/iPhone;FBSN/iPhone OS;FBSV/9.3.2;FBSS/3;FBCR/Telkomsel;FBID/phone;FBLC/en_US;FBOP/5] evaliant',
                 'iPhone unknown', // Actually: iPhone Facebook App 59.0.0.51.142
-            ],
-            [
-                'BlackBerry; U; Blackberry 9900; en',
-                'Mobile unknown',
-            ],
-            [
-                'BlackBerry8520/5.0.0.592 Profile/MIDP-2.1 Configuration/CLDC-1.1 VendorID/168',
-                'Mobile unknown',
             ],
             [