Details
-
Type:
Bug
-
Status:
Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: None
-
Fix Version/s: None
-
Labels:None
-
Patch Submitted:Yes
-
Number of attachments :
Description
String ns1 = "urn://namespace1";
String ns2 = "urn://namespace2";
writer.writeStartDocument();
writer.writeStartElement(ns1, "rootElement");
writer.writeStartElement(ns2, "firstChild");
writer.writeEndElement();
writer.writeStartElement(ns2, "secondChild");
writer.writeEndElement();
writer.writeEndElement();
writer.writeEndDocument();
writer.close();
the code above is expected to give us
<?xml version='1.0' encoding='UTF-8'?>
<wstxns1:rootElement xmlns:wstxns1="urn://namespace1">
<wstxns2:firstChild xmlns:wstxns2="urn://namespace2" />
<wstxns3:secondChild xmlns:wstxns3="urn://namespace2" />
</wstxns1:rootElement>
or
<?xml version='1.0' encoding='UTF-8'?>
<wstxns1:rootElement xmlns:wstxns1="urn://namespace1" xmlns:wstxns2="urn://namespace2">
<wstxns2:firstChild />
<wstxns2:secondChild />
</wstxns1:rootElement>
But, wstx writer create a document such as
<?xml version='1.0' encoding='UTF-8'?>
<wstxns1:rootElement xmlns:wstxns1="urn://namespace1">
<wstxns2:firstChild xmlns:wstxns2="urn://namespace2" />
<wstxns2:secondChild/>
</wstxns1:rootElement>
which has missing ns prefix delaraction at the element named secondChild.
For respects of readability and document size, it would be better all namespace declaration are located at the root element.
But XMLStreamWriter doesn't know which xml namespace prefix delaration would be used and when it would be.
Anyway, I found a problem in the method, writeStartOrEmpty(String localName, String nsURI) in RepairingNsStreamWriter class.
RepairingNsStreamWriter writes StartElement in this order.
1) add a ns-prefix mapping to current element. (it would be parent element)
2) create child element <-- The subject of writeStartElement.
3) current element = child element
4) set ns-prefix of current element
I changed the orthe this way,
1) create child element with null prefix.
2) current element = child element
3) add a ns-prefix mapping to current element
4 set ns-prefix of current element
see a attached source file.( writeStartOrEmpty(String localName, String nsURI) )
Activity
Cool, sounds like an interesting and useful tool/library that you are working on.
Like I said, I should be able to fix the problem as per your suggestion pretty soon now (hopefully over the weekend). And since there have been a few other fixes submitted, should release 3.2.3 as well (something I hadn't planned to do, but bugs need to be squashed ASAP).
As to DOM writer: unfortunately time I can spend on woodstox is somewhat limited now. Also, finishing that part up was not very high on my list, so I wasn't expecting it to be done very soon.
But if you want it to get done, there are 2 things that would help:
(a) If you can file another issue (new feature or improvement) in Jira for implementing namespace-repairing mode for DOMWrappingWriter, that would help (and of course you can also vote for it). That way it's easier for me to track what kinds of things users want. I will try to prioritize things so that it's partially based on wishes of users, not just my own preferences. 
(b) Alternatively, if you (or any other woodstox user) could contribute an implementation, I would be happy to integrate it in. It should not be very hard to implement, although not trivial either (if it was trivial, I would have finished it right away).
One possibility is that if you do end up writing your own namespace context, perhaps that might help in someone implementing equivalent functionality within Woodstox?
At any rate, thank you for reporting the problems, as well as patience in getting them resolved. I really appreciate it, since DOM compatibility part was not heavily used or tested earlier.
This does indeed look like a bug (assuming output you added is what is produced).
One quick question: which version is this against?
Also, thank you for the patch! That is much appreciated.