Changeset 56596

Timestamp:

09/16/2023 06:48:59 AM (13 months ago)

Author:

SergeyBiryukov

Message:

Coding Standards: Remove extra parentheses in a few str_contains() conditionals.

Follow-up to [55988].

Props Cybr.
See #58831.

Location:

trunk/src

Files:

• wp-includes/formatting.php (modified) (1 diff)
• wp-includes/vars.php (modified) (1 diff)
• wp-mail.php (modified) (1 diff)

trunk/src/wp-includes/formatting.php

@@ -4523 +4523 @@
     }
 
-    if ( ( str_contains( $url, '[' ) ) || ( str_contains( $url, ']' ) ) ) {
+    if ( str_contains( $url, '[' ) || str_contains( $url, ']' ) ) {
 
         $parsed = wp_parse_url( $url );

trunk/src/wp-includes/vars.php

@@ -92 +92 @@
     } elseif ( stripos( $_SERVER['HTTP_USER_AGENT'], 'safari' ) !== false ) {
         $is_safari = true;
-    } elseif ( ( str_contains( $_SERVER['HTTP_USER_AGENT'], 'MSIE' ) || str_contains( $_SERVER['HTTP_USER_AGENT'], 'Trident' ) ) && str_contains( $_SERVER['HTTP_USER_AGENT'], 'Win' ) ) {
+    } elseif ( ( str_contains( $_SERVER['HTTP_USER_AGENT'], 'MSIE' ) || str_contains( $_SERVER['HTTP_USER_AGENT'], 'Trident' ) )
+        && str_contains( $_SERVER['HTTP_USER_AGENT'], 'Win' )
+    ) {
         $is_winIE = true;
     } elseif ( str_contains( $_SERVER['HTTP_USER_AGENT'], 'MSIE' ) && str_contains( $_SERVER['HTTP_USER_AGENT'], 'Mac' ) ) {

trunk/src/wp-mail.php

@@ -108 +108 @@
                 $content_transfer_encoding = $content_transfer_encoding[0];
             }
-            if ( ( 'multipart/alternative' === $content_type ) && ( str_contains( $line, 'boundary="' ) ) && ( '' === $boundary ) ) {
+            if ( 'multipart/alternative' === $content_type && str_contains( $line, 'boundary="' ) && '' === $boundary ) {
                 $boundary = trim( $line );
                 $boundary = explode( '"', $boundary );