Changeset 43228 for trunk

Timestamp:

05/10/2018 08:23:33 PM (7 years ago)

Author:

SergeyBiryukov

Message:

Privacy: Avoid a PHP notice in wp_ajax_wp_privacy_erase_personal_data(), make sure $eraser_key is always defined.

Props allendav.
Fixes #44040.

File:

• trunk/src/wp-admin/includes/ajax-actions.php (modified) (4 diffs)

trunk/src/wp-admin/includes/ajax-actions.php

@@ -4477 +4477 @@
     } else {
         // No exporters, so we're done.
+        $exporter_key = '';
+
         $response = array(
             'data' => array(),
@@ -4496 +4498 @@
      * @param int    $request_id      The privacy request post ID associated with this request.
      * @param bool   $send_as_email   Whether the final results of the export should be emailed to the user.
-     * @param int    $exporter_key    The key (slug) of the exporter that provided this data.
+     * @param string $exporter_key    The key (slug) of the exporter that provided this data.
      */
     $response = apply_filters( 'wp_privacy_personal_data_export_page', $response, $exporter_index, $email_address, $page, $request_id, $send_as_email, $exporter_key );
@@ -4693 +4695 @@
     } else {
         // No erasers, so we're done.
+        $eraser_key = '';
+
         $response = array(
             'items_removed'  => false,
@@ -4713 +4717 @@
      * @param int    $page            The page for this response.
      * @param int    $request_id      The privacy request post ID associated with this request.
-     * @param int    $eraser_key      The key (slug) of the eraser that provided this data.
+     * @param string $eraser_key      The key (slug) of the eraser that provided this data.
      */
     $response = apply_filters( 'wp_privacy_personal_data_erasure_page', $response, $eraser_index, $email_address, $page, $request_id, $eraser_key );