Forrest Rae created OFBIZ-6888:
----------------------------------

             Summary: GroovyEngine.serviceInvoker masks Groovy script 
exceptions in some cases
                 Key: OFBIZ-6888
                 URL: https://issues.apache.org/jira/browse/OFBIZ-6888
             Project: OFBiz
          Issue Type: Bug
          Components: framework
    Affects Versions: Release Branch 15.12, Trunk, Release Branch 14.12, 
Release Branch 13.07
            Reporter: Forrest Rae


If GroovyEngine.serviceInvoker catches an exception in a Groovy script, it 
would mask the exception in some cases.  An exception's detailMessage can be 
null.  If it is null, the exception won't be properly returned and logged, and 
that will make spotting problems very difficult.  This was the case for me 
while trying to track down a problem with a 
java.util.ConcurrentModificationException error in a Groovy script.  I suspect 
that this choice to mask exceptions and only return the message has cause bugs 
to not be spotted.

Disabling this for now in favor of returning a proper exception.  
GroovyEngine.serviceInvoker() should throw GenericServiceException if error, 
rather than returning ServiceUtil.returnError(e.getMessage())



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to