I'm not sure why you're using a fixed name instead of the field name fetching mechanism, especially that this is a loop.
Also, I'm not sure the entire try / catch block is necessary. What test made you believe you need to hard code the field in here? On Fri, Mar 31, 2017 at 8:00 PM, <[email protected]> wrote: > Author: jleroux > Date: Fri Mar 31 17:00:58 2017 > New Revision: 1789711 > > URL: http://svn.apache.org/viewvc?rev=1789711&view=rev > Log: > No functional change, fixes a C/P with wrong value name in errMsgList > > Modified: > ofbiz/ofbiz-framework/trunk/applications/content/ > groovyScripts/content/GetContentLookupList.groovy > > Modified: ofbiz/ofbiz-framework/trunk/applications/content/ > groovyScripts/content/GetContentLookupList.groovy > URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/ > applications/content/groovyScripts/content/GetContentLookupList.groovy? > rev=1789711&r1=1789710&r2=1789711&view=diff > ============================================================ > ================== > --- ofbiz/ofbiz-framework/trunk/applications/content/ > groovyScripts/content/GetContentLookupList.groovy (original) > +++ ofbiz/ofbiz-framework/trunk/applications/content/ > groovyScripts/content/GetContentLookupList.groovy Fri Mar 31 17:00:58 2017 > @@ -70,7 +70,7 @@ try { > viewSize = Integer.valueOf((String)parameters.get("VIEW_SIZE")). > intValue() > } catch (NumberFormatException nfe) { > Debug.logError(nfe, "Caught an exception : " + nfe.toString(), > "GetContentLookupList.groovy") > - errMsgList.add("Entered value is non-numeric for numeric field: " + > field.getName()) > + errMsgList.add("Entered value is non-numeric for numeric field: > VIEW_SIZE")) > } > > context.viewSize = viewSize > > >
