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#... Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&a...