Changeset 57880

Timestamp:

03/26/2024 09:25:20 AM (4 months ago)

Author:

swissspidy

Message:

Editor: Relocate font files uploads to the uploads directory.

Relocate the upload of font files uploaded via the Font Library feature to the wp-content/uploads/fonts (or multisite equivalent) directory.

This accounts for immutable file systems in which directories are unable to be created within wp-content and deploy processes which require special consideration of the uploads directory to ensure it remains persistent between deploys.

Reviewed by davidbaumwald.
Merges [57878] to the 6.5 branch.

Props azaozz, burnuser, cbirdsong, christopherplus, costdev, davidbaumwald, desrosj, elrae, euthelup, get_dave, grantmkin, hellofromtonya, janthiel, jazzs3quence, johnbillion, jorbin, justlevine, kraftner, matveb, mcsf, mmaattiiaass, nico23, peterwilsoncc, priethor, rmccue, samuelsidler, swissspidy, youknowriad.
Fixes #60845.

Location:

branches/6.5

Files:

• . (modified) (1 prop)
• src/wp-includes/fonts.php (modified) (1 diff)
• tests/phpunit/tests/fonts/font-library/wpFontsDir.php (modified) (3 diffs)

branches/6.5/src/wp-includes/fonts.php

@@ -168 +168 @@
     }
 
-    $site_path = '';
-    if ( is_multisite() && ! ( is_main_network() && is_main_site() ) ) {
-        $site_path = '/sites/' . get_current_blog_id();
-    }
-
     $font_dir = array(
-        'path'    => path_join( WP_CONTENT_DIR, 'fonts' ) . $site_path,
-        'url'     => untrailingslashit( content_url( 'fonts' ) ) . $site_path,
+        'path'    => ,
+        'url'     => untrailingslashit( ,
         'subdir'  => '',
-        'basedir' => path_join( WP_CONTENT_DIR, 'fonts' ) . $site_path,
-        'baseurl' => untrailingslashit( content_url( 'fonts' ) ) . $site_path,
+        'basedir' => ,
+        'baseurl' => untrailingslashit( ,
         'error'   => false,
     );

branches/6.5/tests/phpunit/tests/fonts/font-library/wpFontsDir.php

@@ -16 +16 @@
     public static function set_up_before_class() {
         parent::set_up_before_class();
+
 
         static::$dir_defaults = array(
-            'path'    => path_join( WP_CONTENT_DIR, 'fonts' ),
-            'url'     => content_url( 'fonts' ),
+            'path'    => ,
+            'url'     => ,
             'subdir'  => '',
-            'basedir' => path_join( WP_CONTENT_DIR, 'fonts' ),
-            'baseurl' => content_url( 'fonts' ),
+            'basedir' => ,
+            'baseurl' => ,
             'error'   => false,
         );
     }
 
+
+
+
     public function test_fonts_dir() {
         $font_dir = wp_get_font_dir();
@@ -33 +37 @@
     }
 
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
     public function test_fonts_dir_with_filter() {
         // Define a callback function to pass to the filter.
@@ -72 +107 @@
 
     /**
+
+
      * @ticket 60652
      */