Axiom
  1. Axiom
  2. AXIOM-180

SOAP11FaultCodeImpl contains SOAP 1.2 local name

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Environment:
      AXIOM 1.0, JDK 1.5

      Description

      The class org.apache.axiom.soap.impl.llom.soap11.SOAP11FaultCodeImpl does not override the localname of its parent org.apache.axiom.soap.impl.llom.SOAPFaultCodeImpl. Since this parent class contains the SOAP 1.2 specific SOAP12Constants.SOAP_FAULT_CODE_LOCAL_NAME as localname, the localname is of the SOAP 1.1 subclass is not set correctly.

      Oviously, this code should be in org.apache.axiom.soap.impl.llom.soap12.SOAP12FaultCodeImpl, and not in the non-version specific superclass. In general, there seems to be a lot of defaulting to SOAP 1.2-specific stuff in the org.apache.axiom.soap.impl.llom package. This should be moved to the version-specific subclasses.

      The errors become clear when obtaining an XMLStreamReader from the fault code, and reading from that. Serializing to an outputstream does work correctly, since the serialize methods are properly overridden.

      1. Tester.java
        1.0 kB
        Arjen Poutsma

        Issue Links

          Activity

          Andreas Veithen made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Andreas Veithen made changes -
          Project WS-Commons [ 12310250 ] Axiom [ 12311190 ]
          Key WSCOMMONS-38 AXIOM-180
          Component/s AXIOM [ 12310703 ]
          Davanum Srinivas made changes -
          Link This issue is blocked by WSCOMMONS-104 [ WSCOMMONS-104 ]
          Eran Chinthaka made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Won't Fix [ 2 ]
          Arjen Poutsma made changes -
          Attachment Tester.java [ 12334967 ]
          Davanum Srinivas made changes -
          Field Original Value New Value
          Assignee Eran Chinthaka [ eran chinthaka ]
          Arjen Poutsma created issue -

            People

            • Assignee:
              Eran Chinthaka
              Reporter:
              Arjen Poutsma
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development