From f821fa413ece0949a2593455300e0d8f32198ff4 Mon Sep 17 00:00:00 2001 From: Sergey Biryukov Date: Mon, 10 Oct 2022 18:22:11 +0000 Subject: [PATCH] Cache API: Introduce `wp_cache_supports()` function. WordPress has recently introduced a variety of caching API improvements: * `wp_cache_add_multiple()` * `wp_cache_set_multiple()` * `wp_cache_get_multiple()` * `wp_cache_delete_multiple()` * `wp_cache_flush_runtime()` * `wp_cache_flush_group()` Although WordPress core provides a compatibility layer if these functions are missing from third-party object cache implementations, there should be a method of checking whether the cache backend supports a particular feature. This commit aims to improve developer experience by allowing third-party object cache plugins to declare a `wp_cache_supports()` function and correctly list their supported features: * `add_multiple` * `set_multiple` * `get_multiple` * `delete_multiple` * `flush_runtime` * `flush_group` Note: The `wp_cache_supports()` function replaces and supersedes the `wp_cache_supports_group_flush()` function added earlier. Follow-up to [47938], [47944], [52700], [52703], [52706], [52708], [53763], [53767], [54423]. Props johnjamesjacoby, tillkruess, spacedmonkey, SergeyBiryukov. Fixes #56605. Built from https://develop.svn.wordpress.org/trunk@54448 git-svn-id: http://core.svn.wordpress.org/trunk@54007 1a063a9b-81f0-0310-95a4-ce76da25c4cd --- wp-includes/cache-compat.php | 29 ++++++++++++++++++++--------- wp-includes/cache.php | 26 +++++++++++++++++++------- wp-includes/version.php | 2 +- 3 files changed, 40 insertions(+), 17 deletions(-) diff --git a/wp-includes/cache-compat.php b/wp-includes/cache-compat.php index 367cff05ef..eafe37cf00 100644 --- a/wp-includes/cache-compat.php +++ b/wp-includes/cache-compat.php @@ -138,7 +138,17 @@ if ( ! function_exists( 'wp_cache_flush_runtime' ) ) : * @return bool True on success, false on failure. */ function wp_cache_flush_runtime() { - return wp_using_ext_object_cache() ? false : wp_cache_flush(); + if ( ! wp_cache_supports( 'flush_runtime' ) ) { + _doing_it_wrong( + __FUNCTION__, + __( 'Your object cache implementation does not support flushing the in-memory runtime cache.' ), + '6.1.0' + ); + + return false; + } + + return wp_cache_flush(); } endif; @@ -147,7 +157,7 @@ if ( ! function_exists( 'wp_cache_flush_group' ) ) : * Removes all cache items in a group, if the object cache implementation supports it. * * Before calling this function, always check for group flushing support using the - * `wp_cache_supports_group_flush()` function. + * `wp_cache_supports( 'flush_group' )` function. * * @since 6.1.0 * @@ -160,7 +170,7 @@ if ( ! function_exists( 'wp_cache_flush_group' ) ) : function wp_cache_flush_group( $group ) { global $wp_object_cache; - if ( ! wp_cache_supports_group_flush() ) { + if ( ! wp_cache_supports( 'flush_group' ) ) { _doing_it_wrong( __FUNCTION__, __( 'Your object cache implementation does not support flushing individual groups.' ), @@ -174,17 +184,18 @@ if ( ! function_exists( 'wp_cache_flush_group' ) ) : } endif; -if ( ! function_exists( 'wp_cache_supports_group_flush' ) ) : +if ( ! function_exists( 'wp_cache_supports' ) ) : /** - * Determines whether the object cache implementation supports flushing individual cache groups. + * Determines whether the object cache implementation supports a particular feature. * * @since 6.1.0 * - * @see WP_Object_Cache::flush_group() - * - * @return bool True if group flushing is supported, false otherwise. + * @param string $feature Name of the feature to check for. Possible values include: + * 'add_multiple', 'set_multiple', 'get_multiple', 'delete_multiple', + * 'flush_runtime', 'flush_group'. + * @return bool True if the feature is supported, false otherwise. */ - function wp_cache_supports_group_flush() { + function wp_cache_supports( $feature ) { return false; } endif; diff --git a/wp-includes/cache.php b/wp-includes/cache.php index dd7038d339..e7fe7b87a8 100644 --- a/wp-includes/cache.php +++ b/wp-includes/cache.php @@ -285,7 +285,7 @@ function wp_cache_flush_runtime() { * Removes all cache items in a group, if the object cache implementation supports it. * * Before calling this function, always check for group flushing support using the - * `wp_cache_supports_group_flush()` function. + * `wp_cache_supports( 'flush_group' )` function. * * @since 6.1.0 * @@ -302,16 +302,28 @@ function wp_cache_flush_group( $group ) { } /** - * Determines whether the object cache implementation supports flushing individual cache groups. + * Determines whether the object cache implementation supports a particular feature. * * @since 6.1.0 * - * @see WP_Object_Cache::flush_group() - * - * @return bool True if group flushing is supported, false otherwise. + * @param string $feature Name of the feature to check for. Possible values include: + * 'add_multiple', 'set_multiple', 'get_multiple', 'delete_multiple', + * 'flush_runtime', 'flush_group'. + * @return bool True if the feature is supported, false otherwise. */ -function wp_cache_supports_group_flush() { - return true; +function wp_cache_supports( $feature ) { + switch ( $feature ) { + case 'add_multiple': + case 'set_multiple': + case 'get_multiple': + case 'delete_multiple': + case 'flush_runtime': + case 'flush_group': + return true; + + default: + return false; + } } /** diff --git a/wp-includes/version.php b/wp-includes/version.php index 7a646d1214..d9db507da6 100644 --- a/wp-includes/version.php +++ b/wp-includes/version.php @@ -16,7 +16,7 @@ * * @global string $wp_version */ -$wp_version = '6.1-beta3-54447'; +$wp_version = '6.1-beta3-54448'; /** * Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.