Author: dan.j.allen
Date: 2008-06-09 13:52:47 -0400 (Mon, 09 Jun 2008)
New Revision: 8361
Modified:
branches/Seam_2_0/src/main/org/jboss/seam/core/ConversationPropagation.java
branches/Seam_2_0/src/main/org/jboss/seam/navigation/ELConversationIdParameter.java
Log:
JBSEAM-3053
Modified: branches/Seam_2_0/src/main/org/jboss/seam/core/ConversationPropagation.java
===================================================================
--- branches/Seam_2_0/src/main/org/jboss/seam/core/ConversationPropagation.java 2008-06-09
17:22:02 UTC (rev 8360)
+++ branches/Seam_2_0/src/main/org/jboss/seam/core/ConversationPropagation.java 2008-06-09
17:52:47 UTC (rev 8361)
@@ -117,11 +117,9 @@
}
// Try to restore the conversation from parameters (the user has specified
the exact conversation to restore using f:param)
conversationId =
currentConversationIdParameter.getRequestConversationId(parameters);
- if (conversationId == null)
- {
- // Try to restore the conversation from the EL expression on the
conversation definition
- conversationId = currentConversationIdParameter.getConversationId();
- }
+
+ // NOTE: If conversationId is still null, don't try to resolve the EL
here because we don't yet
+ // have a conversation and therefore things may blow up; resolve EL in
getInitialConversationId()
}
else
{
Modified:
branches/Seam_2_0/src/main/org/jboss/seam/navigation/ELConversationIdParameter.java
===================================================================
---
branches/Seam_2_0/src/main/org/jboss/seam/navigation/ELConversationIdParameter.java 2008-06-09
17:22:02 UTC (rev 8360)
+++
branches/Seam_2_0/src/main/org/jboss/seam/navigation/ELConversationIdParameter.java 2008-06-09
17:52:47 UTC (rev 8361)
@@ -42,7 +42,7 @@
public String getInitialConversationId(Map parameters)
{
String id = getRequestConversationId(parameters);
- return id==null ? Id.nextId() : id; //TODO: should we try using the expression?
+ return id == null ? getConversationId() : id;
}
public String getRequestConversationId(Map parameters)
Show replies by date