From 81581b537af2d6db045723269e0215dac5aef607 Mon Sep 17 00:00:00 2001 From: Sergey Biryukov Date: Wed, 27 Nov 2019 23:53:31 +0000 Subject: [PATCH] Plugins: Correct default value of `$replacement` parameter in `do_action_deprecated()` and `apply_filters_deprecated()`. This addresses an inconsistency with `_deprecated_hook()`, which uses `is_null()` to check if `$replacement` was provided, however the previous default value was `false`. Props shaampk1, felipeelia. Fixes #48817. git-svn-id: https://develop.svn.wordpress.org/trunk@46792 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-includes/functions.php | 4 ++-- src/wp-includes/plugin.php | 10 +++++----- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/src/wp-includes/functions.php b/src/wp-includes/functions.php index 3ac199e1f0..6102da94f2 100644 --- a/src/wp-includes/functions.php +++ b/src/wp-includes/functions.php @@ -4923,8 +4923,8 @@ function _deprecated_argument( $function, $version, $message = null ) { * * @param string $hook The hook that was used. * @param string $version The version of WordPress that deprecated the hook. - * @param string $replacement Optional. The hook that should have been used. - * @param string $message Optional. A message regarding the change. + * @param string $replacement Optional. The hook that should have been used. Default null. + * @param string $message Optional. A message regarding the change. Default null. */ function _deprecated_hook( $hook, $version, $replacement = null, $message = null ) { /** diff --git a/src/wp-includes/plugin.php b/src/wp-includes/plugin.php index 97a1ffd588..66d2379655 100644 --- a/src/wp-includes/plugin.php +++ b/src/wp-includes/plugin.php @@ -620,10 +620,10 @@ function remove_all_actions( $tag, $priority = false ) { * @param string $tag The name of the filter hook. * @param array $args Array of additional function arguments to be passed to apply_filters(). * @param string $version The version of WordPress that deprecated the hook. - * @param string $replacement Optional. The hook that should have been used. Default false. + * @param string $replacement Optional. The hook that should have been used. Default null. * @param string $message Optional. A message regarding the change. Default null. */ -function apply_filters_deprecated( $tag, $args, $version, $replacement = false, $message = null ) { +function apply_filters_deprecated( $tag, $args, $version, $replacement = null, $message = null ) { if ( ! has_filter( $tag ) ) { return $args[0]; } @@ -647,10 +647,10 @@ function apply_filters_deprecated( $tag, $args, $version, $replacement = false, * @param string $tag The name of the action hook. * @param array $args Array of additional function arguments to be passed to do_action(). * @param string $version The version of WordPress that deprecated the hook. - * @param string $replacement Optional. The hook that should have been used. - * @param string $message Optional. A message regarding the change. + * @param string $replacement Optional. The hook that should have been used. Default null. + * @param string $message Optional. A message regarding the change. Default null. */ -function do_action_deprecated( $tag, $args, $version, $replacement = false, $message = null ) { +function do_action_deprecated( $tag, $args, $version, $replacement = null, $message = null ) { if ( ! has_action( $tag ) ) { return; }