Changeset 57291

Timestamp:

01/16/2024 04:06:41 PM (11 months ago)

Author:

desrosj

Message:

Twenty Seventeen: Revert usage of str_contains() in theme files.

The theme supports WordPress 4.7 or later, while the polyfill for str_contains() only exists in WordPress 5.9 or later.

Follow-up to [55988].

Merges [57275] to the 6.4 branch.

Props poena, SergeyBiryukov.
See #60241.

Location:

branches/6.4

Files:

• . (modified) (1 prop)
• src/wp-content/themes/twentyseventeen/inc/icon-functions.php (modified) (1 diff)
• src/wp-content/themes/twentyseventeen/template-parts/post/content-audio.php (modified) (1 diff)
• src/wp-content/themes/twentyseventeen/template-parts/post/content-video.php (modified) (1 diff)

branches/6.4/src/wp-content/themes/twentyseventeen/inc/icon-functions.php

@@ -131 +131 @@
     if ( 'social' === $args->theme_location ) {
         foreach ( $social_icons as $attr => $value ) {
-            if ( str_contains( $item_output, $attr ) ) {
+            if ( false !== strpos( $item_output, $attr ) ) {
                 $item_output = str_replace( $args->link_after, '</span>' . twentyseventeen_get_svg( array( 'icon' => esc_attr( $value ) ) ), $item_output );
             }

branches/6.4/src/wp-content/themes/twentyseventeen/template-parts/post/content-audio.php

@@ -47 +47 @@
 
         // Only get audio from the content if a playlist isn't present.
-    if ( ! str_contains( $content, 'wp-playlist-script' ) ) {
+    if ( false === strpos( $content, 'wp-playlist-script' ) ) {
         $audio = get_media_embedded_in_content( $content, array( 'audio' ) );
     }

branches/6.4/src/wp-content/themes/twentyseventeen/template-parts/post/content-video.php

@@ -47 +47 @@
 
         // Only get video from the content if a playlist isn't present.
-    if ( ! str_contains( $content, 'wp-playlist-script' ) ) {
+    if ( false === strpos( $content, 'wp-playlist-script' ) ) {
         $video = get_media_embedded_in_content( $content, array( 'video', 'object', 'embed', 'iframe' ) );
     }