[wp-trac] [WordPress Trac] #44133: Should the Data Export indicate when we have no information on the user

WordPress Trac noreply at wordpress.org
Wed May 30 19:04:36 UTC 2018


#44133: Should the Data Export indicate when we have no information on the user
-------------------------------------------------+-------------------------
 Reporter:  garrett-eclipse                      |       Owner:  (none)
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  4.9.7
Component:  Privacy                              |     Version:  4.9.6
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-screenshots needs-     |     Focuses:
  refresh                                        |  administration
-------------------------------------------------+-------------------------
Changes (by desrosj):

 * keywords:  has-patch has-screenshots => has-patch has-screenshots needs-
     refresh


Comment:

 +1 for not generating and sending a ZIP when no data is found for a user.

 @birgire the patch looks good overall and just needs a refresh. Here is
 some general feedback.

 - Can we change the `wp_privacy_send_personal_data_export_email()`
 documentation to make note that an email is sent to a user if no data is
 found.
 - `wp_privacy_personal_data_email_content` `bool` type for the new
 parameter is misaligned.
 -  A `@since 4.9.7` tag should be added to the
 `wp_privacy_send_personal_data_export_email()` function and
 `wp_privacy_personal_data_email_content` filter noting the introduction of
 the new `$has_data_export` parameter.

 Also, is there any reason a plugin would want to override the value of
 `$has_data_export`? If so, a filter could be added there.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/44133#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list