commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Howard Lin <>
Subject Re: [chain] questions about various servlet/*Mapper classes
Date Wed, 02 Mar 2005 22:02:06 GMT
Thanks, I'll file a bug in the Bugzilla. It seems the problem is that
in ChainProcessor, there is no need to save the catalog as a request
attribute since CatalogFactory already had it,  it only needs to save
the catalog name in the context so the next command can retrieve the
catalog name and use that to retrieve the catalog. If CONFIG_ATTR is
set, ChainProcessor should put that value in the context too so the
next command can retrieve value of CONFIG_ATTR and get the catalog
through the ServletContext. If this seems the right way to fix it, I
can submit patches.


On Tue, 1 Mar 2005 21:46:11 -0600, Joe Germuska <> wrote:
> At 4:17 PM -0500 3/1/05, Howard Lin wrote:
> >I'm trying to use ChainProcessor and various servlet/*Mapper classes
> >(like PathInfoMapper, ServletPathMapper etc.)  (version 1.0) and it
> >seems the following always return null catalog:
> >
> >Catalog catalog = (Catalog) context.get(getCatalogKey());
> It looks like the ChainServlet hasn't caught up with the preferred
> mechanism for managing catalogs.
> Since slightly before the 1.0 release, the CatalogFactory class was
> introduced, which manages an arbitrary number of named catalogs (as
> well as one "default" catalog.)  In your XML, the catalog takes a
> "name" attribute, and then is retrieved using
> CatalogFactory.getInstance().getCatalog("name")
> This should be filed as a bug in Bugzilla; I suspect the changes are
> relatively simple, but since I don't ever use the ChainServlet, I'm
> afraid it would take me quite a while to put together a decent test
> environment.  It would be great if someone using it could come up
> with some patches!
> Joe
> --
> Joe Germuska
> "Narrow minds are weapons made for mass destruction"  -The Ex
> ---------------------------------------------------------------------
> To unsubscribe, e-mail:
> For additional commands, e-mail:

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message