Ticket #44644: 44644.diff

src/wp-admin/includes/file.php

@@ -2526 +2526 @@
                 if ( is_wp_error( $mail_success ) ) {
                         wp_send_json_error( $mail_success->get_error_message() );
                 }
+
+                // Update the request to completed state when the export email is sent.
+                _wp_privacy_completed_request( $request_id );
         } else {
                 // Modify the response to include the URL of the export file so the browser can fetch it.
                 $export_file_url = get_post_meta( $request_id, '_export_file_url', true );
@@ -2534 +2537 @@
                 }
         }
 
-        // Update the request to completed state.
-        _wp_privacy_completed_request( $request_id );
-
         return $response;
 }

tests/phpunit/tests/privacy/wpPrivacyProcessPersonalDataExportPage.php

@@ -624 +624 @@
                                 true,
                                 'last',
                         ),
+                        // Leave the request as pending for the last exporter on the last page, without email. 
+                        // This check was updated to account for admin vs user export; we don't want to mark the request as completed without the email as that indicate it's an admin download.
                         array(
-                                'request-completed',
+                                'request-pending',
                                 'last',
                                 'last',
                                 'last',