Details
-
Type: Improvement
-
Status: Closed
-
Priority: Major
-
Resolution: Fixed
-
Affects Version/s: None
-
Fix Version/s: 1.2
-
Component/s: PicoContainer (Java)
-
Labels:None
-
Number of attachments :
Description
The current impl of DefaultLifecycleManger is flawed. The one registered against the root container will affect all child containers regardless of whether they may have been instantiated with their own impl of LifecycleManager. This is because the visitor impementation is a bit overzealous.
Aslak and I have talked of fixing this. I changed it locally and let Aslak look at the changes. He's mulled it and proposed an alternate fix to the problem. That was about eight weeks ago, and we both sit not applying our loal changes.
This issue should give us a chance to upload our patches for some debate on the way ahead.
Activity
Paul Hammant
made changes -
Field | Original Value | New Value |
---|---|---|
Attachment | DefaultLifecycleManager.patch [ 13651 ] |
Paul Hammant
made changes -
Attachment | RootVisitingLifecycleManager.java [ 13652 ] |
Paul Hammant
made changes -
Attachment | RootVisitingLifecycleManagerTestCase.java [ 13653 ] |
Jörg Schaible
made changes -
Environment | ||
Fix Version/s | 1.2 [ 11330 ] |
Paul Hammant
made changes -
Status | Open [ 1 ] | Closed [ 6 ] |
Resolution | Fixed [ 1 ] |
Patch for Pauls 'better' DLM