[wp-trac] [WordPress Trac] #43973: Email user once removal request completed - GDPR
WordPress Trac
noreply at wordpress.org
Thu May 10 18:49:46 UTC 2018
#43973: Email user once removal request completed - GDPR
-------------------------------------------------+-------------------------
Reporter: garrett-eclipse | Owner: (none)
Type: defect (bug) | Status: new
Priority: normal | Milestone: 4.9.6
Component: General | Version: trunk
Severity: normal | Resolution:
Keywords: gdpr needs-design has-patch needs- | Focuses:
testing |
-------------------------------------------------+-------------------------
Comment (by allendav):
@desrosj - this works nicely - I tried it with and without a published
privacy policy and got both texts. I also tried successive ones and
verified only the first erase for a given request generated an email.
I think we should hook this differently though. The
wp_privacy_personal_data_erased do_action you've added to
wp_ajax_wp_privacy_erase_personal_data will actually fire on every eraser
and every page.
Instead, I recommend you 1) add the do_action to
wp_privacy_process_personal_data_erasure_page in user.php instead (e.g.
near the call to _wp_privacy_completed_request) and 2) don't bother with
passing the $response to the action - the $request_id should be
sufficient.
Lastly, there is inconsistent alignment in the email data array
construction that i think violates WP coding standards, but otherwise this
is good to go from my perspective.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/43973#comment:11>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list