Changeset 56043

Timestamp:

06/26/2023 08:55:04 PM (3 years 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 @@
         }
 
+        $previous_status = $post->post_status;
+
         /**
          * 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, $previous_status );
 
         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, $previous_status );
+
+        add_post_meta( $post_id, '_wp_trash_meta_status', $previous_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, $previous_status );
 
         return $post;
@@ -3657 +3665 @@
          *
          * @since 4.9.0
-         * @since 5.6.0 The `$previous_status` parameter was added.
+         * @since 5.6.0 Added the `$previous_status` parameter.
          *
          * @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 Added the `$previous_status` parameter.
          *
          * @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 Added the `$previous_status` parameter.
          *
          * @param int    $post_id         Post ID.