[wp-trac] [WordPress Trac] #28603: Improve "previously approved comment" test for logged-in users

WordPress Trac noreply at wordpress.org
Thu Sep 29 04:41:09 UTC 2016


#28603: Improve "previously approved comment" test for logged-in users
----------------------------------------+------------------
 Reporter:  nevermoor                   |       Owner:
     Type:  enhancement                 |      Status:  new
 Priority:  normal                      |   Milestone:  4.7
Component:  Comments                    |     Version:  4.0
 Severity:  normal                      |  Resolution:
 Keywords:  has-patch needs-unit-tests  |     Focuses:
----------------------------------------+------------------
Changes (by rachelbaker):

 * keywords:  has-patch dev-feedback => has-patch needs-unit-tests
 * milestone:  Future Release => 4.7


Comment:

 In [attachment:28603.5.diff] I took @jorbin's advice and used a
 `get_user_by()` conditional to avoid adding the evil 8th parameter to
 `check_comment()`.  I added one unit test, more would be nice.

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


More information about the wp-trac mailing list