[wp-trac] [WordPress Trac] #43443: Add a method for confirmation of requests for deleting or anonymizing of personal data

WordPress Trac noreply at wordpress.org
Wed Mar 7 11:35:58 UTC 2018


#43443: Add a method for confirmation of requests for deleting or anonymizing of
personal data
-------------------------------------------------+-------------------------
 Reporter:  azaozz                               |       Owner:  mikejolley
     Type:  enhancement                          |      Status:  assigned
 Priority:  normal                               |   Milestone:  5.0
Component:  General                              |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  gdpr has-patch dev-feedback needs-   |     Focuses:
  testing                                        |
-------------------------------------------------+-------------------------

Comment (by ocean90):

 Replying to [comment:3 azaozz]:
 > In [changeset:"42791"]:
 > {{{
 > #!CommitTicketReference repository="" revision="42791"
 > Add a method to confirm user requests by email. First run.
 >
 > Props mikejolley.
 > See #43443.
 > }}}

 * `hash()` may not be available, see #29518.
 * `@return WP_ERROR|bool` should be `@return WP_Error|bool`.
 * There should be no empty line between `@param` and `@return` tags.
 * All the `WP_Error` messages should end with a full stop.
 * In `check_confirm_account_action_key` the function should bail directly
 if one of the parameters is empty. So move that `return new WP_Error(
 'invalid_key', __( 'Invalid key' ) );` line up to avoid one huge if
 condition.
 * All the new actions have no `@since` tag.
 * Why are none of the functions prefixed with `wp_`?

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


More information about the wp-trac mailing list