[wp-trac] [WordPress Trac] #43438: Export registered user's personal data on request

WordPress Trac noreply at wordpress.org
Fri Mar 9 22:59:51 UTC 2018


#43438: Export registered user's personal data on request
----------------------------+------------------
 Reporter:  azaozz          |       Owner:
     Type:  enhancement     |      Status:  new
 Priority:  normal          |   Milestone:  5.0
Component:  General         |     Version:
 Severity:  normal          |  Resolution:
 Keywords:  gdpr has-patch  |     Focuses:
----------------------------+------------------
Changes (by allendav):

 * keywords:  gdpr needs-patch => gdpr has-patch


Comment:

 Replying to [comment:10 azaozz]:
 > Few initial points on 43438.2.diff:

 Thank you so much for the fast turn around!

 > - The admin-ajax endpoint will need a nonce check and
 `current_user_can()` check.

 Done!

 > - Think that we should have a specific DB query for this export,
 something like `SELECT * FROM {$wpdb->comments} WHERE comment_author_email
 = '[email]' LIMIT 10000`. Then can pick only the unique values from that,
 no need to repeat the same data all the time. Also, that DB query can
 possibly be more complex so we get only the unique data out of the DB.

 Done - instead of custom query, get_comments was up to the task. I set the
 limit to 100 per page - let me know if you want something higher. 10000
 seemed uncomfortably high :)

 > - It's using AJAX but overwrites the whole page. That is somewhat
 unexpected and can be confusing. IMHO better to just show it inline. Can
 move the PHP bits in the AJAX endpoint and just send the user_name or
 email to get it started.

 Yep - I didn't make any changes here - this is not meant to be the final
 UX - I'd like us to all discuss what we want to do with the gathered data
 and if name/value is going to work in general.

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


More information about the wp-trac mailing list