[jboss-dev-forums] [Design the new POJO MicroContainer] - Re: Checking DeploymentContext for failures

[alesj]

I believe I'm missing the check of components being in failure in DeployersImpl.checkComplete:

  |    protected final void checkComplete(
  |          DeploymentContext context,
  |          Map<String, Throwable> contextsInError,
  |          Map<String, Set<MissingDependency>> contextsMissingDependencies,
  |          Set<ControllerContext> notInstalled,
  |          List<ControllerState> states)
  |    {
  |       DeploymentControllerContext dcc = context.getTransientAttachments().getAttachment(ControllerContext.class.getName(), DeploymentControllerContext.class);
  |       checkControllerContext(dcc, contextsInError, contextsMissingDependencies, notInstalled, states);
  | 
  |       Set<Object> names = context.getControllerContextNames();
  |       if (names != null && names.isEmpty() == false)
  |       {
  |          for(Object name : names)
  |          {
  |             ControllerContext cc = controller.getContext(name, null);
  |             checkControllerContext(cc, contextsInError, contextsMissingDependencies, notInstalled, states);
  |          }
  |       }
  | 
  |       List<DeploymentContext> children = context.getChildren();
  |       if (children != null && children.isEmpty() == false)
  |       {
  |          for(DeploymentContext child : children)
  |             checkComplete(child, contextsInError, contextsMissingDependencies, notInstalled, states);
  |       }
  |       
  |       // CURRENTLY MISSING
  |       List<DeploymentContext> components = context.getComponents();
  |       if (components != null && components.isEmpty() == false)
  |       {
  |          for(DeploymentContext component : components)
  |             checkComplete(component, contextsInError, contextsMissingDependencies, notInstalled, states);
  |       }
  |    }
  | 

btw: What's the difference between a component and a child context?

View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4123910#4123910

Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4123910