dev@javaserverfaces.java.net

[1108-FlashFacesMessages] Dan Allen: Question

From: Ed Burns <Ed.Burns_at_Sun.COM>
Date: Mon, 15 Jun 2009 09:44:58 -0700

Hello Dan,

Are you sure you want to move those to the request map, rather than the
cookie map?

bash-3.00$ svn log -r 7414
------------------------------------------------------------------------
r7414 | mojavelinux | 2009-06-10 13:14:58 -0700 (Wed, 10 Jun 2009) | 3 lines

fix for issue 1108
put the FacesMessages in the request map rather than the cookie map

------------------------------------------------------------------------
bash-3.00$ svn diff -r 7414 -r 7413
svn: Multiple revision arguments encountered; can't specify -c twice, or both -c and -r
bash-3.00$ svn diff -r 7414:7413
Index: jsf-ri/src/com/sun/faces/context/flash/ELFlash.java
===================================================================
--- jsf-ri/src/com/sun/faces/context/flash/ELFlash.java (revision 7414)
+++ jsf-ri/src/com/sun/faces/context/flash/ELFlash.java (revision 7413)
@@ -322,7 +322,7 @@
         boolean isRedirect = this.isRedirect();
         if (null != allFacesMessages) {
             if (isRedirect) {
- this.getThisRequestMap(context).put(Constants.FACES_MESSAGES_ATTRIBUTE_NAME,
+ this.getMapForCookie(context).put(Constants.FACES_MESSAGES_ATTRIBUTE_NAME,
                         allFacesMessages);
                 
             } else {

They need to be there for the next request, which is why I had them in
the cookie map.

Please respond and tell me why you think this fixed the problem?

Thanks,

Ed

P.S. Please let me know if I should no longer Cc your gmail address when
I write a message to dov_at_javaserverfaces intended for you.
-- 
| ed.burns_at_sun.com  | office: 408 884 9519 OR x31640
| homepage:         | http://ridingthecrest.com/