Changeset 43228 for trunk

Timestamp:

05/10/2018 08:23:33 PM (8 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 );