doo...@apache.org wrote: > Author: doogie > Date: Sun Apr 4 19:08:46 2010 > New Revision: 930737 > > URL: http://svn.apache.org/viewvc?rev=930737&view=rev > Log: > Make use of UtilObject.equalsHelper in storeAll. > > Modified: > ofbiz/trunk/framework/entity/src/org/ofbiz/entity/GenericDelegator.java > > Modified: > ofbiz/trunk/framework/entity/src/org/ofbiz/entity/GenericDelegator.java > URL: > http://svn.apache.org/viewvc/ofbiz/trunk/framework/entity/src/org/ofbiz/entity/GenericDelegator.java?rev=930737&r1=930736&r2=930737&view=diff > ============================================================================== > --- ofbiz/trunk/framework/entity/src/org/ofbiz/entity/GenericDelegator.java > (original) > +++ ofbiz/trunk/framework/entity/src/org/ofbiz/entity/GenericDelegator.java > Sun Apr 4 19:08:46 2010 > @@ -1450,7 +1450,7 @@ public class GenericDelegator implements > if (value.containsKey(fieldName)) { > Object fieldValue = value.get(fieldName); > Object oldValue = existing.get(fieldName); > - if ((fieldValue == null && oldValue != null) || > (fieldValue != null && !fieldValue.equals(oldValue))) { > + if (UtilObject.equalsHelper(oldValue, > fieldValue)) { > toStore.put(fieldName, fieldValue); > atLeastOneField = true; > }
Did anyone else review this commit? Or did you not understand the code? Yes, I screwed it up. But if no one reviews the commits, then how will much bigger problems be found? This was a simple problem to find, an obvious inverted test, yet no one noticed it. What's the point of having commits mailed to the list if no one will review them?