Changeset 49731 for trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-attachments-controller.php

Timestamp:

12/01/2020 08:45:43 PM (4 years ago)

Author:

johnbillion

Message:

REST API: Pass the previous state of the post as a parameter to the wp_after_insert_post hook.

This enables, for example, the previous post status to be used by this hook without the need to first capture it on an earlier hook.

This also fixes the value of the $fire_after_hooks parameter in get_default_post_to_edit() so the wp_after_insert_post action correctly fires just once on the new post screen.

Props Collizo4sky, peterwilsoncc, hellofromTonya, TimothyBlynJacobs, SergeyBiryukov

Fixes #45114

File:

• trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-attachments-controller.php (modified) (3 diffs)

trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-attachments-controller.php

@@ -192 +192 @@
         do_action( 'rest_after_insert_attachment', $attachment, $request, true );
 
-        wp_after_insert_post( $attachment, false );
+        wp_after_insert_post( $attachment, false, null );
 
         if ( defined( 'REST_REQUEST' ) && REST_REQUEST ) {
@@ -322 +322 @@
         }
 
-        $response = parent::update_item( $request );
+        $attachment_before = get_post( $request['id'] );
+        $response          = parent::update_item( $request );
 
         if ( is_wp_error( $response ) ) {
@@ -348 +349 @@
         do_action( 'rest_after_insert_attachment', $attachment, $request, false );
 
-        wp_after_insert_post( $attachment, true );
+        wp_after_insert_post( $attachment, true, $attachment_before );
 
         $response = $this->prepare_item_for_response( $attachment, $request );