[wp-trac] [WordPress Trac] #14997: Single filter for all WP_Query clauses

WordPress Trac wp-trac at lists.automattic.com
Wed Oct 13 11:13:10 UTC 2010


#14997: Single filter for all WP_Query clauses
-------------------------+--------------------------------------------------
 Reporter:  scribu       |        Owner:          
     Type:  enhancement  |       Status:  reopened
 Priority:  normal       |    Milestone:  3.1     
Component:  Query        |      Version:          
 Severity:  normal       |   Resolution:          
 Keywords:  has-patch    |  
-------------------------+--------------------------------------------------

Comment(by scribu):

 > Code should not do notices. So I fixed the code.

 Code should not do notices, with default theme and no plugins.

 For example, if a function expects 3 parameters, but ''non-core'' code
 passes only 2, it will trigger an error (warning or notice, doesn't
 matter). That's ok.

 > 1. Should the SQL query be syntactically verified?

 No.

 > 2. Should not be cared about notices any longer? (that would be new to
 me)

 See above.

 > 3. Should an error be raised if a filter unsets/removes a clause part?

 Maybe.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/14997#comment:20>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list