Changeset 56043

Timestamp:

06/26/2023 08:55:04 PM (14 months ago)

Author:

audrasjb

Message:

Posts, Post Types: Add a $previous_status parameter to wp_trash_post() related hooks.

This adds a $previous_status parameter to the pre_trash_post, wp_trash_post, and trashed_post hooks.

Props mujuonly, mukesh27, nihar007, dhruvishah2203, SergeyBiryukov, costdev, hugod, audrasjb, oglekler.
Fixes #58392.

Location:

trunk

Files:

• src/wp-includes/post.php (modified) (6 diffs)
• tests/phpunit/tests/post/wpTrashPost.php (added)
• tests/phpunit/tests/post/wpUntrashPost.php (added)

trunk/src/wp-includes/post.php

@@ -3577 +3577 @@
     }
 
+
+
     /**
      * Filters whether a post trashing should take place.
      *
      * @since 4.9.0
-     *
-     * @param bool|null $trash Whether to go forward with trashing.
-     * @param WP_Post   $post  Post object.
+     * @since 6.3.0 Added the `$previous_status` parameter.
+     *
+     * @param bool|null $trash           Whether to go forward with trashing.
+     * @param WP_Post   $post            Post object.
+     * @param string    $previous_status The status of the post about to be trashed.
      */
-    $check = apply_filters( 'pre_trash_post', null, $post );
+    $check = apply_filters( 'pre_trash_post', null, $post );
 
     if ( null !== $check ) {
@@ -3595 +3599 @@
      *
      * @since 3.3.0
-     *
-     * @param int $post_id Post ID.
+     * @since 6.3.0 Added the `$previous_status` parameter.
+     *
+     * @param int    $post_id         Post ID.
+     * @param string $previous_status The status of the post about to be trashed.
      */
-    do_action( 'wp_trash_post', $post_id );
-
-    add_post_meta( $post_id, '_wp_trash_meta_status', $post->post_status );
+    do_action( 'wp_trash_post', $post_id );
+
+    add_post_meta( $post_id, '_wp_trash_meta_status', $p_status );
     add_post_meta( $post_id, '_wp_trash_meta_time', time() );
 
@@ -3620 +3626 @@
      *
      * @since 2.9.0
-     *
-     * @param int $post_id Post ID.
+     * @since 6.3.0 Added the `$previous_status` parameter.
+     *
+     * @param int    $post_id         Post ID.
+     * @param string $previous_status The status of the post at the point where it was trashed.
      */
-    do_action( 'trashed_post', $post_id );
+    do_action( 'trashed_post', $post_id );
 
     return $post;
@@ -3657 +3665 @@
      *
      * @since 4.9.0
-     * @since 5.6.0 The `$previous_status` parameter was added.
+     * @since 5.6.0 .
      *
      * @param bool|null $untrash         Whether to go forward with untrashing.
@@ -3672 +3680 @@
      *
      * @since 2.9.0
-     * @since 5.6.0 The `$previous_status` parameter was added.
+     * @since 5.6.0 .
      *
      * @param int    $post_id         Post ID.
@@ -3718 +3726 @@
      *
      * @since 2.9.0
-     * @since 5.6.0 The `$previous_status` parameter was added.
+     * @since 5.6.0 .
      *
      * @param int    $post_id         Post ID.