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