Author: thomas.diesler(a)jboss.com Date: 2008-02-26 13:25:04 -0500 (Tue, 26 Feb 2008) New Revision: 5816 Modified: framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DeploymentAspectManagerImpl.java Log: Improve installer logging Modified: framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DeploymentAspectManagerImpl.java =================================================================== --- framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DeploymentAspectManagerImpl.java 2008-02-26 18:14:52 UTC (rev 5815) +++ framework/trunk/src/main/java/org/jboss/wsf/framework/deployment/DeploymentAspectManagerImpl.java 2008-02-26 18:25:04 UTC (rev 5816) @@ -80,13 +80,13 @@ { depAspects.clear(); depAspects.addAll(aspects); - + // Debug the aspect list StringBuilder builder = new StringBuilder("setDeploymentAspects on " + name); for (DeploymentAspect aspect : aspects) - builder.append("\n " + aspect.getClass().getName()); - - log.debug(builder); + builder.append("\n " + aspect.getClass().getName() + " provides: " + aspect.getProvidesAsSet() + " requires: " + aspect.getRequiresAsSet()); + + log.debug(builder); } /** @@ -105,14 +105,14 @@ // Check that all required aspects are met /* - TODO: This should done by adding all provided conditions to the Deployment - when being executed. Otherwise we will miss the parent provided conditions here - - Set<String> requiredSet = aspect.getRequiresAsSet(); - requiredSet.remove(DeploymentAspect.LAST_DEPLOYMENT_ASPECT); - if (providedConditions.containsAll(requiredSet) == false) - throw new IllegalStateException("Required conditions '" + aspect.getRequires() + "' not satisfied by '" + providedConditions + "' for: " + aspect); - */ + TODO: This should done by adding all provided conditions to the Deployment + when being executed. Otherwise we will miss the parent provided conditions here + + Set<String> requiredSet = aspect.getRequiresAsSet(); + requiredSet.remove(DeploymentAspect.LAST_DEPLOYMENT_ASPECT); + if (providedConditions.containsAll(requiredSet) == false) + throw new IllegalStateException("Required conditions '" + aspect.getRequires() + "' not satisfied by '" + providedConditions + "' for: " + aspect); + */ logInvocation(aspect, "Create"); aspect.create(dep);