Changeset 55888 for trunk/src/wp-includes/script-loader.php

Timestamp:

06/07/2023 06:54:18 AM (3 years ago)

Author:

spacedmonkey

Message:

Script Loader: Improve performance of wp_maybe_inline_styles function.

The wp_maybe_inline_styles function is called twice on the average page load. On it's second run however, it did not check to see if the style had already been processed on the first run. This resulted in calling filesize and get_file_contents unnecessarily, which was bad for performance. Now, the loop around the queued styles, checks to see if the source is set to false, meaning it has already been processed. This change also replaces calls to filesize with the core function wp_filesize, which improves extensibility.

Props spacedmonkey, flixos90, peterwilsoncc, joemcgill.
Fixes #58394.

File:

• trunk/src/wp-includes/script-loader.php (modified) (1 diff)

trunk/src/wp-includes/script-loader.php

@@ -2873 +2873 @@
     // Build an array of styles that have a path defined.
     foreach ( $wp_styles->queue as $handle ) {
-        if ( wp_styles()->get_data( $handle, 'path' ) && file_exists( $wp_styles->registered[ $handle ]->extra['path'] ) ) {
+        $src  = $wp_styles->registered[ $handle ]->src;
+        $path = wp_styles()->get_data( $handle, 'path' );
+        if ( $path && $src ) {
+            $size = wp_filesize( $path );
+            if ( ! $size ) {
+                continue;
+            }
             $styles[] = array(
                 'handle' => $handle,
-                'src'    => $wp_styles->registered[ $handle ]->src,
-                'path'   => $wp_styles->registered[ $handle ]->extra['path'],
-                'size'   => filesize( $wp_styles->registered[ $handle ]->extra['path'] ),
+                'src'    => $src,
+                'path'   => $path,
+                'size'   => $size,
             );
         }