[jboss-dev-forums] [Design the new POJO MicroContainer] - Re: New behaviour of AbstractMetaDataContext.isEmpty()

[adrian@jboss.org]

"adrian at jboss.org" wrote : 
  | But on a related issue, I don't like the fact that this logic is in the bridge.
  | The bridge should be just delegating the request to the retrieval.
  | i.e. a plain retrieval will just match its scope level while a context
  | will have to filter its local retrievals. It's more OO.

i.e. There should be a

  | List<MetaDataRetrieval> getScopedRetrievals(ScopeLevel level)
  | 
on the retrieval instance and the two branches of the if statement
moved into the relevant abstract implementation (context or retrieval).

P.S. I'd also guess that the testcase for this functionality isn't very good. ;-)

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

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