You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by do...@apache.org on 2010/02/13 04:54:42 UTC
svn commit: r909718 -
/ofbiz/trunk/framework/base/src/org/ofbiz/base/conversion/MiscConverters.java
Author: doogie
Date: Sat Feb 13 03:54:42 2010
New Revision: 909718
URL: http://svn.apache.org/viewvc?rev=909718&view=rev
Log:
Fix ordering of Enum converter class comparisons.
Modified:
ofbiz/trunk/framework/base/src/org/ofbiz/base/conversion/MiscConverters.java
Modified: ofbiz/trunk/framework/base/src/org/ofbiz/base/conversion/MiscConverters.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/conversion/MiscConverters.java?rev=909718&r1=909717&r2=909718&view=diff
==============================================================================
--- ofbiz/trunk/framework/base/src/org/ofbiz/base/conversion/MiscConverters.java (original)
+++ ofbiz/trunk/framework/base/src/org/ofbiz/base/conversion/MiscConverters.java Sat Feb 13 03:54:42 2010
@@ -141,7 +141,7 @@
}
public boolean canConvert(Class<?> sourceClass, Class<?> targetClass) {
- return sourceClass.isAssignableFrom(Enum.class) && targetClass.isAssignableFrom(String.class);
+ return Enum.class.isAssignableFrom(sourceClass) && String.class.isAssignableFrom(targetClass);
}
public String convert(Enum obj) throws ConversionException {
@@ -167,7 +167,7 @@
}
public boolean canConvert(Class<?> sourceClass, Class<?> targetClass) {
- return sourceClass.isAssignableFrom(String.class) && targetClass.isAssignableFrom(Enum.class);
+ return String.class.isAssignableFrom(sourceClass) && Enum.class.isAssignableFrom(targetClass);
}
public Enum convert(String obj) throws ConversionException {