[wp-trac] [WordPress Trac] #35090: Query for comments where post isn't set

WordPress Trac noreply at wordpress.org
Sat Jan 23 13:52:28 UTC 2016


#35090: Query for comments where post isn't set
-----------------------------+---------------------------
 Reporter:  danielbachhuber  |       Owner:  boonebgorges
     Type:  defect (bug)     |      Status:  reopened
 Priority:  high             |   Milestone:  4.5
Component:  Comments         |     Version:
 Severity:  normal           |  Resolution:
 Keywords:  needs-patch      |     Focuses:
-----------------------------+---------------------------

Comment (by boonebgorges):

 Replying to [comment:7 ocean90]:
 > [36381] broke the comments list table because we're passing 0 to
 `post_id`, see [source:/trunk/src/wp-admin/includes/class-wp-comments-
 list-table.php?rev=36381&marks=43,130#L43].

 Oy.

 It's trivial to fix the comment list table. But if we're doing something
 like this in core, it's extremely likely that the same technique is used
 extensively throughough the WP universe. This almost certainly means that
 we will not be able to make the change in [36381], and if we want a way to
 query for comments with no post, we'll have to come up with some other way
 of doing it.

 I'll try to take some time to search through the plugin repo to see how
 extensive this kind of thing is. (Namely: either passing `post_id=0` to a
 comment query, or copypasta from the core list table.) In the meantime,
 @ocean90, if you think this merits a revert so that trunk is usable, let
 me know (or be my guest).

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


More information about the wp-trac mailing list