Changeset 58935 for trunk/src/wp-includes/class-wp-dependencies.php

Timestamp:

08/25/2024 11:47:01 PM (10 months ago)

Author:

peterwilsoncc

Message:

Script Loader: Refactor Etag generation for concatenated assets.

Move Etag HTTP header generation in load-scripts.php and load-styles.php to WP_Dependencies.

Introduces the method WP_Dependencies::get_etag() and associated unit tests.

Follow up to [57943].

Props vrajadas, martinkrcho, mukesh27.
Fixes #61485.

File:

• trunk/src/wp-includes/class-wp-dependencies.php (modified) (1 diff)

trunk/src/wp-includes/class-wp-dependencies.php

@@ -491 +491 @@
         return true;
     }
+
+    /**
+     * Get etag header for cache validation.
+     *
+     * @since 6.7.0
+     *
+     * @global string $wp_version The WordPress version string.
+     *
+     * @param string[] $load Array of script or style handles to load.
+     * @return string Etag header.
+     */
+    public function get_etag( $load ) {
+        /*
+         * Note: wp_get_wp_version() is not used here, as this file can be included
+         * via wp-admin/load-scripts.php or wp-admin/load-styles.php, in which case
+         * wp-includes/functions.php is not loaded.
+         */
+        global $wp_version;
+
+        $etag = "WP:{$wp_version};";
+
+        foreach ( $load as $handle ) {
+            if ( ! array_key_exists( $handle, $this->registered ) ) {
+                continue;
+            }
+
+            $ver   = $this->registered[ $handle ]->ver ?? $wp_version;
+            $etag .= "{$handle}:{$ver};";
+        }
+
+        /*
+         * This is not intended to be cryptographically secure, just a fast way to get
+         * a fixed length string based on the script versions. As this file does not
+         * load the full WordPress environment, it is not possible to use the salted
+         * wp_hash() function.
+         */
+        return 'W/"' . md5( $etag ) . '"';
+    }
 }