Changeset 56061 for trunk/src/wp-includes/option.php

Timestamp:

06/27/2023 10:39:55 AM (16 months ago)

Author:

spacedmonkey

Message:

Options, Meta APIs: Prime network options in a single cache call using wp_cache_get_multiple.

Tweak the wp_load_core_site_options function, to also prime network options using wp_cache_get_multiple if persistent object cache is present.

Props mukesh27, oglekler, peterwilsoncc, costdev, jeremyfelt, spacedmonkey.
Fixes #56913.

File:

• trunk/src/wp-includes/option.php (modified) (2 diffs)

trunk/src/wp-includes/option.php

@@ -362 +362 @@
 
 /**
- * Loads and caches certain often requested site options if is_multisite() and a persistent cache is not being used.
+ * Loads and primes caches of certain often requested network options if is_multisite().
  *
  * @since 3.0.0
+ * @since 6.3.0 Also prime caches for network options when persistent object cache is enabled.
  *
  * @global wpdb $wpdb WordPress database abstraction object.
  *
- * @param int $network_id Optional site ID for which to query the options. Defaults to the current site.
+ * @param int $network_id Optional. Network ID of network for which to prime network options cache. Defaults to current network.
  */
 function wp_load_core_site_options( $network_id = null ) {
     global $wpdb;
 
-    if ( ! is_multisite() || wp_using_ext_object_cache() || wp_installing() ) {
+    if ( ! is_multisite() || wp_installing() ) {
         return;
     }
@@ -382 +383 @@
 
     $core_options = array( 'site_name', 'siteurl', 'active_sitewide_plugins', '_site_transient_timeout_theme_roots', '_site_transient_theme_roots', 'site_admins', 'can_compress_scripts', 'global_terms_enabled', 'ms_files_rewriting' );
+
+    if ( wp_using_ext_object_cache() ) {
+        $cache_keys = array();
+        foreach ( $core_options as $option ) {
+            $cache_keys[] = "{$network_id}:{$option}";
+        }
+        wp_cache_get_multiple( $cache_keys, 'site-options' );
+
+        return;
+    }
 
     $core_options_in = "'" . implode( "', '", $core_options ) . "'";