[ 
https://issues.apache.org/jira/browse/COCOON-2078?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Grzegorz Kossakowski closed COCOON-2078.
----------------------------------------

    Resolution: Fixed

Patch committed in r555257.
Closing the issue, reopen if something is wrong.

> Profiling pipelines must not implement Configurable interface as they are 
> already Parameterizable
> -------------------------------------------------------------------------------------------------
>
>                 Key: COCOON-2078
>                 URL: https://issues.apache.org/jira/browse/COCOON-2078
>             Project: Cocoon
>          Issue Type: Bug
>          Components: Blocks: Profiler
>    Affects Versions: 2.2-dev (Current SVN)
>            Reporter: Alexander Klimetschek
>            Assignee: Grzegorz Kossakowski
>            Priority: Critical
>             Fix For: 2.2-dev (Current SVN)
>
>         Attachments: profiler-config-vs-param-fix.patch
>
>
> The most-recent change to both Profiling pipelines (revision 503964 in 
> February 07) added the configuration "sax-stream" option and this was done by 
> additionally implementing the Configurable interface. But as in 
> BaseCachingProcessingPipeline (a base class) the interface Parameterizable is 
> already implemented, and an Avalon component can be either one or the other 
> [1], this is a bug. It's critical since the cache is retrieved in 
> BaseCachingProcessingPipeline.parameterize(), but as Configurable has 
> precedence over Parameterizable [2], the cache will be null and a NPE will be 
> thrown during generation phase.
> [1] 
> http://excalibur.apache.org/apidocs/org/apache/avalon/framework/parameters/Parameterizable.html
> [2] 
> http://svn.apache.org/viewvc/cocoon/trunk/core/cocoon-sitemap/cocoon-sitemap-impl/src/main/java/org/apache/cocoon/core/container/spring/avalon/AvalonBeanPostProcessor.java?view=markup,
>  see method postProcessBeforeInitialization()

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to