Paul Brown wrote: > > > I'm absolutely fine with only supporting DOM Level 2, but > > one thing is missing in the DOMBuilder: the support for > > namespaces. > > Looks good to me other than the trailing slash on the xmlns > namespace (might > trip up people who don't compare URIs properly). > The problem is that the parser is not forced to append the xmlns attributes to an element when sending SAX events. If the parser does not send them, the resulting DOM will not have the namespace attributes either! When you then for example try to serialize the DOM, you get a document without the namespace declarations! The patch fixes this as it also records the namespaces during the startPrefixMapping method and adds these information as attributes to the DOM. We had this problem once in Cocoon as the default configuration for the parser is not to append the xmlns attributes to the sax events. So believe me, this is a problem.
Regards, Carsten > -- Paul >
