Changeset 44785 for trunk/src/wp-includes/deprecated.php

Timestamp:

03/01/2019 08:57:26 PM (7 years ago)

Author:

jorbin

Message:

Replace usages of basename() with wp_basename() in order to support multibyte filenames

This is focused on the pieces of code that touch media files and the tests that support them. basename isn't multibyte compatible out of the box. See ​http://php.net/basename and ​https://bugs.php.net/bug.php?id=62119.

See #43170.
Props Viper007Bond.

File:

• trunk/src/wp-includes/deprecated.php (modified) (3 diffs)

trunk/src/wp-includes/deprecated.php

@@ -1879 +1879 @@
         // We have a thumbnail desired, specified and existing
 
-        $src_file = basename($src);
+        $src_file = wp_basename($src);
     } elseif ( wp_attachment_is_image( $post->ID ) ) {
         // We have an image without a thumbnail
@@ -1889 +1889 @@
 
         $icon_dir = apply_filters( 'icon_dir', get_template_directory() . '/images' );
-        $src_file = $icon_dir . '/' . basename($src);
+        $src_file = $icon_dir . '/' . wp_basename($src);
     }
 
@@ -3059 +3059 @@
 function get_theme_data( $theme_file ) {
     _deprecated_function( __FUNCTION__, '3.4.0', 'wp_get_theme()' );
-    $theme = new WP_Theme( basename( dirname( $theme_file ) ), dirname( dirname( $theme_file ) ) );
+    $theme = new WP_Theme( wp_basename( dirname( $theme_file ) ), dirname( dirname( $theme_file ) ) );
 
     $theme_data = array(