admin@glassfish.java.net

Shale Issue #360

From: Ken Paulsen <Ken.Paulsen_at_Sun.COM>
Date: Thu, 12 Apr 2007 13:37:50 -0700

Hi Craig,

I believe this issue still isn't resolved. I also think the fix is very
simple, shale needs to check to see if the response is complete before
doing it's processing. Here is a proposed fix:

Index: RemotingPhaseListener.java
===================================================================
--- RemotingPhaseListener.java (revision 528221)
+++ RemotingPhaseListener.java (working copy)
@@ -94,6 +94,12 @@

         // Acquire a reference to the FacesContext for this request
         FacesContext context = event.getFacesContext();
+
+ // Make sure the request isn't already complete
+ if (context.getResponseComplete()) {
+ return;
+ }
+
         if (log().isDebugEnabled()) {
             log().debug("Checking view identifier '" +
context.getViewRoot().getViewId() + "'");
         }



Can you please add this to RemotingPhaseListener.java?

Thanks!

Ken