Issue Details (XML | Word | Printable)

Key: JELLY-61
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: peter royal
Reporter: Brian Ewins
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Commons Jelly

InvocationTargetExceptions not unwrapped by DynamicBeanTag

Created: 17/Jul/03 10:14 PM   Updated: 04/Sep/03 11:36 PM
Return to search
Component/s: core / taglib.core
Affects Version/s: None
Fix Version/s: None

Time Tracking:
Not Specified

Environment: all

Resolution Date: 04/Sep/03 11:36 PM


 Description  « Hide
In org.apache.commons.jelly.impl.DynamicBeanTag.doTag(), there is a typo that is preventing clients seeing the correct exception:

catch (InvocationTargetException e) {
// methodInvocationError(bean, method, e);
Throwable inner = e.getTargetException();
throw new JellyTagException(e);
}

should read:
catch (InvocationTargetException e) {
// methodInvocationError(bean, method, e);
Throwable inner = e.getTargetException();
throw new JellyTagException(inner);
}

(typo is on the last line)



 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
peter royal added a comment - 21/Jul/03 10:33 PM
looks pretty easy.. talking over the issue so i remember to do it

peter royal added a comment - 28/Jul/03 02:50 AM
applied, thanks! please cross-check.