[wp-trac] Re: [WordPress Trac] #7890: Internal server error when submitting comment without name or email

[WordPress Trac]

#7890: Internal server error when submitting comment without name or email
---------------------------------------+------------------------------------
 Reporter:  gadlen                     |        Owner:  Libor Jelinek
     Type:  defect                     |       Status:  assigned     
 Priority:  normal                     |    Milestone:  2.7          
Component:  Comments                   |      Version:  2.6.1        
 Severity:  normal                     |   Resolution:               
 Keywords:  2nd-opinion, dev-feedback  |  
---------------------------------------+------------------------------------
Comment (by DD32):

 > DD32: 200 isn't a correct response for an error message?
 > Uhh, no offense, but...

 My point was, That the HTTP status explains ''to the browser'' OR ''to the
 crawler'' that the request was not 100% correct. A 404 is sent for pages
 which do not exist AND a helpful message is sent for the human, A 403 is
 sent for authentication failures AND a helpful message is sent for humans.

 A 200 "All OK" response with a error message is ''OK for a human'' but
 you're telling the browser, ''this request has completed successfully, no
 problems'', Now that might not seem like a great problem, and it isnt, Its
 just returning the correct status, for the correct operations.

 How does a 404 error differ from a input validation error? They're both
 exactly the same thing, The client has inputted an incorrect piece of
 data, whether its come from a form, or the URL doesnt matter.

 The much better way would be for the theme to handle this stuff correctly,
 ie. redirect back to commenting form with the form pre-filled with the
 right details, and a note to fill in the email/name.

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