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/25 02:50:40 UTC

svn commit: r1206035 - /jmeter/trunk/src/core/org/apache/jmeter/testbeans/gui/WrapperEditor.java

Author: sebb
Date: Fri Nov 25 01:50:39 2011
New Revision: 1206035

URL: http://svn.apache.org/viewvc?rev=1206035&view=rev
Log:
Better diagnostic

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

Modified: jmeter/trunk/src/core/org/apache/jmeter/testbeans/gui/WrapperEditor.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/testbeans/gui/WrapperEditor.java?rev=1206035&r1=1206034&r2=1206035&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/testbeans/gui/WrapperEditor.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/testbeans/gui/WrapperEditor.java Fri Nov 25 01:50:39 2011
@@ -381,7 +381,7 @@ class WrapperEditor extends PropertyEdit
         } else if (!acceptsExpressions || !isExpression(text)) {
             // not an expression (can't be or isn't), not null.
             try {
-                typeEditor.setAsText(text);
+                typeEditor.setAsText(text); // ensure value is propagated to editor
             } catch (IllegalArgumentException e) {
                 shouldNeverHappen(e);
             }
@@ -437,7 +437,7 @@ class WrapperEditor extends PropertyEdit
             firePropertyChange();
         } else {
             if (GuiPackage.getInstance() == null){
-                log.warn("Invalid value: "+text+" "+guiEditor);
+                log.warn("Invalid value: "+text+" "+typeEditor);
             } else {
                 JOptionPane.showMessageDialog(guiEditor.getCustomEditor().getParent(),
                    JMeterUtils.getResString("property_editor.value_is_invalid_message"),//$NON-NLS-1$