[wp-trac] [WordPress Trac] #44721: The privacy data erase fulfillment email should be in the user's language
WordPress Trac
noreply at wordpress.org
Fri Mar 15 22:40:27 UTC 2019
#44721: The privacy data erase fulfillment email should be in the user's language
-------------------------------------------------+-------------------------
Reporter: desrosj | Owner: desrosj
Type: defect (bug) | Status: assigned
Priority: normal | Milestone: 5.2
Component: I18N | Version: 4.9.6
Severity: normal | Resolution:
Keywords: has-patch has-unit-tests needs- | Focuses: privacy
testing |
-------------------------------------------------+-------------------------
Comment (by birgire):
Great, thanks for the tests @desrosj
They run successfully on my test install.
From a quick skimming I noticed that in the tests in
[attachment:"44721.3.diff"], there's the {{{'erase_personal_data'}}}
action name instead of the usual {{{'remove_personal_data'}}} one.
Example:
{{{
$request_id = wp_create_user_request( 'export-user-not-
registered at example.com', 'erase_personal_data' );
}}}
and perhaps in this context, the email could be like {{{erasure-user-not-
registered at example.com}}}.
I always have to remind myself that the {{{erase_}}} prefix is sometimes
{{{remove_}}} :-)
PS: I noticed that there's no erasure action check in
{{{_wp_privacy_send_erasure_fulfillment_notification()}}}, so I think any
kind of a user request, would go through with
{{{_wp_privacy_send_erasure_fulfillment_notification()}}}. Even this one:
{{{
$request_id = wp_create_user_request( 'export-user-not-
registered at example.com', 'any-kind-of-action-here' );
}}}
might go through?
This could be investigated in another ticket if needed.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/44721#comment:20>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list