[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