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

Adjusts to solve JAX-WS 2.2 issue. #2065

Closed

Conversation

h3nrique
Copy link

@h3nrique h3nrique commented Feb 9, 2017

Included method to solve WSFException. "Caused by: org.jboss.wsf.spi.WSFException: JBWS024104: Service class org.jboss.as.quickstarts.wsat.simple.jaxws.RestaurantServiceATService is missing required JAX-WS 2.2 additional constructors" and "Caused by: java.lang.NoSuchMethodException: org.jboss.as.quickstarts.wsat.simple.jaxws.RestaurantServiceATService.(java.net.URL, javax.xml.namespace.QName, [Ljavax.xml.ws.WebServiceFeature;)"

Included method to solve exception "Caused by: org.jboss.wsf.spi.WSFException: JBWS024104: Service class org.jboss.as.quickstarts.wsat.simple.jaxws.RestaurantServiceATService is missing required JAX-WS 2.2 additional constructors" and "Caused by: java.lang.NoSuchMethodException: org.jboss.as.quickstarts.wsat.simple.jaxws.RestaurantServiceATService.<init>(java.net.URL, javax.xml.namespace.QName, [Ljavax.xml.ws.WebServiceFeature;)"
@sgilda
Copy link
Contributor

sgilda commented Feb 9, 2017

@ctomc : Do you know who should review this change?

@paulrobinson : You were the original owner of this one. Would you like to review it?

@paulrobinson
Copy link
Contributor

This looks fine to me. I'm very rusty in this area now, but I don't see a reason to not add this additional constructor.

@sgilda
Copy link
Contributor

sgilda commented Feb 9, 2017

@paulrobinson : Thanks for the response. I will wait for @ctomc to respond before I merge.

@h3nrique : Thanks for submitting the fix!

Copy link
Contributor

@ctomc ctomc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks fine.
But please do not merge it here yet, I will cherry-pick it as part of other changes to my development branch.

@sgilda
Copy link
Contributor

sgilda commented Feb 9, 2017

@ctomc : Sure thing.

@h3nrique
Copy link
Author

h3nrique commented Feb 9, 2017

@sgilda Thanks!

@ctomc
Copy link
Contributor

ctomc commented Feb 10, 2017

cherry-picked this into my dev branch and pushed it to downstream repos.
closing this here.

@ctomc ctomc closed this Feb 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants