[wp-trac] [WordPress Trac] #12668: Better support for custom comment types

WordPress Trac noreply at wordpress.org
Wed Oct 15 00:39:22 UTC 2014


#12668: Better support for custom comment types
----------------------------------------+------------------------------
 Reporter:  ptahdunbar                  |       Owner:  ptahdunbar
     Type:  enhancement                 |      Status:  reopened
 Priority:  normal                      |   Milestone:  Awaiting Review
Component:  Comments                    |     Version:
 Severity:  normal                      |  Resolution:
 Keywords:  has-patch needs-unit-tests  |     Focuses:
----------------------------------------+------------------------------

Comment (by dancameron):

 Replying to [comment:54 boonebgorges]:
 > There are pieces of this that I would be happy to help shepherd through
 the process

 Thank you!

 > ...it would be nice if they were accompanied by a succinct statement
 about what just what it is that we're trying to accomplish.

 I'll try my best to be as descriptive as possible with my patches.

 > Yes, this. Adding piecemeal support to `get_approved_comments()` is
 unappealing. We'll need unit tests for that function before rewriting its
 internals, though.

 I'd be happy to do this since I've never done WP core unit tests before
 (forgive me later).

 > (Also, in `WP_Comment_Query`, be sure to `array_map( 'esc_sql',
 $this->query_vars['type'] )`.)

 Done. Thanks, this is something I actually forgot to bring up because I
 didn't know what callback to use.

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


More information about the wp-trac mailing list