[Jboss-cvs] JBossAS SVN: r55146 - branches/MC_VDF_WORK/system/src/main/org/jboss/deployers/plugins

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

Author: scott.stark at jboss.org
Date: 2006-08-03 19:48:43 -0400 (Thu, 03 Aug 2006)
New Revision: 55146

Modified:
   branches/MC_VDF_WORK/system/src/main/org/jboss/deployers/plugins/MainDeployerImpl.java
Log:
Improve the error reporting for a deployment for which no deployers exist.

Modified: branches/MC_VDF_WORK/system/src/main/org/jboss/deployers/plugins/MainDeployerImpl.java
===================================================================
--- branches/MC_VDF_WORK/system/src/main/org/jboss/deployers/plugins/MainDeployerImpl.java	2006-08-03 23:38:03 UTC (rev 55145)
+++ branches/MC_VDF_WORK/system/src/main/org/jboss/deployers/plugins/MainDeployerImpl.java	2006-08-03 23:48:43 UTC (rev 55146)
@@ -270,12 +270,17 @@
       try
       {
          if( parse(root, rootDeployment, null) == false )
-            throw new DeploymentException("Failed to parse deployment");
+            throw new DeploymentException("No deployers accepted: "+deployURL);
       }
       catch(IOException e)
       {
          throw new DeploymentException("Failed to parse deployment", e);
       }
+      // If no deployer accepted this fail
+      if( rootDeployment.getType() == null )
+         throw new DeploymentException("No deployers available for: "+deployURL);
+      if( rootDeployment.getRootContext() == null )
+         throw new DeploymentException("No deployment root created for: "+deployURL);
       return rootDeployment;
    }