You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@ofbiz.apache.org by mb...@apache.org on 2017/11/03 17:47:41 UTC

svn commit: r1814230 - /ofbiz/ofbiz-framework/trunk/framework/entity/src/main/java/org/apache/ofbiz/entity/GenericEntity.java

Author: mbrohl
Date: Fri Nov  3 17:47:41 2017
New Revision: 1814230

URL: http://svn.apache.org/viewvc?rev=1814230&view=rev
Log:
Fixed: Corrected wrong FindBugs/refactoring change introduced in 
r1813640.
(OFBIZ-9716)

Thanks Suraj Khurana for reporting and Dennis Balkir for providing the
patch.

Modified:
    ofbiz/ofbiz-framework/trunk/framework/entity/src/main/java/org/apache/ofbiz/entity/GenericEntity.java

Modified: ofbiz/ofbiz-framework/trunk/framework/entity/src/main/java/org/apache/ofbiz/entity/GenericEntity.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/entity/src/main/java/org/apache/ofbiz/entity/GenericEntity.java?rev=1814230&r1=1814229&r2=1814230&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/entity/src/main/java/org/apache/ofbiz/entity/GenericEntity.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/entity/src/main/java/org/apache/ofbiz/entity/GenericEntity.java Fri Nov  3 17:47:41 2017
@@ -637,7 +637,7 @@ public class GenericEntity implements Ma
         Object obj = get(name);
 
         if (obj == null) {
-            return false;
+            return null;
         }
         if (obj instanceof Boolean) {
             return (Boolean) obj;