You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by im...@apache.org on 2007/02/23 00:38:06 UTC
svn commit: r510709 - in
/myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion:
conversation/ConversationRequestParameterProvider.java
listeners/FusionPhaseListener.java
Author: imario
Date: Thu Feb 22 15:38:05 2007
New Revision: 510709
URL: http://svn.apache.org/viewvc?view=rev&rev=510709
Log:
ensure we have the conversationContext parameter starting with the first request
Modified:
myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/conversation/ConversationRequestParameterProvider.java
myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/listeners/FusionPhaseListener.java
Modified: myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/conversation/ConversationRequestParameterProvider.java
URL: http://svn.apache.org/viewvc/myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/conversation/ConversationRequestParameterProvider.java?view=diff&rev=510709&r1=510708&r2=510709
==============================================================================
--- myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/conversation/ConversationRequestParameterProvider.java (original)
+++ myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/conversation/ConversationRequestParameterProvider.java Thu Feb 22 15:38:05 2007
@@ -46,16 +46,19 @@
{
throw new IllegalStateException("can find the conversationManager");
}
- if (!conversationManager.hasConversationContext())
+
+ Long conversationContextId = conversationManager.getConversationContextId();
+ if (conversationContextId == null)
{
return null;
}
- return Long.toString(conversationManager.getConversationContextId().longValue(), Character.MAX_RADIX);
+ return Long.toString(conversationContextId.longValue(), Character.MAX_RADIX);
}
public String[] getFields()
{
+ /*
if (UISeparateConversationContext.isInSeparationMode())
{
return null;
@@ -70,6 +73,7 @@
{
return null;
}
+ */
return REQUEST_PARAMETERS;
}
Modified: myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/listeners/FusionPhaseListener.java
URL: http://svn.apache.org/viewvc/myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/listeners/FusionPhaseListener.java?view=diff&rev=510709&r1=510708&r2=510709
==============================================================================
--- myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/listeners/FusionPhaseListener.java (original)
+++ myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/listeners/FusionPhaseListener.java Thu Feb 22 15:38:05 2007
@@ -19,6 +19,8 @@
package org.apache.myfaces.fusion.listeners;
+import org.apache.myfaces.fusion.conversation.ConversationManager;
+
import javax.faces.event.PhaseEvent;
import javax.faces.event.PhaseId;
import javax.faces.event.PhaseListener;
@@ -41,6 +43,9 @@
{
if (PhaseId.RESTORE_VIEW.equals(event.getPhaseId()))
{
+ // ensure correct setup of system
+ ConversationManager.getInstance(event.getFacesContext());
+
// AUTOMATIC_CONVERSATION_CREATION.set(Boolean.FALSE);
AUTOMATIC_CONVERSATION_CREATION.set(Boolean.TRUE);
}