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

'saveRequired()' does not recognize if a SOAPMessage has been changed #69

Closed
Tomas-Kraus opened this issue Apr 11, 2012 · 6 comments
Closed

Comments

@Tomas-Kraus
Copy link
Member

If a SOAPMessage has been altered 'saveRequired()' does not recognize this.

Therefor the construct

if (soapMessage.saveRequired())

{ soapMessage.saveChanges(); }

does not work and also 'writeTo(Outputstream)' does not call 'saveChanges()' before writing out the message (see also 'http://java.net/jira/browse/SAAJ-68').

Output from running the test class attached

[java] final message :
[java] <env:Envelope xmlns:env='http://schemas.xmlsoap.org/soap/envelope/'>env:Header<abc:Extension1 xmlns:abc='http://example.org/2001/06/ext' env:actor='http://schemas.xmlsoap.org/soap/actor/next' env:mustUnderstand='1'/><def:Extension2 xmlns:def='http://example.com/stuff' env:mustUnderstand="1"/></env:Header>env:Body/</env:Envelope>
[java] final message ends here.

Environment

Metro 2.2, SAAJ 1.3.18, Oracle HotSpot VM 1.6.0_29

Affected Versions

[1.3.15]

@Tomas-Kraus
Copy link
Member Author

@glassfishrobot Commented
Reported by bthalmayr

@Tomas-Kraus
Copy link
Member Author

@glassfishrobot Commented
File: SOAPMessageSaveRequiredFailing.java
Attached By: bthalmayr

@Tomas-Kraus
Copy link
Member Author

@glassfishrobot Commented
gagordon said:
Same issue as #68 does not 'save' the message before writing it out to the stream").

@Tomas-Kraus
Copy link
Member Author

@glassfishrobot Commented
Marked as duplicate on Tuesday, November 27th 2012, 6:32:05 am

@Tomas-Kraus
Copy link
Member Author

@glassfishrobot Commented
This issue was imported from java.net JIRA SAAJ-69

@Tomas-Kraus
Copy link
Member Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant