thank you matt, Daniel can you include this if correct?
yes you are right, we never put the "how to contribute" online
thank you
marc
|-----Original Message-----
|From: [EMAIL PROTECTED]
|[mailto:[EMAIL PROTECTED]]On Behalf Of Matthew Cooper
|Sent: Tuesday, November 21, 2000 5:32 PM
|To: jboss dev
|Subject: [jBoss-Dev] Contribution : J2eeDeployer patch to
|reportRuntimeErrorException detail
|
|
|Hi guys,
|
|I am having a problem with my servlets that was throwing a
|ClassNotFoundException. This is just being reported as a
|RuntimeErrorException so you cannot see what the cause is. I have made a
|fix to the J2eeDeployer to log the underlying error.
|
|I could not find anywhere on the site how to make contributions - has
|that info been lost in the site re-orgs?
|
|Anyway, I have pasted in a diff file but please let me know how to make
|contributions correctly.
|
|Regards, Matty.
|
|Matthew Cooper
|NUIX Pty Ltd
|
|
|
|
|--- src/main/org/jboss/deployment/J2eeDeployer.java 2000/11/10
|23:33:30 1.11
|+++ src/main/org/jboss/deployment/J2eeDeployer.java 2000/11/22
|01:01:48
|@@ -27,6 +27,7 @@
| import javax.management.MBeanServer;
| import javax.management.MBeanException;
| import javax.management.JMException;
|+import javax.management.RuntimeErrorException;
| import javax.management.ObjectName;
|
| import org.jboss.logging.Log;
|@@ -137,7 +138,20 @@
| startApplication (d);
|
| log.log ("J2EE application: " + _url + " is deployed.");
|- }
|+ }
|+ // not sure why the catch is coded as "if (.. instanceof ..)"
|rather than using
|+ // the built in catch structures so I'm doing mine in a more
|traditional
|+ // (IMHO) java way (Matty)
|+ catch (RuntimeErrorException e)
|+ {
|+ // log the cause
|+ log.exception(e.getTargetError());
|+ // log the exception
|+ log.exception(e);
|+ uninstallApplication (getAppName (url));
|+ // and report the underlying exception (or should this be the
|RuntimeErrorException?)
|+ throw new J2eeDeploymentException ("Fatal error:
|"+e.getTargetError().toString());
|+ }
| catch (Exception _e)
| {
| if (_e instanceof J2eeDeploymentException)
|
|
