cocoon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Reinhard Poetz <reinh...@apache.org>
Subject Re: svn commit: r638759 - /cocoon/trunk/core/cocoon-sitemap/cocoon-sitemap-impl/src/main/java/org/apache/cocoon/components/treeprocessor/sitemap/PipelineNode.java
Date Wed, 19 Mar 2008 17:21:17 GMT
Vadim Gritsenko wrote:
> On Mar 19, 2008, at 5:39 AM, reinhard@apache.org wrote:
> 
>> Author: reinhard
>> Date: Wed Mar 19 02:39:07 2008
>> New Revision: 638759
>>
>> URL: http://svn.apache.org/viewvc?rev=638759&view=rev
>> Log:
>> fix a rather obscure problem with per-pipeline error handlers:
>>  If there is a per-pipeline handler in the last! pipeline of a sitemap,
>>  it also handled ResourceNotFoundExceptions although it is not 
>> responsible for it.
>> The solution is to move throwing this exception outside of the catch 
>> section which is responsible for catching pipeline errors.
> 
> You probably forgot that this was done intentionally. After your change, 
> error handlers do not work anymore the way they were supposed to work 
> (starting with Cocoon 2.0). Try this URL:
> 
>   http://localhost:8888/foo
> 
> It soon also should be available on zones (zones did not pick up this 
> changes yet):
> 
>   http://cocoon.zones.apache.org/demos/trunk/foo

Thanks for spotting this! Since I don't have the time to find a better solution, 
I reverted the change and also changed the integration tests so that they don't 
fail because of this bug anymore.

-- 
Reinhard Pötz                            Managing Director, {Indoqa} GmbH
                           http://www.indoqa.com/en/people/reinhard.poetz/

Member of the Apache Software Foundation
Apache Cocoon Committer, PMC member, PMC Chair        reinhard@apache.org
_________________________________________________________________________

Mime
View raw message