Changeset 56224

Timestamp:

07/13/2023 12:56:19 AM (19 months ago)

Author:

peterwilsoncc

Message:

General: Update Opera browser sniff follow UA string changes.

Since switching to the Blink engine, Opera browsers contain both the strings Chrome and OPR\. This change relocates the Opera test to take place before the Chrome test to ensure the correct browser is detected.

Props brasofilo, desrosj, costdev, mukesh27, swissspidy.
Fixes #46132.

File:

• trunk/src/wp-includes/vars.php (modified) (2 diffs)

trunk/src/wp-includes/vars.php

@@ -70 +70 @@
     } elseif ( str_contains( $_SERVER['HTTP_USER_AGENT'], 'Edg' ) ) {
         $is_edge = true;
+    } elseif ( str_contains( $_SERVER['HTTP_USER_AGENT'], 'Opera' ) || str_contains( $_SERVER['HTTP_USER_AGENT'], 'OPR/' ) ) {
+        $is_opera = true;
     } elseif ( stripos( $_SERVER['HTTP_USER_AGENT'], 'chrome' ) !== false ) {
         if ( stripos( $_SERVER['HTTP_USER_AGENT'], 'chromeframe' ) !== false ) {
@@ -96 +98 @@
     } elseif ( str_contains( $_SERVER['HTTP_USER_AGENT'], 'Gecko' ) ) {
         $is_gecko = true;
-    } elseif ( str_contains( $_SERVER['HTTP_USER_AGENT'], 'Opera' ) ) {
-        $is_opera = true;
     } elseif ( str_contains( $_SERVER['HTTP_USER_AGENT'], 'Nav' ) && str_contains( $_SERVER['HTTP_USER_AGENT'], 'Mozilla/4.' ) ) {
         $is_NS4 = true;