You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by sk...@apache.org on 2008/02/17 11:17:32 UTC

svn commit: r628466 - /myfaces/orchestra/trunk/core/src/main/java/org/apache/myfaces/orchestra/conversation/spring/SpringConversationScope.java

Author: skitching
Date: Sun Feb 17 02:17:31 2008
New Revision: 628466

URL: http://svn.apache.org/viewvc?rev=628466&view=rev
Log:
Minor code tidyup. There was no bug here before, but this change avoids any confusion.

Modified:
    myfaces/orchestra/trunk/core/src/main/java/org/apache/myfaces/orchestra/conversation/spring/SpringConversationScope.java

Modified: myfaces/orchestra/trunk/core/src/main/java/org/apache/myfaces/orchestra/conversation/spring/SpringConversationScope.java
URL: http://svn.apache.org/viewvc/myfaces/orchestra/trunk/core/src/main/java/org/apache/myfaces/orchestra/conversation/spring/SpringConversationScope.java?rev=628466&r1=628465&r2=628466&view=diff
==============================================================================
--- myfaces/orchestra/trunk/core/src/main/java/org/apache/myfaces/orchestra/conversation/spring/SpringConversationScope.java (original)
+++ myfaces/orchestra/trunk/core/src/main/java/org/apache/myfaces/orchestra/conversation/spring/SpringConversationScope.java Sun Feb 17 02:17:31 2008
@@ -84,7 +84,7 @@
 	public static final String LIFETIME_MANUAL = "manual";
 
 	private Integer timeout;
-	private String lifetime = "manual";
+	private String lifetime = LIFETIME_MANUAL;
 
 	/**
 	 * See {@link #setTimeout}.
@@ -172,7 +172,7 @@
 		}
 
 		// conversation lifetime
-		if (LIFETIME_ACCESS == lifetime)
+		if (LIFETIME_ACCESS.equals(lifetime))
 		{
 			ConversationAspect aspect = new ConversationAccessLifetimeAspect(conversation);
 			conversation.addAspect(aspect);