Author: jbq
Date: Fri Mar  9 13:41:23 2007
New Revision: 516562

URL: http://svn.apache.org/viewvc?view=rev&rev=516562
Log:
WICKET-313 Use standard exception handling in AjaxRequestTarget

Do not set SC_OK status code here, SC_INTERNAL_ERROR should not be there in the
first place.  The problem is fixed properly in revision 516552 in
BufferedHttpServletResponse

Modified:
    
incubator/wicket/branches/wicket-1.x/wicket/src/main/java/wicket/protocol/http/WebResponse.java

Modified: 
incubator/wicket/branches/wicket-1.x/wicket/src/main/java/wicket/protocol/http/WebResponse.java
URL: 
http://svn.apache.org/viewvc/incubator/wicket/branches/wicket-1.x/wicket/src/main/java/wicket/protocol/http/WebResponse.java?view=diff&rev=516562&r1=516561&r2=516562
==============================================================================
--- 
incubator/wicket/branches/wicket-1.x/wicket/src/main/java/wicket/protocol/http/WebResponse.java
 (original)
+++ 
incubator/wicket/branches/wicket-1.x/wicket/src/main/java/wicket/protocol/http/WebResponse.java
 Fri Mar  9 13:41:23 2007
@@ -192,10 +192,11 @@
 
                                        if (isAjax())
                                        {
-                                               // Reset the HTTP response 
status code to 200, otherwise
-                                               // wicket-ajax.js does not 
process the Ajax-Location
-                                               // header
-                                               
httpServletResponse.setStatus(HttpServletResponse.SC_OK);
+                                               /*
+                                                * By reaching this point, make 
sure the HTTP response
+                                                * status code is set to 200, 
otherwise wicket-ajax.js
+                                                * will not process the 
Ajax-Location header
+                                                */
                                                
httpServletResponse.addHeader("Ajax-Location", url);
 
                                                // safari chokes on empty 
response. but perhaps this is


Reply via email to