|
Moved to 2.0.1 since this seems to be an issue with Jetty.
Reverted patch from r2941 as it causes success messages to not be rendered:
http://fisheye4.cenqua.com/changelog/appfuse?cs=2941 404 doesn't work with JSF on Jetty, but it works with Tomcat. Seems to be a bug in Jetty or MyFaces. Since it's not solved by upgrading to Jetty 6.1.6, I'm deferring this bug until it's fixed by one of those libraries.
Appears to be fixed by upgrading to MyFaces 1.2.2.
|
||||||||||||||||||||||||||||||||||
Index: src/main/java/org/appfuse/webapp/filter/MessageFilter.java
===================================================================
--- src/main/java/org/appfuse/webapp/filter/MessageFilter.java (revision 2936)
+++ src/main/java/org/appfuse/webapp/filter/MessageFilter.java (working copy)
@@ -1,6 +1,7 @@
package org.appfuse.webapp.filter;
import java.io.IOException;
+import java.util.ArrayList;
import javax.servlet.Filter;
import javax.servlet.FilterChain;
@@ -31,6 +32,10 @@
if (messages != null) {
request.setAttribute("messages", messages);
request.getSession().removeAttribute("messages");
+ } else {
+ // workaround for issue with Jetty 6.1.5 (Maven Plugin) and MyFaces 1.2.0
+ // http://issues.appfuse.org/browse/APF-856
+ request.setAttribute("messages", new ArrayList());
}
// grab errors from the session and put them into request
@@ -40,6 +45,10 @@
if (errors != null) {
request.setAttribute("errors", errors);
request.getSession().removeAttribute("errors");
+ } else {
+ // workaround for issue with Jetty 6.1.5 (Maven Plugin) and MyFaces 1.2.0
+ // http://issues.appfuse.org/browse/APF-856
+ request.setAttribute("errors", new ArrayList());
}
chain.doFilter(req, res);