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

WordPress Trac wp-trac at lists.automattic.com
Wed Oct 13 12:57:05 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 hakre):

 Replying to [comment:20 scribu]:
 > > 2. Should not be cared about notices any longer? (that would be new to
 me)
 >
 > See above.

 Probably that question was too unconcrete, I need to ask more:

  2.1. Previous to the done commit, were filters able to break the SQL
 syntax without giving notices?

  2.2. You classified it a problem that a filter can break the SQL syntax
 without giving notcies. If so, do you think it's wished to give notices
 now when a SQL related clause filter breaks the SQL syntax?

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

  3.1. Can you name reason(s) for the cases under which circumstances an
 error should be raised?

  3.2. can you name reason(s) for the cases an error should not be raised?

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


More information about the wp-trac mailing list