DO NOT REPLY [Bug 15505] New: - Impl and Intf namespace names are identical in the WSDL generated

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=15505>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=15505

Impl and Intf namespace names are identical in the WSDL generated

           Summary: Impl and Intf namespace names are identical in the WSDL
                    generated
           Product: Axis
           Version: 1.1beta
          Platform: Other
        OS/Version: Other
            Status: NEW
          Severity: Major
          Priority: Other
         Component: WSDL processing
        AssignedTo: axis-dev@xml.apache.org
        ReportedBy: anou_mana@hotmail.com


As per "http://www.w3.org/TR/REC-xml-names/#uniqAttrs"

5.3 Uniqueness of Attributes
In XML documents conforming to this specification, no tag may contain two 
attributes which: 

have identical names, or 
have qualified names with the same local part and with prefixes which have been 
bound to namespace names that are identical. 

For example, each of the bad start-tags is illegal in the following: 
<!-- http://www.w3.org is bound to n1 and n2 -->
<x xmlns:n1="http://www.w3.org" 
   xmlns:n2="http://www.w3.org" >
  <bad a="1"     a="2" />
  <bad n1:a="1"  n2:a="2" />
</x>

The WSDL generated has,

xmlns:impl="http://dnsrv:8080/web/services/StockQuotes" 
xmlns:intf="http://dnsrv:8080/web/services/StockQuotes" 

Looking at the code, it looks like it was made to be same by default. 
As per the spec, it is not a valid xml.

Hacked it to make it add -impl,
            // Default the implementation namespace to the interface
            // namespace if not split wsdl mode.
            if (implNS == null) {
                //if (mode == MODE_ALL) {
                //    implNS = intfNS;
                //} else {
                    implNS = intfNS + "-impl";
                //}
            }