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

[2.7] Bug 543265 - WSDL namespace order is different on ITEMSERVICEPORT?WSDL - backport from master #1879

Merged

Conversation

rfelcman
Copy link
Contributor

Bug 543265 - WSDL namespace order is different on ITEMSERVICEPORT?WSDL

This is fix for this bug with unit test ClassArraySchemaGenTestCases.testSchemaGenFromClassArrayVerifyOrder()
There are two sort calls.
First sort in SchemaGenerator.java to sort input classes before schema name (like schema1.xsd, schema2.xsd....) is generated and assigned.
Second sort in Generator.java to ensure that schemas will be passed to the output in specified order (by namespace).
This fix leads into some changes in current unit tests in MOXy (schema generation), because some schemes are generated in different order. In some cases there are changes in order of complexType definition inside schemes.

Signed-off-by: Radek Felcman radek.felcman@oracle.com
(cherry picked from commit 68fd69e)

Verified

This commit was signed with the committer’s verified signature. The key has expired.
radovanradic Radovan Radic
Signed-off-by: Radek Felcman <radek.felcman@oracle.com>
@rfelcman rfelcman force-pushed the bug_moxy_543265_WSDLNamespaceOrder_2.7 branch from 4e88df2 to d9f601c Compare May 11, 2023 12:21
Copy link
Member

@lukasj lukasj left a comment

Choose a reason for hiding this comment

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

LGTM

@rfelcman rfelcman merged commit c5c8da6 into eclipse-ee4j:2.7 May 11, 2023
@rfelcman rfelcman deleted the bug_moxy_543265_WSDLNamespaceOrder_2.7 branch May 11, 2023 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants