commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject [jira] Commented: (JELLY-116) [PATCH] InvokeTag and InvokeStaticTag should export exception
Date Mon, 16 Aug 2004 03:31:21 GMT
The following comment has been added to this issue:

     Author: Felipe Leme
    Created: Sun, 15 Aug 2004 8:31 PM
Sorry, my bad, I used the wrong casts.

Yes, Throwable.getCause() is JDK 1.4, but I'm using InvocationTarget.getTargetMessage() instead:

I mean, there is also a call to jellyException.getCause(), but JellyException defines it indenpendently
on the JDK version.

Anyway, I've just ran maven on it using JDK 1.3.1, and it worked fine. It's just a matter
of modifying the test cases, changing the following line:

 Exception jellyException = (Exception) getJellyContext().getVariable("jellyException");

 JellyException jellyException = (JellyException) getJellyContext().getVariable("jellyException");

And also adding a 

import org.apache.commons.jelly.JellyException;

(Sorry, I'm not with CVS set up right now to send you a patch)
View this comment:

View the issue:

Here is an overview of the issue:
        Key: JELLY-116
    Summary: [PATCH] InvokeTag and InvokeStaticTag should export exception
       Type: New Feature

     Status: Closed
   Priority: Major
 Resolution: FIXED

    Project: jelly
   Fix Fors:

   Assignee: dion gillard
   Reporter: Felipe Leme

    Created: Mon, 9 Aug 2004 6:55 PM
    Updated: Sun, 15 Aug 2004 8:31 PM

If the method invoked throws a InvocationTargetException, invoke and invokeStatic should allow
the caller to obtain the original exception.

So, I'm providing a patch (including test cases) that export that exception if the exceptionVar
paramater is set (and in this case, it does not throw a JellyTagException). If that parameter
is not set, it assumes the old behaviour (i.e., throws a JellyTagException).

This message is automatically generated by JIRA.

If you think it was sent incorrectly contact one of the administrators:

If you want more information on JIRA, or have a bug to report see:

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message