Changeset 57740 for trunk/src/wp-includes/fonts.php

Timestamp:

02/29/2024 10:15:20 AM (2 years ago)

Author:

youknowriad

Message:

Editor: Prevent infinite loops when filtering the font library folder.

Changing the font library is something we expect hosts to perform.
It's important that we make this filter as seemless as possible.
This commit prevents a potential infinite loop caused by calling wp_get_upload_dir() within the font_dir filter.

Props mmaattiiaass, ironprogrammer, costdev, swissspidy.
Fixes #60652.

File:

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

trunk/src/wp-includes/fonts.php

@@ -97 +97 @@
  * @since 6.5.0
  *
- * @param array $defaults {
+ * @return array $defaults {
  *     Array of information about the upload directory.
  *
@@ -107 +107 @@
  *     @type string|false $error   False or error message.
  * }
- * @return array $defaults {
- *     Array of information about the upload directory.
- *
- *     @type string       $path    Base directory and subdirectory or full path to the fonts upload directory.
- *     @type string       $url     Base URL and subdirectory or absolute URL to the fonts upload directory.
- *     @type string       $subdir  Subdirectory
- *     @type string       $basedir Path without subdir.
- *     @type string       $baseurl URL path without subdir.
- *     @type string|false $error   False or error message.
- * }
- */
-function wp_get_font_dir( $defaults = array() ) {
+ */
+function wp_get_font_dir() {
         $site_path = '';
         if ( is_multisite() && ! ( is_main_network() && is_main_site() ) ) {
@@ -124 +114 @@
         }
 
-        // Sets the defaults.
-        $defaults['path']    = path_join( WP_CONTENT_DIR, 'fonts' ) . $site_path;
-        $defaults['url']     = untrailingslashit( content_url( 'fonts' ) ) . $site_path;
-        $defaults['subdir']  = '';
-        $defaults['basedir'] = path_join( WP_CONTENT_DIR, 'fonts' ) . $site_path;
-        $defaults['baseurl'] = untrailingslashit( content_url( 'fonts' ) ) . $site_path;
-        $defaults['error']   = false;
+        $defaults = array(
+                'path'    => path_join( WP_CONTENT_DIR, 'fonts' ) . $site_path,
+                'url'     => untrailingslashit( content_url( 'fonts' ) ) . $site_path,
+                'subdir'  => '',
+                'basedir' => path_join( WP_CONTENT_DIR, 'fonts' ) . $site_path,
+                'baseurl' => untrailingslashit( content_url( 'fonts' ) ) . $site_path,
+                'error'   => false,
+        );
 
         /**