Re: svn commit: r1532085 - /jmeter/trunk/src/core/org/apache/jmeter/control/TransactionController.java

[Milamber]

Le 14/10/2013 21:47, pmoua...@apache.org a ecrit :
Author: pmouawad
Date: Mon Oct 14 20:47:58 2013
New Revision: 1532085
URL: http://svn.apache.org/r1532085
Log:
Add constant

Perhaps, It would be great to transform this default value to a property 
for allow user to change this.




Modified:
     jmeter/trunk/src/core/org/apache/jmeter/control/TransactionController.java

Modified: 
jmeter/trunk/src/core/org/apache/jmeter/control/TransactionController.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/control/TransactionController.java?rev=1532085&r1=1532084&r2=1532085&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/control/TransactionController.java 
(original)
+++ jmeter/trunk/src/core/org/apache/jmeter/control/TransactionController.java 
Mon Oct 14 20:47:58 2013
@@ -53,6 +53,8 @@ public class TransactionController exten
      
      private static final Logger log = LoggingManager.getLoggerForClass();
  
+    private static final boolean DEFAULT_VALUE_FOR_INCLUDE_TIMERS = true; // default true for compatibility
+
      /**
       * Only used in parent Mode
       */
@@ -293,7 +295,7 @@ public class TransactionController exten
       * @param includeTimers
       */
      public void setIncludeTimers(boolean includeTimers) {
-        setProperty(INCLUDE_TIMERS, includeTimers, true); // default true for 
compatibility
+        setProperty(INCLUDE_TIMERS, includeTimers, 
DEFAULT_VALUE_FOR_INCLUDE_TIMERS);
      }
  
      /**
@@ -302,6 +304,6 @@ public class TransactionController exten
       * @return boolean (defaults to true for backwards compatibility)
       */
      public boolean isIncludeTimers() {
-        return getPropertyAsBoolean(INCLUDE_TIMERS, true);
+        return getPropertyAsBoolean(INCLUDE_TIMERS, 
DEFAULT_VALUE_FOR_INCLUDE_TIMERS);
      }
  }