DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://nagoya.apache.org/bugzilla/show_bug.cgi?id=16502>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE.
http://nagoya.apache.org/bugzilla/show_bug.cgi?id=16502 [configuration][patch] Enhancement: multiple interpolation Summary: [configuration][patch] Enhancement: multiple interpolation Product: Commons Version: unspecified Platform: All OS/Version: All Status: NEW Severity: Enhancement Priority: Other Component: Sandbox AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] As posted by Jeff Barrett [[EMAIL PROTECTED]] Here's my original message w/ a more informative subject line and patches split up into different files, zipped up. ======================================================= I don't have much experience sending patches, so please let me know if I could have sent this in a better way. This patch does two things: 1) fixes configuration so it builds w/ maven 2) adds support for multiple levels of interpolation 1)a) maven.xml had a bad variable reference that caused test.properties to be copied to the wrong place which caused TestClassPropertiesConfiguration to fail 1)b) project.xml had a bad unit test excludes that didn't exclude an abstract TestCase class 2)a) BaseConfiguration.java now supports multiple levels of interpolation. for example: base.prop = /base first.prop = ${base.prop}/first second.prop = ${first.prop}/second second.prop now interpolates "/base/first/second" whereas before it interpolated to "${base.prop}/first/second". Checks are also in place to catch the case of looping references, e.g. prop.a=${prop.b} prop.b=${prop.a} 2)b) TestBaseConfiguration.java has tests to check for proper behavior 2)c) BasePropertiesConfiguration.java now has class javadocs that note support for interpolation +jeff -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>