Re: svn commit: r161263 - in cocoon/trunk/src/java/org/apache/cocoon: generation/JXTemplateGenerator.java xml/NamespaceSupport.java xml/RedundantNamespacesFilter.java

[Leszek Gawron]

Daniel Fagerstrom wrote:
[EMAIL PROTECTED] wrote:
Author: sylvain
Date: Thu Apr 14 05:26:52 2005
New Revision: 161263
URL: http://svn.apache.org/viewcvs?view=rev&rev=161263
Log:
Fix JXTG namespace issue
Added:
   cocoon/trunk/src/java/org/apache/cocoon/xml/NamespaceSupport.java   
(with props)
   
cocoon/trunk/src/java/org/apache/cocoon/xml/RedundantNamespacesFilter.java   
(with props)
Modified:
   
cocoon/trunk/src/java/org/apache/cocoon/generation/JXTemplateGenerator.java 

Modified: 
cocoon/trunk/src/java/org/apache/cocoon/generation/JXTemplateGenerator.java 

URL: 
http://svn.apache.org/viewcvs/cocoon/trunk/src/java/org/apache/cocoon/generation/JXTemplateGenerator.java?view=diff&r1=161262&r2=161263 

============================================================================== 

--- 
cocoon/trunk/src/java/org/apache/cocoon/generation/JXTemplateGenerator.java 
(original)
+++ 
cocoon/trunk/src/java/org/apache/cocoon/generation/JXTemplateGenerator.java 
Thu Apr 14 05:26:52 2005
@@ -55,6 +55,7 @@
import org.apache.cocoon.environment.SourceResolver;
import org.apache.cocoon.transformation.ServiceableTransformer;
import org.apache.cocoon.xml.IncludeXMLConsumer;
+import org.apache.cocoon.xml.RedundantNamespacesFilter;
import org.apache.cocoon.xml.XMLConsumer;
import org.apache.cocoon.xml.XMLUtils;
import org.apache.cocoon.xml.dom.DOMBuilder;
@@ -2482,7 +2483,7 @@
    private void performGeneration(final XMLConsumer consumer, 
MyJexlContext jexlContext, JXPathContext jxpathContext,
                StartElement macroCall, Event startEvent, Event 
endEvent) throws SAXException {
        cocoon.put("consumer", consumer);
shouldn't this consumer also be wrapped? Apart from the fact that
final XMLConsumer consumer is totally useless here as we can access it 
via this.xmlConsumer. Fixed in refactored JXTG.


-        execute(this.xmlConsumer, globalJexlContext, jxpathContext, 
null,
+        execute(new RedundantNamespacesFilter(this.xmlConsumer), 
globalJexlContext, jxpathContext, null,
                startEvent, null);
    }
 

Care to keep the refactored JXTG in synch ;)
http://svn.apache.org/viewcvs.cgi/cocoon/blocks/unsupported/template/trunk/java/org/apache/cocoon/template/jxtg/JXTemplateGenerator.java?view=markup 

/Daniel

--
Leszek Gawron                                      [EMAIL PROTECTED]
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