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

update GroovyXmlTransform to understand new first-level XML elements #74

Merged
merged 1 commit into from
Feb 22, 2016

Conversation

Ladicek
Copy link
Contributor

@Ladicek Ladicek commented Feb 22, 2016

WildFly added new elements to the set of possible first-level elements in the XML configuration files. This updates the Subtree class to understand them (which requires making a difference between host.xml and domain.xml, as host.xml is suddenly very similar to standalone.xml).

Also, to maintain correct order of the XML elements, it is no longer sufficient to maintain a single list which blends all the possibilities, so the list was split into three separate lists -- for the domain root element, for host and for server.

@Ladicek Ladicek self-assigned this Feb 22, 2016
@Ladicek Ladicek added this to the 0.9.6 milestone Feb 22, 2016
@Ladicek Ladicek changed the title fix ordering of first-level XML elements update GroovyXmlTransform to understand new first-level XML elements Feb 22, 2016
WildFly added new elements to the set of possible first-level elements
in the XML configuration files. This commit updates the Subtree class
to understand them (which requires making a difference between host.xml
and domain.xml, as host.xml is suddenly very similar to standalone.xml).

Also, to maintain correct order of the XML elements, it is no longer
sufficient to maintain a single list which blends all the possibilities,
so the list was split into three separate lists -- for the "domain" root
element, for "host" and for "server".
@Ladicek Ladicek merged commit fd9d09b into wildfly-extras:master Feb 22, 2016
@Ladicek Ladicek deleted the fix-xml-ordering branch February 22, 2016 14:49
olukas pushed a commit to olukas/creaper that referenced this pull request Mar 20, 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
Development

Successfully merging this pull request may close these issues.

1 participant