svn commit: r1789736 - /ofbiz/ofbiz-framework/trunk/applications/content/groovyScripts/content/GetContentLookupList.groovy

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r1789736 - /ofbiz/ofbiz-framework/trunk/applications/content/groovyScripts/content/GetContentLookupList.groovy

jleroux@apache.org
Author: jleroux
Date: Fri Mar 31 19:23:12 2017
New Revision: 1789736

URL: http://svn.apache.org/viewvc?rev=1789736&view=rev
Log:
No functional change.

Michael suggested in dev ML that we don't show the error in UI because VIEW_SIZE
 is only technical, which I agree with. The log is enough.
 
Actually it was already the case because I missed to move the
if (errMsgList) {
    request.setAttribute("_ERROR_MESSAGE_LIST_", errMsgList)
}
block below the now caught NumberFormatException

So I just need to remove the 2nd line in the catch, done.

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=1789736&r1=1789735&r2=1789736&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 19:23:12 2017
@@ -70,7 +70,6 @@ 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: VIEW_SIZE"))
 }
 
 context.viewSize = viewSize