Details
-
Type:
New Feature
-
Status:
Closed
-
Priority:
Major
-
Resolution: Won't Fix
-
Affects Version/s: None
-
Fix Version/s: None
-
Component/s: PicoContainer (Java)
-
Labels:None
-
Number of attachments :
Description
Here's a patch which implements a MapPicoContainer to support registering of components in a PicoContainer.
There are some minor patches to AggregateContainersContainer to make it easier to reuse.
Activity
james strachan
made changes -
| Field | Original Value | New Value |
|---|---|---|
| Attachment | patch.txt [ 10450 ] |
Here's a later patch which includes 100% coverage of aggregated and map packages
Show
james strachan
added a comment - Here's a later patch which includes 100% coverage of aggregated and map packages
james strachan
added a comment - Here's a later patch which includes 100% coverage of aggregated and map packages james strachan
made changes -
| Attachment | patch.txt [ 10452 ] |
Doh! Previous patches didn't include the new source files.
Sorry for this. This new patch includes all the others and includes the new Java files and JUnit test cases with 100% test coverage on the aggregate and map packages.
The included MapPicoContainer behaves like a keyed registry container.
Show
james strachan
added a comment - Doh! Previous patches didn't include the new source files.
Sorry for this. This new patch includes all the others and includes the new Java files and JUnit test cases with 100% test coverage on the aggregate and map packages.
The included MapPicoContainer behaves like a keyed registry container.
james strachan
added a comment - Doh! Previous patches didn't include the new source files.
Sorry for this. This new patch includes all the others and includes the new Java files and JUnit test cases with 100% test coverage on the aggregate and map packages.
The included MapPicoContainer behaves like a keyed registry container. james strachan
made changes -
| Attachment | patch.txt [ 10453 ] |
Please close this issue now
Show
james strachan
added a comment - Please close this issue now
james strachan
added a comment - Please close this issue now Aslak Hellesøy
made changes -
| Resolution | Won't Fix [ 2 ] | |
| Status | Unassigned [ 1 ] | Closed [ 6 ] |
Here's the patch. It also includes an Eclipse project file which you can ignore if you wish together with a minor patch to maven.xml which seems to fix the build on 1.0 b9 which again by all means ignore. Its the Java code changes in src/test and src/java that are important.