[wp-trac] [WordPress Trac] #46303: Update wp_privacy_send_personal_data_export_email to provide the same filters as _wp_privacy_send_erasure_fulfillment_notification (was: Create wp_privacy_personal_data_email_subject to match user_erasure_complete_email_subject for privacy export emails)

WordPress Trac noreply at wordpress.org
Tue Mar 26 10:16:41 UTC 2019


#46303: Update wp_privacy_send_personal_data_export_email to provide the same
filters as _wp_privacy_send_erasure_fulfillment_notification
-------------------------------------------------+-------------------------
 Reporter:  garrett-eclipse                      |       Owner:  garrett-
                                                 |  eclipse
     Type:  enhancement                          |      Status:  assigned
 Priority:  normal                               |   Milestone:  5.3
Component:  Privacy                              |     Version:  4.9.6
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch needs-testing needs-unit-  |     Focuses:
  tests                                          |
-------------------------------------------------+-------------------------
Changes (by garrett-eclipse):

 * status:  new => assigned
 * focuses:  privacy =>
 * component:  Administration => Privacy
 * owner:  (none) => garrett-eclipse
 * milestone:  Awaiting Review => 5.3
 * keywords:  has-patch => has-patch needs-testing needs-unit-tests


Comment:

 Thanks for the refresh @thakkarhardik and the feedback @birgire while
 working through this and comparing with the
 `_wp_privacy_send_erasure_fulfillment_notification` function I found
 several inconsistencies and decided to address them in
 [https://core.trac.wordpress.org/attachment/ticket/46303/46303.3.diff
 46303.3.diff].

 1. I've introduced the `wp_privacy_personal_data_email_to` filter to match
 `user_erasure_fulfillment_email_to`.
 2. I've setup the $email_data array and used it in the new subject filter
 as well as the content filter.
 3. Introducing the `wp_privacy_personal_data_email_subject` that
 @thakkarhardik worked on I added the $site_name and $email_data.

 I've moved this to the 5.3.0 milestone and versioned as such. Please test
 when you have a chance and we'll work on unit tests.

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


More information about the wp-trac mailing list