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/03/01 08:59:27 UTC

svn commit: r513219 - /myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/conversation/Conversation.java

Author: imario
Date: Wed Feb 28 23:59:26 2007
New Revision: 513219

URL: http://svn.apache.org/viewvc?view=rev&rev=513219
Log:
delayed marking the conversation as closed

Modified:
    myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/conversation/Conversation.java

Modified: myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/conversation/Conversation.java
URL: http://svn.apache.org/viewvc/myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/conversation/Conversation.java?view=diff&rev=513219&r1=513218&r2=513219
==============================================================================
--- myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/conversation/Conversation.java (original)
+++ myfaces/fusion/trunk/core/src/main/java/org/apache/myfaces/fusion/conversation/Conversation.java Wed Feb 28 23:59:26 2007
@@ -140,7 +140,7 @@
 	{
 		if (isInvalid())
 		{
-			throw new IllegalStateException("conversation " + getName() + " closed");
+			throw new IllegalStateException("conversation '" + getName() + "' closed");
 		}
 	}
 
@@ -212,8 +212,6 @@
 	 */
 	protected void destroy()
 	{
-		invalid = true;
-
 		if (log.isDebugEnabled())
 		{
 			log.debug("end conversation:" + name);
@@ -229,6 +227,8 @@
 		}
 
 		conversationContext.removeConversation(getName());
+		
+		invalid = true;
 	}
 
 	/**