svn commit: r1649742 - /ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelFormField.java

[ja...@apache.org]

Author: jacopoc
Date: Tue Jan  6 07:43:00 2015
New Revision: 1649742

URL: http://svn.apache.org/r1649742
Log:
OFBIZ-5944: before trying to pull the values for the form's fields, check if the key exists in order to avoid an IllegalArgumentException if the map is a GenericEntity. Thanks to Christian Carlow for the report and to Deepak Dixit for the research and useful comments.

Modified:
    ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelFormField.java

Modified: ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelFormField.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelFormField.java?rev=1649742&r1=1649741&r2=1649742&view=diff
==============================================================================
--- ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelFormField.java (original)
+++ ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelFormField.java Tue Jan  6 07:43:00 2015
@@ -289,7 +289,6 @@ public class ModelFormField {
             }
             Object retVal = null;
             if (UtilValidate.isNotEmpty(this.entryAcsr)) {
-                //Debug.logInfo("Getting entry, using entryAcsr for field " + this.getName() + " of form " + this.modelForm.getName(), module);
                 if (dataMap instanceof GenericEntity) {
                     GenericEntity genEnt = (GenericEntity) dataMap;
                     if (genEnt.getModelEntity().isField(this.entryAcsr.getOriginalName())) {
@@ -301,9 +300,10 @@ public class ModelFormField {
                     retVal = this.entryAcsr.get(dataMap, locale);
                 }
             } else {
-                //Debug.logInfo("Getting entry, no entryAcsr so using field name " + this.name + " for field " + this.getName() + " of form " + this.modelForm.getName(), module);
                 // if no entry name was specified, use the field's name
-                retVal = dataMap.get(this.name);
+                if (dataMap.containsKey(this.name)) {
+                    retVal = dataMap.get(this.name);
+                }
             }
 
             // this is a special case to fill in fields during a create by default from parameters passed in