You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by se...@apache.org on 2011/11/24 11:56:21 UTC

svn commit: r1205785 - /jmeter/trunk/src/core/org/apache/jmeter/testbeans/TestBeanHelper.java

Author: sebb
Date: Thu Nov 24 10:56:21 2011
New Revision: 1205785

URL: http://svn.apache.org/viewvc?rev=1205785&view=rev
Log:
Ensure NOT_UNDEFINED defaults to FALSE

Modified:
    jmeter/trunk/src/core/org/apache/jmeter/testbeans/TestBeanHelper.java

Modified: jmeter/trunk/src/core/org/apache/jmeter/testbeans/TestBeanHelper.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/testbeans/TestBeanHelper.java?rev=1205785&r1=1205784&r2=1205785&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/testbeans/TestBeanHelper.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/testbeans/TestBeanHelper.java Thu Nov 24 10:56:21 2011
@@ -120,7 +120,10 @@ public class TestBeanHelper {
         {
             value = unwrapCollection((MultiProperty)jprop,(String)desc.getValue(TableEditor.CLASSNAME));
         }
-        else if (jprop instanceof NullProperty && Boolean.FALSE.equals(desc.getValue(GenericTestBeanCustomizer.NOT_UNDEFINED)))  // value was not provided, and this is allowed
+        // value was not provided, and this is allowed
+        else if (jprop instanceof NullProperty &&
+                // use negative condition so missing (null) value is treated as FALSE
+                ! Boolean.TRUE.equals(desc.getValue(GenericTestBeanCustomizer.NOT_UNDEFINED)))
         {    
             value=null;
         }