You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by st...@apache.org on 2011/10/10 16:40:05 UTC

svn commit: r1181000 - /myfaces/extensions/cdi/trunk/jee-modules/jsf20-module/impl/src/main/java/org/apache/myfaces/extensions/cdi/jsf2/impl/scope/conversation/JsfRestParameters.java

Author: struberg
Date: Mon Oct 10 14:40:05 2011
New Revision: 1181000

URL: http://svn.apache.org/viewvc?rev=1181000&view=rev
Log:
EXTCDI-232 improve handling if bean gets used from different views

Modified:
    myfaces/extensions/cdi/trunk/jee-modules/jsf20-module/impl/src/main/java/org/apache/myfaces/extensions/cdi/jsf2/impl/scope/conversation/JsfRestParameters.java

Modified: myfaces/extensions/cdi/trunk/jee-modules/jsf20-module/impl/src/main/java/org/apache/myfaces/extensions/cdi/jsf2/impl/scope/conversation/JsfRestParameters.java
URL: http://svn.apache.org/viewvc/myfaces/extensions/cdi/trunk/jee-modules/jsf20-module/impl/src/main/java/org/apache/myfaces/extensions/cdi/jsf2/impl/scope/conversation/JsfRestParameters.java?rev=1181000&r1=1180999&r2=1181000&view=diff
==============================================================================
--- myfaces/extensions/cdi/trunk/jee-modules/jsf20-module/impl/src/main/java/org/apache/myfaces/extensions/cdi/jsf2/impl/scope/conversation/JsfRestParameters.java (original)
+++ myfaces/extensions/cdi/trunk/jee-modules/jsf20-module/impl/src/main/java/org/apache/myfaces/extensions/cdi/jsf2/impl/scope/conversation/JsfRestParameters.java Mon Oct 10 14:40:05 2011
@@ -73,7 +73,10 @@ public class JsfRestParameters extends R
         if (!currentViewParams.equals(oldViewParams))
         {
             viewParametersForViewId.put(viewId, currentViewParams);
-            return true;
+
+            // only reset the rest context if the oldViewParamaeters were different
+            // but not if they didn't got set yet
+            return oldViewParams != null;
         }
 
         return false;