WordPress.org

Make WordPress Core


Ignore:
Timestamp:
05/08/2018 11:28:47 PM (2 years ago)
Author:
iandunn
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.
Fixes #43913.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-admin/includes/user.php

    r43175 r43183  
    610610
    611611/**
    612  * Marks a request as completed by the admin and logs the datetime.
     612 * Marks a request as completed by the admin and logs the current timestamp.
    613613 *
    614614 * @since 4.9.6
    615615 * @access private
    616616 *
    617  * @param int $request_id Request ID.
    618  * @return int|WP_Error Request ID on succes or WP_Error.
     617 * @param  int          $request_id Request ID.
     618 * @return int|WP_Error $request    Request ID on success or WP_Error.
    619619 */
    620620function _wp_privacy_completed_request( $request_id ) {
     
    626626    }
    627627
    628     update_post_meta( $request_id, '_wp_user_request_confirmed_timestamp', time() );
     628    update_post_meta( $request_id, '_wp_user_request_completed_timestamp', time() );
    629629
    630630    $request = wp_update_post( array(
    631631        'ID'          => $request_id,
    632         'post_status' => 'request-confirmed',
     632        'post_status' => 'request-completed',
    633633    ) );
    634634
Note: See TracChangeset for help on using the changeset viewer.