cocoon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Grzegorz Kossakowski <gkossakow...@apache.org>
Subject Re: svn commit: r586238 [2/2] - in /cocoon/trunk/core: cocoon-core/src/main/java/org/apache/cocoon/servlet/ cocoon-core/src/main/java/org/apache/cocoon/servlet/multipart/ cocoon-core/src/main/java/org/apache/cocoon/sitemap/ cocoon-core/src/main/java/org/ap...
Date Mon, 22 Oct 2007 17:30:39 GMT
vgritsenko@apache.org pisze:
> Modified: cocoon/trunk/core/cocoon-pipeline/cocoon-pipeline-components/src/main/java/org/apache/cocoon/transformation/TraxTransformer.java
> URL: http://svn.apache.org/viewvc/cocoon/trunk/core/cocoon-pipeline/cocoon-pipeline-components/src/main/java/org/apache/cocoon/transformation/TraxTransformer.java?rev=586238&r1=586237&r2=586238&view=diff
> ==============================================================================
> --- cocoon/trunk/core/cocoon-pipeline/cocoon-pipeline-components/src/main/java/org/apache/cocoon/transformation/TraxTransformer.java
(original)
> +++ cocoon/trunk/core/cocoon-pipeline/cocoon-pipeline-components/src/main/java/org/apache/cocoon/transformation/TraxTransformer.java
Thu Oct 18 19:45:18 2007
> @@ -24,9 +24,8 @@
>  import java.util.HashMap;
>  import java.util.Iterator;
>  import java.util.Map;
> -import java.util.Set;
>  import java.util.Map.Entry;
> -
> +import java.util.Set;
>  import javax.servlet.http.Cookie;
>  import javax.servlet.http.HttpSession;
>  import javax.xml.transform.sax.SAXResult;
> @@ -41,6 +40,14 @@
>  import org.apache.avalon.framework.service.ServiceException;
>  import org.apache.avalon.framework.service.ServiceManager;
>  import org.apache.avalon.framework.service.Serviceable;
> +import org.apache.commons.lang.BooleanUtils;
> +import org.apache.commons.lang.exception.NestableRuntimeException;
> +import org.apache.excalibur.source.Source;
> +import org.apache.excalibur.source.SourceException;
> +import org.apache.excalibur.source.SourceValidity;
> +import org.apache.excalibur.xml.xslt.XSLTProcessor;
> +import org.apache.excalibur.xml.xslt.XSLTProcessorException;
> +
>  import org.apache.cocoon.ProcessingException;
>  import org.apache.cocoon.caching.CacheableProcessingComponent;
>  import org.apache.cocoon.components.source.util.SourceUtil;
> @@ -48,14 +55,9 @@
>  import org.apache.cocoon.environment.ObjectModelHelper;
>  import org.apache.cocoon.environment.Request;
>  import org.apache.cocoon.environment.SourceResolver;
> +import org.apache.cocoon.util.avalon.CLLoggerWrapper;
>  import org.apache.cocoon.xml.XMLConsumer;
> -import org.apache.commons.lang.BooleanUtils;
> -import org.apache.commons.lang.exception.NestableRuntimeException;
> -import org.apache.excalibur.source.Source;
> -import org.apache.excalibur.source.SourceException;
> -import org.apache.excalibur.source.SourceValidity;
> -import org.apache.excalibur.xml.xslt.XSLTProcessor;
> -import org.apache.excalibur.xml.xslt.XSLTProcessorException;
> +
>  import org.xml.sax.SAXException;
>  
>  /**
> @@ -382,9 +384,9 @@
>  
>          super.setContentHandler(this.transformerHandler);
>          super.setLexicalHandler(this.transformerHandler);
> -
> +        // Is there even single implementation of LogEnabled TransformerHandler?
>          if (this.transformerHandler instanceof LogEnabled) {
> -        	((LogEnabled)this.transformerHandler).enableLogging(getLogger());
> +        	((LogEnabled) this.transformerHandler).enableLogging(new CLLoggerWrapper(getLogger()));
>          }

This commit is causing compilation errors on my side. Nothing surprising here, there is no
valid
constructor in CLLoggerWrapper suitable for this change.

Vadim, I'm going to revert this change. Could you take a look at it afterwards?

-- 
Grzegorz Kossakowski
Committer and PMC Member of Apache Cocoon
http://reflectingonthevicissitudes.wordpress.com/

Mime
View raw message