Author: hansbak
Date: Fri Jan 30 06:02:31 2009
New Revision: 739188
URL:
http://svn.apache.org/viewvc?rev=739188&view=revLog:
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());