[JBoss JIRA] Created: (JBMICROCONT-330) ScopeInfo should use the existing Mutable metadata if it already exists
by Adrian Brock (JIRA)
ScopeInfo should use the existing Mutable metadata if it already exists
-----------------------------------------------------------------------
Key: JBMICROCONT-330
URL: https://jira.jboss.org/jira/browse/JBMICROCONT-330
Project: JBoss MicroContainer
Issue Type: Bug
Reporter: Adrian Brock
Fix For: JBossMC.2.0.0.CR1
Currently the AbstractScopeInfo class is always creating a new Mutable MetaMeta context within the MDR.
This would cause a problem where the scope has already been constructed, e.g. by the deployers.
public void addMetaData(MutableMetaDataRepository repository, ControllerContext context)
{
this.repository = repository;
ScopeKey scope = getMutableScope();
// Needs a check here for already exists - see the version of getMutableMetaData in AbstractDeploymentContext
MemoryMetaDataLoader mutable = new MemoryMetaDataLoader(scope);
repository.addMetaDataRetrieval(mutable);
addMetaData(repository, context, mutable);
}
Similarly ScopeInfo::removeMetaData(), should not be removing MDR contexts it didn't construct.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
17 years, 11 months
[JBoss JIRA] Created: (JBMICROCONT-332) Logic error when checking controller context has no dependencies
by Adrian Brock (JIRA)
Logic error when checking controller context has no dependencies
----------------------------------------------------------------
Key: JBMICROCONT-332
URL: https://jira.jboss.org/jira/browse/JBMICROCONT-332
Project: JBoss MicroContainer
Issue Type: Bug
Components: Dependency
Affects Versions: JBossMC.2.0.0.Beta16
Reporter: Adrian Brock
Assignee: Adrian Brock
Fix For: JBossMC.2.0.0.CR1
There is a logic error in AbstractController when a ControllerContext has no dependencies
if (dependencies != null && dependencies.resolveDependencies(this, state))
result.add(ctx);
Where result is the set of contexts that are eligible to advance to the next level.
This should of course be:
if (dependencies == null || dependencies.resolveDependencies(this, state))
result.add(ctx);
Since a context with no dependencies can always advance to the next level.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
17 years, 11 months