[PICO-380] Allow specifying annotations to use in PicoBuilder.withAnnotated*Injection()

Details

Type: Improvement
Status: Resolved
Priority: Major
Resolution: Fixed
Affects Version/s: None
Fix Version/s: 3.0
Component/s: PicoContainer (Java)
Labels:
None

Patch Submitted:

Yes
Number of attachments :
2

Description

Here's a patch for this.

Let me know what you think.

Additionally, I wish there was a way to simply add other injectors. It arguably makes PicoBuilder more complex, but at least a protected method subclasses could use would be sweet.

Options
- Sort By Name
- Sort By Date
- Ascending
- Descending
- Download All

Attachments

PICO-380.patch

20/Jan/11 1:17 PM

2 kB

Grégory Joseph
PICO-380-bis.patch

20/Jan/11 1:31 PM

4 kB

Grégory Joseph

Activity

Hide

Permalink

Grégory Joseph added a comment - 20/Jan/11 1:31 PM

Here's a second version of this patch which extracts a protected addInjector() method, allowing subclassing of PicoBuilder for added functionality. (i.e this would save me now, I'd be able to add my injectors-with-custom-annotations)

Show

Grégory Joseph added a comment - 20/Jan/11 1:31 PM Here's a second version of this patch which extracts a protected addInjector() method, allowing subclassing of PicoBuilder for added functionality. (i.e this would save me now, I'd be able to add my injectors-with-custom-annotations)

Hide

Permalink

Paul Hammant added a comment - 02/Feb/11 7:34 AM

Applied to Pico2 and Pico3

Show

Paul Hammant added a comment - 02/Feb/11 7:34 AM Applied to Pico2 and Pico3

People

Assignee:

Paul Hammant

Reporter:

Grégory Joseph

Votes:

0 Vote for this issue

Watchers:

0 Start watching this issue

Dates

Created:

20/Jan/11 1:17 PM

Updated:

02/Feb/11 7:34 AM

Resolved:

02/Feb/11 7:34 AM