While it looks like this change made it into the 5-24 build regarding bug
19001, it doesn't appear to have fixed my problem.  I suspect due to the
same behavior of WSDL2Java and MessageElement.toString() that the problem
may in fact be in the SerializationContextImpl.  I think actually the issue
may be in the startElement method where the call to qname2String is made
with the Boolean value for writing the namespace always set to true.  There
may be cases where you might want to convert unqualified namespaces to "",
but you should also have a way to disable this I believe.

The impact of the above is that in cases where you might call
SOAPEnvelope.getAsDocument, these empty namespace get added without the
ability to control whether they should in fact be added.

Is this a correct assumption?

Thx,

Mark

-----Original Message-----
From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]
Sent: Friday, May 23, 2003 12:44 AM
To: [EMAIL PROTECTED]
Subject: cvs commit: xml-axis/java/src/org/apache/axis/utils
DOM2Writer.java


gdaniels    2003/05/22 23:44:24

  Modified:    java/src/org/apache/axis/utils DOM2Writer.java
  Log:
  Fix bug 19001.  Don't allow empty prefixes.
  
  Thanks to Andrew Vardeman for the patch.
  
  Revision  Changes    Path
  1.15      +3 -1
xml-axis/java/src/org/apache/axis/utils/DOM2Writer.java
  
  Index: DOM2Writer.java
  ===================================================================
  RCS file:
/home/cvs/xml-axis/java/src/org/apache/axis/utils/DOM2Writer.java,v
  retrieving revision 1.14
  retrieving revision 1.15
  diff -u -r1.14 -r1.15
  --- DOM2Writer.java   22 Apr 2003 19:36:03 -0000      1.14
  +++ DOM2Writer.java   23 May 2003 06:44:24 -0000      1.15
  @@ -166,7 +166,9 @@
                   String elPrefix = node.getPrefix();
                   String elNamespaceURI = node.getNamespaceURI();
   
  -                if (elPrefix != null && elNamespaceURI != null)
  +                if (elPrefix != null &&
  +                        elNamespaceURI != null &&
  +                        elPrefix.length() > 0)
                   {
                       boolean prefixIsDeclared = false;
   
  
  
  

Reply via email to