Changeset 43187 for branches/4.9/src/wp-admin/includes/user.php

Timestamp:

05/09/2018 02:30:59 PM (7 years ago)

Author:

SergeyBiryukov

Message:

Privacy: Mark processed requests as completed instead of confirmed.

r43008 refactored the request flow to make several improvements, but accidentally marked completed requests as confirmed. This commit restores the intended statuses, so that the data and corresponding UI reflect reality.

Props allendav, birgire.
Merges [43183] to the 4.9 branch.
Fixes #43913.

Location:

branches/4.9

Files:

• . (modified) (1 prop)
• src/wp-admin/includes/user.php (modified) (2 diffs)

branches/4.9/src/wp-admin/includes/user.php

@@ -569 +569 @@
 
 /**
- * Marks a request as completed by the admin and logs the datetime.
+ * Marks a request as completed by the admin and logs the current timestamp.
  *
  * @since 4.9.6
  * @access private
  *
- * @param int $request_id Request ID.
- * @return int|WP_Error Request ID on succes or WP_Error.
+ * @param  int          $request_id Request ID.
+ * @return int|WP_Error $request    Request ID on success or WP_Error.
  */
 function _wp_privacy_completed_request( $request_id ) {
@@ -585 +585 @@
     }
 
-    update_post_meta( $request_id, '_wp_user_request_confirmed_timestamp', time() );
+    update_post_meta( $request_id, '_wp_user_request_completed_timestamp', time() );
 
     $request = wp_update_post( array(
         'ID'          => $request_id,
-        'post_status' => 'request-confirmed',
+        'post_status' => 'request-completed',
     ) );