Author: vdedik
Date: 2012-08-13 07:46:56 -0400 (Mon, 13 Aug 2012)
New Revision: 15032

Modified:
   
branches/community/Seam_2_3/jboss-seam/src/main/java/org/jboss/seam/faces/FacesManager.java
   
branches/community/Seam_2_3/jboss-seam/src/main/java/org/jboss/seam/web/RedirectFilter.java
Log:
JBSEAM-4968 - cleaner solution

Modified: 
branches/community/Seam_2_3/jboss-seam/src/main/java/org/jboss/seam/faces/FacesManager.java
===================================================================
--- 
branches/community/Seam_2_3/jboss-seam/src/main/java/org/jboss/seam/faces/FacesManager.java
 2012-08-10 18:40:28 UTC (rev 15031)
+++ 
branches/community/Seam_2_3/jboss-seam/src/main/java/org/jboss/seam/faces/FacesManager.java
 2012-08-13 11:46:56 UTC (rev 15032)
@@ -223,6 +223,10 @@
       {
          throw new RedirectException(ioe);
       }
+      catch (IllegalStateException ise)
+      {
+         throw new RedirectException(ise.getMessage());
+      }
       finally
       {
          Contexts.getEventContext().remove(REDIRECT_FROM_MANAGER);

Modified: 
branches/community/Seam_2_3/jboss-seam/src/main/java/org/jboss/seam/web/RedirectFilter.java
===================================================================
--- 
branches/community/Seam_2_3/jboss-seam/src/main/java/org/jboss/seam/web/RedirectFilter.java
 2012-08-10 18:40:28 UTC (rev 15031)
+++ 
branches/community/Seam_2_3/jboss-seam/src/main/java/org/jboss/seam/web/RedirectFilter.java
 2012-08-13 11:46:56 UTC (rev 15032)
@@ -69,11 +69,7 @@
                   }
                }
             }
-            //if the response is already committed (e.g. a redirect has been 
already sent) do nothing
-            if ( !super.getResponse().isCommitted() )
-            {
-               super.sendRedirect(url);
-            }
+            super.sendRedirect(url);
          }
       };
    }

_______________________________________________
seam-commits mailing list
[email protected]
https://lists.jboss.org/mailman/listinfo/seam-commits

Reply via email to