From 0c53d8ef73149b21d9657d819c383dcd2cffd02e Mon Sep 17 00:00:00 2001 From: Sergey Biryukov Date: Tue, 14 Mar 2023 16:55:19 +0000 Subject: [PATCH] Docs: Fix typo in `_validate_cache_id()` description. Includes: * Capitalizing "ID" in a consistent way. * Expanding the comment on not using `filter_var()`. * Adding a `@covers` tag for the function in unit tests. * Minor tweak to the `_doing_it_wrong()` message. Follow-up to [53818], [55543]. See #57593. Built from https://develop.svn.wordpress.org/trunk@55549 git-svn-id: http://core.svn.wordpress.org/trunk@55061 1a063a9b-81f0-0310-95a4-ce76da25c4cd --- wp-includes/class-wp-object-cache.php | 2 +- wp-includes/functions.php | 20 ++++++++++++-------- wp-includes/version.php | 2 +- 3 files changed, 14 insertions(+), 10 deletions(-) diff --git a/wp-includes/class-wp-object-cache.php b/wp-includes/class-wp-object-cache.php index cd84b17753..8136ea3ad9 100644 --- a/wp-includes/class-wp-object-cache.php +++ b/wp-includes/class-wp-object-cache.php @@ -156,7 +156,7 @@ class WP_Object_Cache { $message = is_string( $key ) ? __( 'Cache key must not be an empty string.' ) /* translators: %s: The type of the given cache key. */ - : sprintf( __( 'Cache key must be integer or non-empty string, %s given.' ), $type ); + : sprintf( __( 'Cache key must be an integer or a non-empty string, %s given.' ), $type ); _doing_it_wrong( sprintf( '%s::%s', __CLASS__, debug_backtrace( DEBUG_BACKTRACE_IGNORE_ARGS, 2 )[1]['function'] ), diff --git a/wp-includes/functions.php b/wp-includes/functions.php index e3e281b1d6..2112323704 100644 --- a/wp-includes/functions.php +++ b/wp-includes/functions.php @@ -7034,24 +7034,28 @@ function _get_non_cached_ids( $object_ids, $cache_key ) { } /** - * Checks whether the given cache ID is either an integer or iterger-like strings. - * Both `16` and `"16"` are considered valid, other numeric types and numeric - * strings (`16.3` and `"16.3"`) are considered invalid. + * Checks whether the given cache ID is either an integer or an integer-like string. + * + * Both `16` and `"16"` are considered valid, other numeric types and numeric strings + * (`16.3` and `"16.3"`) are considered invalid. * * @since 6.3.0 * - * @param mixed $object_id The cache id to validate. - * @return bool Whether the given $object_id is a valid cache id. + * @param mixed $object_id The cache ID to validate. + * @return bool Whether the given $object_id is a valid cache ID. */ function _validate_cache_id( $object_id ) { - // Unfortunately filter_var() is considered an optional extension + /* + * filter_var() could be used here, but the `filter` PHP extension + * is considered optional and may not be available. + */ if ( is_int( $object_id ) || ( is_string( $object_id ) && (string) (int) $object_id === $object_id ) ) { return true; } - /* translators: %s: The type of the given object id. */ - $message = sprintf( __( 'Object id must be integer, %s given.' ), gettype( $object_id ) ); + /* translators: %s: The type of the given object ID. */ + $message = sprintf( __( 'Object ID must be an integer, %s given.' ), gettype( $object_id ) ); _doing_it_wrong( '_get_non_cached_ids', $message, '6.3.0' ); return false; diff --git a/wp-includes/version.php b/wp-includes/version.php index a3e12ee04b..c2ab5f266f 100644 --- a/wp-includes/version.php +++ b/wp-includes/version.php @@ -16,7 +16,7 @@ * * @global string $wp_version */ -$wp_version = '6.3-alpha-55547'; +$wp_version = '6.3-alpha-55549'; /** * Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.