Re: [REVIEW] ConfigureListener - check for FacesServlet before performing faces configuration steps

From: Ed Burns <Ed.Burns_at_Sun.COM>
Date: Wed, 23 Feb 2005 14:21:52 -0800

>>>>> On Wed, 23 Feb 2005 16:14:02 -0500, Ryan Lubke <Ryan.Lubke_at_Sun.COM> said:

RL> Updated to include a context init parameter
RL> 'com.sun.faces.forceLoadConfiguration' which will
RL> cause the configuration logic to be processed whether or not an entry
RL> for the FacesServet is
RL> present.

RL> Ryan Lubke wrote:

>> Consider the case where JSF is installed in the common classloader of
>> Tomcat, or SJSAS.
>> With this configuration, the ConfigureListener will be invoked for
>> every web application
>> whether or not it uses JSF.
>> This modification to ConfigureListener will process the web.xml of the
>> current web application
>> scanning for the presence of javax.faces.webapp.FacesServlet. If
>> found, perform the normal
>> application processing, otherwise return.
>> I've validated that this is working using Tomcat and deploying some
>> applications that
>> have a FacesServlet entry and others that do not.
RL> SECTION: Modified Files
RL> ----------------------------
RL> M src/com/sun/faces/config/

I don't like how this checkin mixes code formatting with logic, but
let's ignore that nit.

Otherwise, looks good. r=edburns


|  | {home: 407 294 2468, office: 408 884 9519 OR x31640}
| homepage:         |
| aim: edburns0sunw | iim: