@javax.inject.Inject support

Bug #460626 reported by Guillaume Bort
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
play framework
Status tracked in 1.0
1.0
Fix Released
Undecided
Guillaume Bort
1.1
Fix Committed
Undecided
Unassigned

Bug Description

We need to support this annotation in the Spring module. It seems a little tricky to do however. My first idea is to instrument all constructors in order to correctly set the annotated fields. We will need to instrument static fields as well, using a static block initializer.

Revision history for this message
Nicolas Leroux (nicolas-lunatech) wrote :

Does that only concern the spring module? I can think of other possible use cases... (Inject pojo, look up EJBs for easy migration to play), however that makes it a rather large issue.

Revision history for this message
Guillaume Bort (guillaume-bort) wrote : Re: [Bug 460626] Re: @javax.inject.Inject support

yes of course I think we could make it built-in in play.

On Sun, Oct 25, 2009 at 9:02 PM, Nicolas <email address hidden> wrote:
> Does that only concern the spring module? I can think of other possible
> use cases... (Inject pojo, look up EJBs for easy migration to play),
> however that makes it a rather large issue.
>
> --
> @javax.inject.Inject support
> https://bugs.launchpad.net/bugs/460626
> You received this bug notification because you are a member of play
> framework developers, which is subscribed to play framework.
>

Revision history for this message
Guillaume Bort (guillaume-bort) wrote :

Injection in controller is available since 1.0r648. But do we need more ?

Changed in play:
status: New → In Progress
assignee: nobody → Guillaume Bort (guillaume-bort)
Changed in play:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.