cocoon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Leszek Gawron <lgaw...@mobilebox.pl>
Subject Re: svn commit: r437365 - in /cocoon/trunk/core/cocoon-core/src/main: java/org/apache/cocoon/components/treeprocessor/sitemap/ java/org/apache/cocoon/core/container/spring/ resources/org/apache/cocoon/core/container/spring/
Date Mon, 28 Aug 2006 07:40:28 GMT
cziegeler@apache.org wrote:
> Modified: cocoon/trunk/core/cocoon-core/src/main/java/org/apache/cocoon/core/container/spring/CocoonNamespaceHandler.java
> URL: http://svn.apache.org/viewvc/cocoon/trunk/core/cocoon-core/src/main/java/org/apache/cocoon/core/container/spring/CocoonNamespaceHandler.java?rev=437365&r1=437364&r2=437365&view=diff
> ==============================================================================
> --- cocoon/trunk/core/cocoon-core/src/main/java/org/apache/cocoon/core/container/spring/CocoonNamespaceHandler.java
(original)
> +++ cocoon/trunk/core/cocoon-core/src/main/java/org/apache/cocoon/core/container/spring/CocoonNamespaceHandler.java
Sun Aug 27 04:03:35 2006
> @@ -24,6 +24,8 @@
>   * (in the namespace "http://org.apache.cocoon/core"):
>   * "settings" : This sets up the Cocoon Settings object (by reading the property
>   *              files located under /WEB-INF/cocoon/properties.
> + *              By specifying the attribute "processorClassName" an own implementation
> + *              can be used (this should be a subclass of the {@link SettingsBeanFactoryPostProcessor}).
Wouldn't it be better if instead of processorClassName it was a bean 
reference? There is no way to do any configuration if only class name is 
passed.

I am still pushing for the ability to test developments blocks in 
isolation which would require something like:

<cocoon:properties additionalProcessor="localProcessor"/>

<bean id="localProcessor" class="sth">
	<property name="propertyPath"
           value="src/main/resources/META-INF/properties"/>
</bean>

-- 
Leszek Gawron, IT Manager                          MobileBox sp. z o.o.
+48 (61) 855 06 67                              http://www.mobilebox.pl
mobile: +48 (501) 720 812                       fax: +48 (61) 853 29 65

Mime
View raw message