Re: [weld-dev] @Inject FacesContext
Ah. Hrm. A good point, which I had not thought of, is that you can't
assign it to an instance variable.
But you can use a static import, and just write: getFacesContext()
everywhere. Easy.
Like I said, I can add this in a maintenance release if its important.
On Tue, Nov 10, 2009 at 5:21 PM, Dan Allen <dan.j.allen(a)gmail.com> wrote:
> The reason for injection is to be able to stub and unit test easily. Right
> now the getCurrentInstance() is static and you'd need to use ugly
> constructions to replace it with a stub (I am not saying it's impossible but
> I am talking here from the feedback I've gathered and personal experience
> when unit testing pure JSF).
And the other, more subtle annoyance is the you have to eventually make that
decision in a method when to create a local variable for FacesContext
because your code is getting too verbose.
FacesContext ctx = FacesContext.getCurrentInstance();
I hate writing that line. Every time I do, I think to myself "I hate this
stupid line of code."
-Dan
--
Dan Allen
Senior Software Engineer, Red Hat | Author of Seam in Action
Registered Linux User #231597
http://mojavelinux.com
http://mojavelinux.com/seaminaction
http://www.google.com/profiles/dan.j.allen
--
Gavin King
gavin.king(a)gmail.com
http://in.relation.to/Bloggers/Gavin
http://hibernate.org
http://seamframework.org