Skip to content

Support property references in XML files#22

Open
rougeSE wants to merge 1 commit intoNASAWorldWind:masterfrom
rougeSE:property-substitution
Open

Support property references in XML files#22
rougeSE wants to merge 1 commit intoNASAWorldWind:masterfrom
rougeSE:property-substitution

Conversation

@rougeSE
Copy link
Copy Markdown

@rougeSE rougeSE commented Oct 15, 2016

When deploying Worldwind within isolated environments with isloated
WMS servers, there is no way to centralize the definition of the server
name, instead, each of the layer XML files have to be updated.
This change allows any of the WW XML file to contain a property
reference, ${property.name}, and have these properties resolved when the
text is retrieved. Properties can be defined in three different ways;
1)worldwind.xml , 2) Java system property (-Dproperty.name),
or 3) using the legacy WW properties file.

Testing:
UnitTest for WWUtilTest was updated to Perform testing of property reference resolution from Configuration and system properties. Test included nested references and logger generation for missing properties.

When deploying Worldwind within an isolated environments with isloated
WMS servers, there is no way to centralize the definition of the server
name, instead, each of the layer XML files have to be updated.
This change allows any of the WW XML file to contain a property
reference, ${property.name}, and have these properties resolved when the
text is retrieved. Properties can be defined in three different ways;
1)worldwind.xml <Property>, 2) Java system property (-Dproperty.name),
or 3) using the legacy WW properties file.
@zglueck
Copy link
Copy Markdown
Contributor

zglueck commented Oct 17, 2016

@rougeSE thanks for the contribution. The team is heavily focused on some other tasks right now so it may take a bit of time to review your pull request. We do appreciate your willingness to share code!

@rougeSE rougeSE changed the title All for property references in XML files Support property references in XML files Oct 20, 2016
@pdavidc pdavidc removed the request label Jun 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants