[wp-trac] Re: [WordPress Trac] #5185: SQL Error on feeds for invalid posts

WordPress Trac wp-trac at lists.automattic.com
Thu Oct 18 15:07:18 GMT 2007


#5185: SQL Error on feeds for invalid posts
--------------------------------------------------+-------------------------
 Reporter:  robertaccettura                       |        Owner:  anonymous
     Type:  defect                                |       Status:  new      
 Priority:  normal                                |    Milestone:  2.4      
Component:  General                               |      Version:  2.3      
 Severity:  major                                 |   Resolution:           
 Keywords:  sql, error, feed, query.php, invalid  |  
--------------------------------------------------+-------------------------
Changes (by markjaquith):

  * milestone:  2.5 => 2.4

Comment:

 Reproduced.

 Can prevent it with a bandaid.  It serves a 404, so that's good.  What
 output do we want to send?  Is there any standard for serving 404 content
 to a feed reader?  Or is the 404 sufficient?  i.e. Can we just die with a
 human-readable error?

 Bandaid:

 wp-includes/query.php:1258

 {{{
                 if ( $this->is_comment_feed && $this->is_singular &&
 $this->have_posts() ) {
 }}}

-- 
Ticket URL: <http://trac.wordpress.org/ticket/5185#comment:3>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list