[wp-trac] [WordPress Trac] #35601: WP_Query: comment_status and ping_status

WordPress Trac noreply at wordpress.org
Tue Jan 26 02:41:35 UTC 2016


#35601: WP_Query: comment_status and ping_status
---------------------------------------------+---------------------------
 Reporter:  birgire                          |       Owner:  boonebgorges
     Type:  feature request                  |      Status:  assigned
 Priority:  normal                           |   Milestone:  4.5
Component:  Query                            |     Version:  4.4
 Severity:  normal                           |  Resolution:
 Keywords:  needs-unit-tests good-first-bug  |     Focuses:
---------------------------------------------+---------------------------
Changes (by boonebgorges):

 * keywords:  needs-patch needs-unit-tests good-first-bug => needs-unit-
     tests good-first-bug
 * owner:   => boonebgorges
 * status:  new => assigned
 * milestone:  Future Release => 4.5


Comment:

 Thanks for the patch, @birgire. There's nothing preventing people from
 storing custom comment_status and ping_status in the database, and there's
 no reason I can think of why we need to prevent querying by custom values.
 The only exception would be if we had some "magic" values - kinda like how
 `post_type=all` gets translated under the hood to all publicly queryable
 post types.

 The only kind of sanitization necessary here is against SQL injection,
 which is handled by `$wpdb->prepare()`.

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


More information about the wp-trac mailing list