Details
Improvement Request
Closed
Minor
-
- Description:
-
Hide
Because of the Java doc of this method, completed by the wiki, I think this method should be changed, by raising the limitation on the exception type
Keep MessageExchange#setError( Exception ), as stated by the JBI specification.
And implement Exchange#setError( Exception ) with:if (exception instanceof MessagingException || ename.startsWith("java.") || ename.startsWith("org.w3c.") || ename.startsWith("org.xml.")) { exchange.setError(exception); } else { MessagingException msgException = new MessagingException(ExceptionUtil .getExtendedMessage(exception)); exchange.setError(msgException); }
That would be useful in the developer kit.
ShowBecause of the Java doc of this method, completed by the wiki, I think this method should be changed, by raising the limitation on the exception type Keep MessageExchange#setError( Exception ), as stated by the JBI specification. And implement Exchange#setError( Exception ) with:That would be useful in the developer kit.if (exception instanceof MessagingException || ename.startsWith("java.") || ename.startsWith("org.w3c.") || ename.startsWith("org.xml.")) { exchange.setError(exception); } else { MessagingException msgException = new MessagingException(ExceptionUtil .getExtendedMessage(exception)); exchange.setError(msgException); }
-
- Environment:
- All.
Activity
| Field | Original Value | New Value |
|---|---|---|
| Status | New [ 10000 ] | Open [ 10002 ] |
| Priority | Minor [ 4 ] |
| Description |
According to the Java doc of this method, and completed by [the wiki|http://doc.petalslink.com/display/petalsesb30/Advanced+topics+and+guidelines#Advancedtopicsandguidelines-Theproblematic], this method should be changed, by raising the limitation on the exception type
Keep MessageExchange#setError( Exception ), as stated by the JBI specification. And implement Exchange#setError( Exception ) with: {code:lang=java} if (exception instanceof MessagingException || ename.startsWith("java.") || ename.startsWith("org.w3c.") || ename.startsWith("org.xml.")) { exchange.setError(exception); } else { MessagingException msgException = new MessagingException(ExceptionUtil .getExtendedMessage(exception)); exchange.setError(msgException); } {code} That would be useful in the developer kit. ;) |
Because of the Java doc of this method, completed by [the wiki|http://doc.petalslink.com/display/petalsesb30/Advanced+topics+and+guidelines#Advancedtopicsandguidelines-Theproblematic], I think this method should be changed, by raising the limitation on the exception type
Keep MessageExchange#setError( Exception ), as stated by the JBI specification. And implement Exchange#setError( Exception ) with: {code:lang=java} if (exception instanceof MessagingException || ename.startsWith("java.") || ename.startsWith("org.w3c.") || ename.startsWith("org.xml.")) { exchange.setError(exception); } else { MessagingException msgException = new MessagingException(ExceptionUtil .getExtendedMessage(exception)); exchange.setError(msgException); } {code} That would be useful in the developer kit. ;) |
| Status | Open [ 10002 ] | In Progress [ 10003 ] |
| Status | In Progress [ 10003 ] | Resolved [ 10004 ] |
| Fix Version/s | 5.1.1 [ 10152 ] | |
| Resolution | Fixed [ 1 ] |
| Status | Resolved [ 10004 ] | Closed [ 10005 ] |
| Transition | Status Change Time | Execution Times | Last Executer | Last Execution Date | |||||||||
|
|
|
|
|
|||||||||
|
|
|
|
|
|||||||||
|
|
|
|
|
|||||||||
|
|
|
|
|
great idea