You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by lu...@apache.org on 2011/01/21 04:56:34 UTC

svn commit: r1061617 - /myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/el/FlashELResolver.java

Author: lu4242
Date: Fri Jan 21 03:56:34 2011
New Revision: 1061617

URL: http://svn.apache.org/viewvc?rev=1061617&view=rev
Log:
MYFACES-3009 Flash scope looses values on redirect

Modified:
    myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/el/FlashELResolver.java

Modified: myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/el/FlashELResolver.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/el/FlashELResolver.java?rev=1061617&r1=1061616&r2=1061617&view=diff
==============================================================================
--- myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/el/FlashELResolver.java (original)
+++ myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/el/FlashELResolver.java Fri Jan 21 03:56:34 2011
@@ -147,6 +147,7 @@ public class FlashELResolver extends ELR
                 // be called again but this time the property name
                 // to be resolved will be called, so we can do keep
                 // promotion.
+                elContext.setPropertyResolved(true);
                 return base;
             }
             else if (NOW.equals(strProperty))
@@ -166,10 +167,10 @@ public class FlashELResolver extends ELR
             {
                 //Resolve property calling get or keep
                 elContext.setPropertyResolved(true);
-                //Obtain the value on requestMap if any
-                Object value = externalContext.getRequestMap().get(strProperty);
                 //promote it to flash scope
                 flash.keep(strProperty);
+                //Obtain the value on requestMap if any
+                Object value = externalContext.getRequestMap().get(strProperty);
                 return value;
             }
             else