[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