Uploaded image for project: 'Spring Web Services'
  1. Spring Web Services
  2. SWS-672

Allow MarshallingPayloadMethodProcessor to have a null marshaller or unmarshaller


    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.0 RC1
    • Fix Version/s: 2.0 RC2
    • Component/s: Core
    • Labels:
    • Environment:
      Spring-ws 2.0.0-RC1, Java 5.0


      With Spring 2.0 I like how it allows you to configure different binding mechanisms and marshallers for the request and response to a single operation. In my case, I am using the @XPathParam configuration to parse the request to my operation and using a custom Stax marshaller for the response. Below is my custom marshaller configuration in my spring context file. As you can see below, I have no Unmarshaller configured on MarshallingPayloadMethodProcessor because I am not doing any unmarshalling in my service using the Unmarshaller interface. The issue is that MarshallingPayloadMethodProcessor requires both a marshaller and unmarshaller be set and will throw an exception if they are not both set.

      Would it possible to change the MarshallingPayloadMethodProcessor's validation logic so it only checks that at least a marshaller or an unmarshaller is set instead of always requiring both?

      <bean class="org.springframework.ws.server.endpoint.adapter.DefaultMethodEndpointAdapter">
      <property name="methodReturnValueHandlers">
      <bean class="org.springframework.ws.server.endpoint.adapter.method.MarshallingPayloadMethodProcessor">
      <property name="marshaller" ref="pingMarshaller" />

      <bean id="pingMarshaller" class="...


        swanjr Josh Swan created issue -
        arjen.poutsma Arjen Poutsma made changes -
        Field Original Value New Value
        Assignee Arjen Poutsma [ arjen.poutsma ]
        arjen.poutsma Arjen Poutsma made changes -
        Fix Version/s 2.0 RC2 [ 11747 ]
        arjen.poutsma Arjen Poutsma logged work - 20/Dec/10 12:43 AM
        • Time Spent:
          <No comment>
        arjen.poutsma Arjen Poutsma made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        arjen.poutsma Arjen Poutsma made changes -
        Time Spent 17m [ 1020 ]
        Remaining Estimate 0d [ 0 ]
        arjen.poutsma Arjen Poutsma made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        arjen.poutsma Arjen Poutsma added a comment -

        Closing old issues

        arjen.poutsma Arjen Poutsma added a comment - Closing old issues
        arjen.poutsma Arjen Poutsma made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        2d 16h 19m 1 Arjen Poutsma 20/Dec/10 1:08 AM
        In Progress In Progress Resolved Resolved
        17m 32s 1 Arjen Poutsma 20/Dec/10 1:25 AM
        Resolved Resolved Closed Closed
        501d 5h 38m 1 Arjen Poutsma 04/May/12 7:03 AM


          • Assignee:
            arjen.poutsma Arjen Poutsma
            swanjr Josh Swan
          • Votes:
            0 Vote for this issue
            0 Start watching this issue


            • Created:

              Time Tracking

              Original Estimate - Not Specified
              Not Specified
              Remaining Estimate - 0d
              Time Spent - 17m