[seam-commits] Seam SVN: r15032 - in branches/community/Seam_2_3/jboss-seam/src/main/java/org/jboss/seam: web and 1 other directory.

[seam-commits at lists.jboss.org]

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);
          }
       };
    }