svn commit: r739188 - /ofbiz/trunk/framework/entity/src/org/ofbiz/entity/condition/EntityExpr.java

[hansbak-2]

Author: hansbak
Date: Fri Jan 30 06:02:31 2009
New Revision: 739188

URL: http://svn.apache.org/viewvc?rev=739188&view=rev
Log:
give an error message instead of NPE when a field is not found

Modified:
    ofbiz/trunk/framework/entity/src/org/ofbiz/entity/condition/EntityExpr.java

Modified: ofbiz/trunk/framework/entity/src/org/ofbiz/entity/condition/EntityExpr.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/entity/src/org/ofbiz/entity/condition/EntityExpr.java?rev=739188&r1=739187&r2=739188&view=diff
==============================================================================
--- ofbiz/trunk/framework/entity/src/org/ofbiz/entity/condition/EntityExpr.java (original)
+++ ofbiz/trunk/framework/entity/src/org/ofbiz/entity/condition/EntityExpr.java Fri Jan 30 06:02:31 2009
@@ -27,6 +27,7 @@
 
 import org.ofbiz.base.util.Debug;
 import org.ofbiz.base.util.ObjectType;
+import org.ofbiz.base.util.UtilValidate;
 import org.ofbiz.entity.EntityCryptoException;
 import org.ofbiz.entity.GenericDelegator;
 import org.ofbiz.entity.GenericEntity;
@@ -250,6 +251,9 @@
         }
         
         ModelField curField = modelEntity.getField(fieldName);
+        if (UtilValidate.isEmpty(curField)) {
+            throw new IllegalArgumentException("FieldName " + fieldName + " not found for entity: " + modelEntity.getEntityName());
+        }
         ModelFieldType type = null;
         try {
             type = delegator.getEntityFieldType(modelEntity, curField.getType());