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

WordPress Trac noreply at wordpress.org
Wed Oct 15 00:10:54 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    |     Focuses:
-------------------------+------------------------------

Comment (by dancameron):

 12668-comments-query.2.patch doesn't do much more than change the
 formatting of the code that @mordauk submitted, in order to conform to
 whatever standards are applied to the rest of that query method.

 I really want to change the get_approved_comments function, I don't know
 how to describe my angst towards it. Firstly, it's not even being used by
 anything in core than comments-popup (which brings back memories);
 Secondly, I feel that it should be using WP_Comment_Query...I'll create a
 totally separate patch.

 Replying to [comment:52 mordauk]:

 > I also went ahead and separated each modified file into separate
 patches. Not sure if that makes it easier or more difficult to review :)

 In order to keep track it's gonna be a real hassle if the diff/patch files
 get changed. So let's try to keep them as is (although I wouldn't be
 apposed to naming the files based off the file names).

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


More information about the wp-trac mailing list