[jboss-dev-forums] [Design the new POJO MicroContainer] - Re: Bean property inspection and actual injection

[adrian@jboss.org]

What were you trying to fix here?


  | Author: alesj
  | Date: 2006-08-13 12:26:29 -0400 (Sun, 13 Aug 2006)
  | New Revision: 55853
  | 
  | Modified:
  |    projects/microcontainer/trunk/dependency/src/main/org/jboss/dependency/plugins/AbstractController.java
  |    projects/microcontainer/trunk/kernel/src/tests/org/jboss/test/kernel/inject/test/PropertyContextualInjectionTestCase.java
  | Log:
  | AbstractController.allContexts.remove after failure in confiruration phase.
  | Added NullName expected throwable.
  | 
  | Modified: projects/microcontainer/trunk/dependency/src/main/org/jboss/dependency/plugins/AbstractController.java
  | ===================================================================
  | --- projects/microcontainer/trunk/dependency/src/main/org/jboss/dependency/plugins/AbstractController.java      2006-08-13 15:45:00 UTC (rev 55852)
  | +++ projects/microcontainer/trunk/dependency/src/main/org/jboss/dependency/plugins/AbstractController.java      2006-08-13 16:26:29 UTC (rev 55853)
  | @@ -443,6 +443,7 @@
  |           {
  |              log.error("Error installing to " + toState.getStateString() + ": " + context.toShortString(), error);
  |              uninstallContext(context, ControllerState.NOT_INSTALLED, trace);
  | +            allContexts.remove(context.getName()); //missing if contexts failes during configuration phase
  |              errorContexts.add(context);
  |              context.setError(error);
  |              return false;
  | 

This breaks the validate() method and makes it impossible
to retrieve the error that occurred since the installed context
no longer exists. (The controller has lost all reference to it).

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

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