Author: sebb
Date: Sat Mar 12 23:30:26 2016
New Revision: 1734746

URL: http://svn.apache.org/viewvc?rev=1734746&view=rev
Log:
SampleResult SaveConfig does not allow some fields to be disabled
Previous fix did not work for idleTime because the ctor did not set all the 
defaulted fields
Bugzilla Id: 59172

Modified:
    
jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java

Modified: 
jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java?rev=1734746&r1=1734745&r2=1734746&view=diff
==============================================================================
--- 
jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java 
(original)
+++ 
jmeter/trunk/src/core/org/apache/jmeter/samplers/SampleSaveConfiguration.java 
Sat Mar 12 23:30:26 2016
@@ -42,6 +42,7 @@ import org.apache.log.Logger;
  * - setXyz(boolean)
  * - saveXyz()
  * - update SampleSaveConfigurationConverter to add new fields to marshall() 
and shouldSerialiseMember()
+ * - update ctor SampleSaveConfiguration(boolean value) to set the value if it 
is a boolean property
  * - update SampleResultConverter and/or HTTPSampleConverter
  * - update CSVSaveService: CSV_XXXX, makeResultFromDelimitedString, 
printableFieldNamesToString, static{}
  * - update messages.properties to add save_xyz entry
@@ -436,26 +437,27 @@ public class SampleSaveConfiguration imp
         assertions = value;
         bytes = value;
         code = value;
+        connectTime = value;
         dataType = value;
         encoding = value;
         fieldNames = value;
         fileName = value;
         hostname = value;
+        idleTime = value;
         label = value;
         latency = value;
-        connectTime = value;
         message = value;
         printMilliseconds = _printMilliseconds;//is derived from properties 
only
         requestHeaders = value;
         responseData = value;
         responseDataOnError = value;
         responseHeaders = value;
+        sampleCount = value;
         samplerData = value;
         saveAssertionResultsFailureMessage = value;
         subresults = value;
         success = value;
         threadCounts = value;
-        sampleCount = value;
         threadName = value;
         time = value;
         timestamp = value;


Reply via email to