cocoon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Vadim Gritsenko <>
Subject Re: JXTemplate is broken by implementing Recycable interface
Date Thu, 11 Oct 2007 22:00:12 GMT
Leszek Gawron wrote:
> Vadim Gritsenko wrote:
>> Leszek Gawron wrote:
>>> Joerg Heinicke wrote:
>>>> On 10.10.2007 4:45 Uhr, Leszek Gawron wrote:
>>>>> 2. Look at SitemapModelComponent:
>>>>>>     void setup(SourceResolver resolver, Map objectModel, String 
>>>>>> src, Parameters par)     throws ProcessingException, SAXException,

>>>>>> IOException;
>>>>>> }
>>>>> It looks like we could drop the source resolver and objectModel. 
>>>>> Both can be injected and AFAIU both are properly scoped with some 
>>>>> custom scope of ours.
>>>> But that's an interface and the method is probably called by the 
>>>> pipeline. You still have to support both Avalon and Spring and need 
>>>> at least a migration path in the pipeline.
>>> Of course not now but that is just a pointer that spring invalidates 
>>> somehow this interface.
>> May be a bit of RT, but would it be possible to replace this interface 
>> with pipeline scope and injection of parameters. Resolver and object 
>> model already can be injected, IIUC. Same could be extended to source 
>> and parameters. WDYT?
> This is exactly what I indicated. There is a little problem with this 
> though. Currently SitemapModelComponent.setup() apart from setting 
> component context is also able to perform some logic (like in JXTG 
> resolving and parsing jx template). If there is no such method this code 
> would have to be moved to generate() which may introduce unknown side 
> effects.

We could have empty setup() method for components in this scope. WDYT?


View raw message