Re: [xwiki-devs] [PROPOSAL] Provide an AbstractScriptService best practice helper

On Sep 28, 2012, at 11:37 AM, Thomas Mortagne <thomas.mortagne(a)xwiki.com> wrote: I'm still hesitating. I've discussed this in the past and my idea has always been to try to hide this as a generic feature of Velocity rather than making it visible. Let me explain. My ideal solution is this: * Add a velicity uberspector so that any method that is called that throws an exception returns a null instead and saves the exception automatically in the context and then provide some Velocity tool to easily access the exception My rationale is the following: * We have this need for Velocity but not for other scripting languages such as Groovy, Ruby, Python. Basically we have this issue only for templating languages that don't support exception handling. * It's a pity that when coding in groovy I can't use Groovy's exception handling mechanism and have to do something hackish instead. Thus to conclude, my preference is: * Don't change ScriptService * Add an uberspector to handle exceptions in Velocity * Progressively modify our existing script apis to not do this hacky try/catch and null setting and for new script apis do the normal thing Now, if we really want to have all our script languages return null instead of throw an exception, then I'd suggest that this is not the job of the implementer of Script Service to do this but instead use a Dynamic Proxy to wrap ScriptService and delegate to the script service implementation and in the dynamic proxy do the try/catch and null setting. Thanks -Vincent