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

XomPayloadMethodProcessor is not added methodReturnValueHandlers in DefaultMethodEndpointAdapter

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Complete
    • Affects Version/s: 2.0 GA
    • Fix Version/s: 2.0.1
    • Component/s: Core, XML
    • Labels:
      None
    • Environment:
      all

      Description

            private void initMethodReturnValueHandlers() {
                  ...
                  if (isPresent(XOM_CLASS_NAME)) {
                      methodArgumentResolvers.add(new XomPayloadMethodProcessor());
                  }
                  ...
            }

      as you can see the XomPayloadMethodProcessor is added to the methodArgumentResolvers and not to the methodReturnValueHandlers.

      To fix the problem just replace the methodArgumentResolvers.add(...) with methodReturnValueHandlers.add(...)

        Activity

        sannies Sebastian Annies 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.1 [ 11810 ]
        arjen.poutsma Arjen Poutsma made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        arjen.poutsma Arjen Poutsma logged work - 28/Mar/11 2:31 AM
        • Time Spent:
          0.1h
           
          <No comment>
        arjen.poutsma Arjen Poutsma made changes -
        Remaining Estimate 0d [ 0 ]
        Time Spent 0.1h [ 360 ]
        arjen.poutsma Arjen Poutsma logged work - 28/Mar/11 2:31 AM
        • Time Spent:
          0.1h
           
          <No comment>
        arjen.poutsma Arjen Poutsma made changes -
        Time Spent 0.1h [ 360 ] 0.2h [ 720 ]
        arjen.poutsma Arjen Poutsma made changes -
        Resolution Complete [ 8 ]
        Status In Progress [ 3 ] Resolved [ 5 ]
        Hide
        arjen.poutsma Arjen Poutsma added a comment -

        Fixed. Thanks for spotting this!

        Show
        arjen.poutsma Arjen Poutsma added a comment - Fixed. Thanks for spotting this!
        Hide
        arjen.poutsma Arjen Poutsma added a comment -

        Closing old issues

        Show
        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
        3d 20h 46m 1 Arjen Poutsma 28/Mar/11 2:24 AM
        In Progress In Progress Resolved Resolved
        6m 59s 1 Arjen Poutsma 28/Mar/11 2:31 AM
        Resolved Resolved Closed Closed
        403d 4h 31m 1 Arjen Poutsma 04/May/12 7:03 AM

          People

          • Assignee:
            arjen.poutsma Arjen Poutsma
            Reporter:
            sannies Sebastian Annies
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - Not Specified
              Not Specified
              Remaining:
              Remaining Estimate - 0d
              0d
              Logged:
              Time Spent - 0.2h
              0.2h