> -----Original Message-----
> From: Craig R. McClanahan [mailto:[EMAIL PROTECTED]]
> Sent: Saturday, October 12, 2002 10:56 AM
> To: Struts Developers List
> Subject: RE: cvs commit:
> jakarta-struts/src/share/org/apache/struts/valida tor
> ValidatorPlugIn.java
> 
> 
> 
> 
> On Fri, 11 Oct 2002, Martin Cooper wrote:
> 
> > Date: Fri, 11 Oct 2002 15:56:37 -0700
> > From: Martin Cooper <[EMAIL PROTECTED]>
> > Reply-To: Struts Developers List <[EMAIL PROTECTED]>
> > To: 'Struts Developers List' <[EMAIL PROTECTED]>
> > Subject: RE: cvs commit:
> >     jakarta-struts/src/share/org/apache/struts/valida tor 
> ValidatorPlugIn.java
> >
> > The new version of getValidatorResources() added here takes 
> a ServletContext
> > and an HttpServletRequest. It seems to me that it should 
> only take the
> > request, since the servlet context can be obtained from 
> that. It would be a
> > little less error-prone this way, too.
> >
> 
> Umm, how does one get access to the ServletContext via a request?

I was thinking that we could get it via the ApplicationConfig, which used to
hold a reference to the ActionServlet. I'd forgotten, however, that that
reference had to be removed in the interests of serialisability.

So the current code stands.

> 
> An unpleasant way is 
> request.getSession().getServletContext(), but that
> requires creating a session.  Worse, it's a Servlet 2.3 thing and not
> available on our minimum required platform for Struts.

No, I wasn't thinking about that.

--
Martin Cooper


> 
> > My 2 cents.
> >
> > --
> > Martin Cooper
> 
> Craig
> 
> 
> --
> To unsubscribe, e-mail:   
> <mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail: 
> <mailto:[EMAIL PROTECTED]>
> 
> 


--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to