You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by lu...@apache.org on 2008/08/07 21:40:14 UTC

svn commit: r683690 - /myfaces/core/trunk_1.2.x/impl/src/main/java/org/apache/myfaces/application/jsp/JspViewHandlerImpl.java

Author: lu4242
Date: Thu Aug  7 12:40:14 2008
New Revision: 683690

URL: http://svn.apache.org/viewvc?rev=683690&view=rev
Log:
MYFACES-1848 JSP regular exceptions causes lack page output

Modified:
    myfaces/core/trunk_1.2.x/impl/src/main/java/org/apache/myfaces/application/jsp/JspViewHandlerImpl.java

Modified: myfaces/core/trunk_1.2.x/impl/src/main/java/org/apache/myfaces/application/jsp/JspViewHandlerImpl.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk_1.2.x/impl/src/main/java/org/apache/myfaces/application/jsp/JspViewHandlerImpl.java?rev=683690&r1=683689&r2=683690&view=diff
==============================================================================
--- myfaces/core/trunk_1.2.x/impl/src/main/java/org/apache/myfaces/application/jsp/JspViewHandlerImpl.java (original)
+++ myfaces/core/trunk_1.2.x/impl/src/main/java/org/apache/myfaces/application/jsp/JspViewHandlerImpl.java Thu Aug  7 12:40:14 2008
@@ -478,8 +478,14 @@
         ViewResponseWrapper wrappedResponse = new ViewResponseWrapper((HttpServletResponse) response);
 
         externalContext.setResponse(wrappedResponse);
-        externalContext.dispatch(viewId);
-        externalContext.setResponse(response);
+        try
+        {
+            externalContext.dispatch(viewId);
+        }
+        finally
+        {
+            externalContext.setResponse(response);
+        }
 
         boolean errorResponse = wrappedResponse.getStatus() < 200 || wrappedResponse.getStatus() > 299;
         if (errorResponse)