[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