Changeset 5989

Timestamp:

10/02/2017 07:44:28 PM (9 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',
                         ],
                         [