Adrian,
I did not thought that way, and thought only about primitives wrappers.
Done at r889319, much cleaner and OO, thanks
Jacques
() ascii ribbon campaign against HTML e-mail
/\ www.asciiribbon.org
From: "Adrian Crum" <
[hidden email]>
> Jacques,
>
> Most numeric classes extend Number, so you could test for Number and eliminate a lot of those conditionals.
>
> -Adrian
>
>
[hidden email] wrote:
>> Author: jleroux
>> Date: Wed Dec 9 23:15:19 2009
>> New Revision: 889015
>>
>> URL:
http://svn.apache.org/viewvc?rev=889015&view=rev>> Log:
>> I found Long to appear too much. While at it I add all primtives wrappers
>>
>> Modified:
>> ofbiz/trunk/framework/base/src/org/ofbiz/base/util/ObjectType.java
>>
>> Modified: ofbiz/trunk/framework/base/src/org/ofbiz/base/util/ObjectType.java
>> URL:
>>
http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/util/ObjectType.java?rev=889015&r1=889014&r2=889015&view=diff>> ==============================================================================
>> --- ofbiz/trunk/framework/base/src/org/ofbiz/base/util/ObjectType.java (original)
>> +++ ofbiz/trunk/framework/base/src/org/ofbiz/base/util/ObjectType.java Wed Dec 9 23:15:19 2009
>> @@ -766,7 +766,13 @@
>> if (value instanceof Boolean) return false; if (value instanceof Integer) return false; if (value
>> instanceof Long) return false; - if (value instanceof java.math.BigDecimal) return false; + if
>> (value instanceof Float) return false; + if (value instanceof Double) return false; + if (value
>> instanceof Short) return false; + if (value instanceof Byte) return false; + if (value instanceof
>> Character) return false; + if (value instanceof java.math.BigDecimal) return false;
>> + if (value instanceof java.math.BigInteger) return false; if (value instanceof java.sql.Timestamp) return false;
>> Debug.logWarning("In ObjectType.isEmpty(Object value) returning false for " + value.getClass() + " Object.", module);
>>
>>
>>
>
Locked
