[jboss-svn-commits] JBL Code SVN: r20379 - in labs/jbossesb/workspace/maeste: product/rosetta/src/org/jboss/soa/esb/notification and 1 other directory.

[jboss-svn-commits at lists.jboss.org]

Author: maeste
Date: 2008-06-09 11:30:58 -0400 (Mon, 09 Jun 2008)
New Revision: 20379

Modified:
   labs/jbossesb/workspace/maeste/
   labs/jbossesb/workspace/maeste/product/rosetta/src/org/jboss/soa/esb/notification/NotificationTarget.java
Log:
Merged revisions 20357-20378 via svnmerge from 
https://svn.labs.jboss.com/labs/jbossesb/trunk

........
  r20370 | tfennelly | 2008-06-09 15:04:58 +0200 (Mon, 09 Jun 2008) | 1 line
  
  http://jira.jboss.com/jira/browse/JBESB-1798
........
  r20371 | tfennelly | 2008-06-09 16:31:08 +0200 (Mon, 09 Jun 2008) | 3 lines
  
  http://jira.jboss.com/jira/browse/JBESB-1798
  
  I gambled and lost :-)
........



Property changes on: labs/jbossesb/workspace/maeste
___________________________________________________________________
Name: svnmerge-integrated
   - /labs/jbossesb/trunk:1-20356
   + /labs/jbossesb/trunk:1-20378

Modified: labs/jbossesb/workspace/maeste/product/rosetta/src/org/jboss/soa/esb/notification/NotificationTarget.java
===================================================================
--- labs/jbossesb/workspace/maeste/product/rosetta/src/org/jboss/soa/esb/notification/NotificationTarget.java	2008-06-09 15:16:36 UTC (rev 20378)
+++ labs/jbossesb/workspace/maeste/product/rosetta/src/org/jboss/soa/esb/notification/NotificationTarget.java	2008-06-09 15:30:58 UTC (rev 20379)
@@ -22,13 +22,14 @@
 
 package org.jboss.soa.esb.notification;
 
-import java.lang.reflect.Constructor;
-
 import org.jboss.soa.esb.ConfigurationException;
 import org.jboss.soa.esb.helpers.ConfigTree;
 import org.jboss.soa.esb.message.Message;
 import org.jboss.soa.esb.util.ClassUtil;
 
+import java.lang.reflect.Constructor;
+import java.lang.reflect.InvocationTargetException;
+
 /**
  * Abstract class to define expected behaviour of all NotificationTargets and
  * provide some common functionality to all of them
@@ -142,13 +143,15 @@
 		try
 		{
 			oRet = oCons.newInstance(new Object[] { p_oP });
-		}
-		catch (Exception e)
-		{
-			e.printStackTrace();
-		}
-		if (null == oRet || (!(oRet instanceof NotificationTarget)))
-			throw new ConfigurationException(sClass + "  does not extend NotificationTarget");
+        } catch (InvocationTargetException e) {
+            Throwable cause = e.getCause();
+            throw new ConfigurationException("Failed to create an instance of NotificationTarget class '" + sClass + "'.", (cause != null?cause:e));
+        } catch (Exception e) {
+            throw new ConfigurationException("Failed to create an instance of NotificationTarget class '" + sClass + "'.", e);
+        }
+        if (!(oRet instanceof NotificationTarget)) {
+            throw new ConfigurationException(sClass + "  does not extend NotificationTarget");
+        }
 
 		return (NotificationTarget) oRet;
 	} // __________________________________