On 15 Jan 2016 at 07:24:35, Marius Dumitru Florea ([email protected](mailto:[email protected])) wrote:
On Thu, Jan 14, 2016 at 6:51 PM, [email protected] wrote:
Hi devs,
Right now our strategy is for script services and script APIs in general to catch exceptions, store them and offer a getLastError() method to get them (see http://extensions.xwiki.org/xwiki/bin/view/Extension/Script+Module#HBestPrac... )
However it would be much nicer to: * Let our script services generate exceptions * Offer a velocity script service to get the last exception raised by a java call from velocity * Implement this uberspector to catch the exceptions and to set them in the execution context
That should be quite easy to implement IMO.
WDYT?
+1, it's a pain to call setLastError() everywhere there can be an exception thrown, and we almost always forget to do it (for this reason).
Note that we also have the #try() directive now.
Yes, I should have mentioned that there’s indeed also this possibility: * Have script API throw Exceptions * Force velocity script users to wrap their code with the try directive when they need to catch exceptions I still believe that the use of the Exception-catching uberspector is better. WDYT? Thanks -Vincent
Thanks, Marius
Thanks -Vincent
PS: This is http://jira.xwiki.org/browse/XWIKI-2374