Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

XomPayloadMethodProcessor is not added methodReturnValueHandlers in DefaultMethodEndpointAdapter [SWS-699] #795

Closed
gregturn opened this issue Mar 24, 2011 · 2 comments
Assignees
Milestone

Comments

@gregturn
Copy link
Member

@gregturn gregturn commented Mar 24, 2011

Sebastian Annies opened SWS-699 and commented

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(...)


Affects: 2.0 GA

Referenced from: commits 7b1862a

@gregturn
Copy link
Member Author

@gregturn gregturn commented Mar 28, 2011

Arjen Poutsma commented

Fixed. Thanks for spotting this!

@gregturn
Copy link
Member Author

@gregturn gregturn commented May 4, 2012

Arjen Poutsma commented

Closing old issues

@gregturn gregturn closed this May 4, 2012
@gregturn gregturn added this to the 2.0.1 milestone Sep 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.