[
https://issues.apache.org/jira/browse/OFBIZ-6888?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jacques Le Roux reassigned OFBIZ-6888:
--------------------------------------
Assignee: Jacques Le Roux
> 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 13.07, Release Branch 14.12, Trunk, Release Branch 15.12
> Reporter: Forrest Rae
> Assignee: Jacques Le Roux
> Labels: errorhandling
> Attachments: OFBIZ-6888.patch
>
>
> 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)
Locked
