[jboss-dev-forums] [Design the new POJO MicroContainer] - Re: Wrong dependency info after module uninstall

[adrian@jboss.org]

Ok, now I've seen the commit, I understand what you are talking about.
You changed the api in an incompatible way and didn't update the class hierarchy.

There is a similar issue with the demand dependency which is the original
version of depending on an abstract concept and what I copied to create
the RequirementDependencyItem..

I've added @Override to the methods to ensure they are actually overridding
things, but more importantly, I've changed the AbstractDependencyItem
to do what you should have done in the first place:

Compatible change:

  |    // Old method possibly overridden by subclasses
  |    public void unresolved()
  |    {
  |       if (resolved)
  |       {
  |          resolved = false;
  |          flushJBossObjectCache();
  |          log.trace("Forced unresolved " + this);
  |       }
  |    }
  |    
  |    // New method just calls the old method
  |    public boolean unresolved(Controller controller)
  |    {
  |       unresolved();
  |       return true;
  |    }
  | 

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

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