WordPress.org

Make WordPress Core

Changeset 43187


Ignore:
Timestamp:
05/09/2018 02:30:59 PM (3 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:
2 edited
1 copied

Legend:

Unmodified
Added
Removed
  • branches/4.9

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

    r43176 r43187  
    569569
    570570/**
    571  * Marks a request as completed by the admin and logs the datetime.
     571 * Marks a request as completed by the admin and logs the current timestamp.
    572572 *
    573573 * @since 4.9.6
    574574 * @access private
    575575 *
    576  * @param int $request_id Request ID.
    577  * @return int|WP_Error Request ID on succes or WP_Error.
     576 * @param  int          $request_id Request ID.
     577 * @return int|WP_Error $request    Request ID on success or WP_Error.
    578578 */
    579579function _wp_privacy_completed_request( $request_id ) {
     
    585585    }
    586586
    587     update_post_meta( $request_id, '_wp_user_request_confirmed_timestamp', time() );
     587    update_post_meta( $request_id, '_wp_user_request_completed_timestamp', time() );
    588588
    589589    $request = wp_update_post( array(
    590590        'ID'          => $request_id,
    591         'post_status' => 'request-confirmed',
     591        'post_status' => 'request-completed',
    592592    ) );
    593593
Note: See TracChangeset for help on using the changeset viewer.