WordPress.org

Make WordPress Core

Ticket #44644: 44644.2.diff

File 44644.2.diff, 1.8 KB (added by garrett-eclipse, 8 months ago)

Updated long comment to break onto multiple lines

  • src/wp-admin/includes/file.php

     
    25502550                if ( is_wp_error( $mail_success ) ) {
    25512551                        wp_send_json_error( $mail_success->get_error_message() );
    25522552                }
     2553
     2554                // Update the request to completed state when the export email is sent.
     2555                _wp_privacy_completed_request( $request_id );
    25532556        } else {
    25542557                // Modify the response to include the URL of the export file so the browser can fetch it.
    25552558                $export_file_url = get_post_meta( $request_id, '_export_file_url', true );
     
    25582561                }
    25592562        }
    25602563
    2561         // Update the request to completed state.
    2562         _wp_privacy_completed_request( $request_id );
    2563 
    25642564        return $response;
    25652565}
  • tests/phpunit/tests/privacy/wpPrivacyProcessPersonalDataExportPage.php

     
    615615         */
    616616        public function data_export_page_status_transitions() {
    617617                return array(
    618                         // Mark the request as completed for the last exporter on the last page, with and without email.
     618                        // Mark the request as completed for the last exporter on the last page, with email.
    619619                        array(
    620620                                'request-completed',
    621621                                'last',
     
    624624                                true,
    625625                                'last',
    626626                        ),
     627                        // Leave the request as pending for the last exporter on the last page, without email.
     628                        // This check was updated to account for admin vs user export.
     629                        // Don't mark the request as completed when it's an admin download.
    627630                        array(
    628                                 'request-completed',
     631                                'request-pending',
    629632                                'last',
    630633                                'last',
    631634                                'last',