[wp-trac] [WordPress Trac] #44141: Privacy: Don't replace comment author URL with anything

WordPress Trac noreply at wordpress.org
Thu May 24 15:21:28 UTC 2018


#44141: Privacy: Don't replace comment author URL with anything
--------------------------------------+----------------------
 Reporter:  johnbillion               |       Owner:  (none)
     Type:  defect (bug)              |      Status:  new
 Priority:  normal                    |   Milestone:  4.9.7
Component:  Comments                  |     Version:  4.9.6
 Severity:  normal                    |  Resolution:
 Keywords:  has-patch has-unit-tests  |     Focuses:  privacy
--------------------------------------+----------------------
Changes (by desrosj):

 * keywords:  has-patch needs-unit-tests => has-patch has-unit-tests
 * milestone:  4.9.8 => 4.9.7


Comment:

 In [attachment:44141.diff]:

 - Add `@group privacy` to the tests in `Tests_Comments` that test the
 anonymization of comment data.
 - Adds translator context to the `Anonymous` string that replaces the
 commenter's name. This could be important for some languages.
 - Continue passing the comment author's URL through
 `wp_privacy_anonymize_data()` so that plugins and themes can filter the
 URL if desired.
 - Adjust tests for anonymizing comment author URLs and emails.
 - Add a `_wp_anonymized` comment meta value of `true` when a comment is
 successfully anonymized.
 - Change `wp_privacy_anonymize_data()` to return an empty string by
 default for both emails & URLs.

 The main change here is that a comment meta value is added flagging a
 comment as anonymized. It allows the URL to be empty but it's still
 obvious that there was potentially data there that was redacted. The
 `_wp_anonymized` key could also contain an array of the fields that did
 contain data before being erased if knowing that is important.

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


More information about the wp-trac mailing list