10:14 a.m.
I'm not disputing that CDI should provide it there, but Weld is the *reference*
implementation of the CDI spec so needs to provide a reference for other people to
follow.
So IOW I would like guidance from the CDI EG that they intend to change it so servlet
supports conversations in the next iteration of the CDI spec.
Does that make some sense?
On 9 Apr 2010, at 16:09, Lincoln Baxter, III wrote:
I think that limiting the conversation to the JSF lifecycle is a big
drawback for out-of-the-box experience -- we should be providing this for the entire
servlet lifecycle. I'd argue that it is in the spirit of the new JavaEE 6 spec, which
clearly provides injection in Filters, Servlets and Listeners now -- we should be
providing Conversation support there as well. Filters and Servlets are still a strong part
of Java EE, the core part, in my opinion -- JSF is just one option.
--Lincoln
On Fri, Apr 9, 2010 at 5:57 AM, Pete Muir <pmuir(a)redhat.com> wrote:
I agree it's not in direct contradiction, but it doesn't seem to be in the spirit
of the spec either.
On 9 Apr 2010, at 10:52, Nicklas Karlsson wrote:
> We had been discussing with Dan and Lincoln on activating the conversation context
for the entire request as it didn't seem to be a direct contradiction of the spec and
Shane needed it for something about the same time so it got committed. I mailed the EG and
asked about it but we can revert for now.
>
> On Fri, Apr 9, 2010 at 12:13 PM, Pete Muir <pmuir(a)redhat.com> wrote:
> Guys,
>
> This doesn't seem in line with the spec to me, and I think we should revert it.
What is the motivation for doing it?
>
> Pete
>
> Begin forwarded message:
>
> > From: weld-commits(a)lists.jboss.org
> > Date: 1 April 2010 07:48:18 GMT+01:00
> > To: weld-commits(a)lists.jboss.org
> > Subject: [weld-commits] Weld SVN: r6071 - in
core/trunk/impl/src/main/java/org/jboss/weld: servlet and 1 other directory.
> > Reply-To: weld-commits(a)lists.jboss.org
> >
> > Author: shane.bryzak(a)jboss.com
> > Date: 2010-04-01 02:48:18 -0400 (Thu, 01 Apr 2010)
> > New Revision: 6071
> >
> > Modified:
> > core/trunk/impl/src/main/java/org/jboss/weld/jsf/WeldPhaseListener.java
> > core/trunk/impl/src/main/java/org/jboss/weld/servlet/ServletLifecycle.java
> > Log:
> > moved conversation setup/teardown out of jsf phase listener into servlet
lifecycle (Nik says to blame me if this doesn't work ;)
> >
> >
> > Modified:
core/trunk/impl/src/main/java/org/jboss/weld/jsf/WeldPhaseListener.java
> > ===================================================================
> > --- core/trunk/impl/src/main/java/org/jboss/weld/jsf/WeldPhaseListener.java
2010-03-31 03:21:16 UTC (rev 6070)
> > +++ core/trunk/impl/src/main/java/org/jboss/weld/jsf/WeldPhaseListener.java
2010-04-01 06:48:18 UTC (rev 6071)
> > @@ -118,7 +118,7 @@
> > private void beforeRestoreView(FacesContext facesContext)
> > {
> > log.trace(INITIATING_CONVERSATION, "Restore View");
> > - initiateSessionAndConversation(facesContext);
> > + //initiateSessionAndConversation(facesContext);
> > }
> >
> > /**
> > @@ -127,7 +127,7 @@
> > private void afterRenderResponse(FacesContext facesContext)
> > {
> > log.trace(CLEANING_UP_CONVERSATION, "Render Response",
"response complete");
> > - getConversationManager().teardownConversation();
> > + //getConversationManager().teardownConversation();
> > }
> >
> > /**
> > @@ -136,7 +136,7 @@
> > private void afterResponseComplete(FacesContext facesContext, PhaseId
phaseId)
> > {
> > log.trace(CLEANING_UP_CONVERSATION, phaseId, "the response has been
marked complete");
> > - getConversationManager().teardownConversation();
> > + //getConversationManager().teardownConversation();
> > }
> >
> > /**
> >
> > Modified:
core/trunk/impl/src/main/java/org/jboss/weld/servlet/ServletLifecycle.java
> > ===================================================================
> > --- core/trunk/impl/src/main/java/org/jboss/weld/servlet/ServletLifecycle.java
2010-03-31 03:21:16 UTC (rev 6070)
> > +++ core/trunk/impl/src/main/java/org/jboss/weld/servlet/ServletLifecycle.java
2010-04-01 06:48:18 UTC (rev 6071)
> > @@ -162,6 +162,7 @@
> > request.setAttribute(REQUEST_ATTRIBUTE_NAME, beanStore);
> > lifecycle.beginRequest(request.getRequestURI(), beanStore);
> > restoreSessionContext(request);
> > +
conversationManager(request.getSession().getServletContext()).setupConversation(request.getParameter("cid"));
> > }
> > }
> >
> > @@ -176,10 +177,12 @@
> > {
> > return;
> > }
> > - teardownSession(request);
> > +
> > +
conversationManager(request.getSession().getServletContext()).teardownConversation();
> > teardownRequest(request);
> > lifecycle.getConversationContext().setBeanStore(null);
> > lifecycle.getConversationContext().setActive(false);
> > + teardownSession(request);
> > }
> >
> > private void teardownSession(HttpServletRequest request)
> >
> > _______________________________________________
> > weld-commits mailing list
> > weld-commits(a)lists.jboss.org
> > https://lists.jboss.org/mailman/listinfo/weld-commits
>
>
> _______________________________________________
> weld-dev mailing list
> weld-dev(a)lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/weld-dev
>
>
>
> --
> ---
> Nik
_______________________________________________
weld-dev mailing list
weld-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/weld-dev
--
Lincoln Baxter, III
http://ocpsoft.com
http://scrumshark.com
"Keep it Simple"