You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jmeter-dev@jakarta.apache.org by se...@apache.org on 2008/05/10 15:13:04 UTC
svn commit: r655079 -
/jakarta/jmeter/trunk/src/core/org/apache/jmeter/testbeans/gui/WrapperEditor.java
Author: sebb
Date: Sat May 10 06:13:04 2008
New Revision: 655079
URL: http://svn.apache.org/viewvc?rev=655079&view=rev
Log:
Rework fix to avoid use of sun class reference
Modified:
jakarta/jmeter/trunk/src/core/org/apache/jmeter/testbeans/gui/WrapperEditor.java
Modified: jakarta/jmeter/trunk/src/core/org/apache/jmeter/testbeans/gui/WrapperEditor.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/testbeans/gui/WrapperEditor.java?rev=655079&r1=655078&r2=655079&view=diff
==============================================================================
--- jakarta/jmeter/trunk/src/core/org/apache/jmeter/testbeans/gui/WrapperEditor.java (original)
+++ jakarta/jmeter/trunk/src/core/org/apache/jmeter/testbeans/gui/WrapperEditor.java Sat May 10 06:13:04 2008
@@ -29,8 +29,6 @@
import org.apache.jorphan.logging.LoggingManager;
import org.apache.log.Logger;
-import sun.beans.editors.NumberEditor;
-
/**
* This is an implementation of a full-fledged property editor, providing both
* object-text transformation and an editor GUI (a custom editor component),
@@ -285,10 +283,16 @@
try {
// Bug 44314 Number field does not seem to accept ""
- if (text.length()==0 && typeEditor instanceof NumberEditor){
- text="0";//$NON-NLS-1$
- }
- typeEditor.setAsText(text);
+ try {
+ typeEditor.setAsText(text);
+ } catch (NumberFormatException e) {
+ if (text.length()==0){
+ text="0";//$NON-NLS-1$
+ typeEditor.setAsText(text);
+ } else {
+ shouldNeverHappen(e);
+ }
+ }
} catch (IllegalArgumentException e) {
shouldNeverHappen(e);
}
---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org