Carsten Ziegeler wrote:





-----Original Message-----
From: Vadim Gritsenko [mailto:[EMAIL PROTECTED] Sent: Friday, June 18, 2004 2:56 PM
To: [EMAIL PROTECTED]
Subject: Re: [BUG] XMLUtils method has chaneged


Carsten Ziegeler wrote:



-----Original Message-----
From: Klaus Bertram [mailto:[EMAIL PROTECTED]
Sent: Thursday, June 17, 2004 12:35 PM
To: cocoon-dev
Subject: [BUG] XMLUtils method has chaneged

Hi all,

the samples binding.js with XML call the "old" function at LN 96
org.apache.cocoon.xml.XMLUtils.serializeNodeToXML(node)
but this not at the class

I think it must be:
Packages.org.apache.cocoon.xml.XMLUtils.serializeNode(node,
Packages.org.apache.cocoon.xml.XMLUtils.createPropertiesForX


ML(true));


but not sure ;)




Hmm, not sure, but I think the argument should be "false".

I don't have CVS access right now, can someone please correct this?




Ok, corrected.

But I see at least one issue with XMLUtils usage like:

   Properties props = XMLUtils.createPropertiesForXML(false);
   props.put(OutputKeys.ENCODING, "ISO-8859-1");
   final String content = XMLUtils.serializeNode(frag, props);


At least Xalan's TransformerIdentityImpl does not uses OutputKeys.ENCODING, as far as I can see. Also, why ISO-8859-1 is hardcoded. May be this encoding stuff should be removed from properties at all?




Where do you see this? We have removed the encoding stuff from XMLUtils
some time ago, so it shouldn't be there :)



Several places: SourceUtil 541, 582 DASLTransformer, 188 SessionPostTransformer, 347, 375

Vadim



Reply via email to