DO NOT REPLY [Bug 24849] New: - ExtendedProperties combine and pre unescaped strings-values

[bu...@apache.org]

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=24849>.
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=24849

ExtendedProperties combine and pre unescaped strings-values

           Summary: ExtendedProperties combine and pre unescaped strings-
                    values
           Product: Commons
           Version: 2.1 Final
          Platform: All
        OS/Version: Other
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: Collections
        AssignedTo: commons-dev@jakarta.apache.org
        ReportedBy: dean.arnold@digitalunion.com


If you have 2 extended props (prop1 & prop2). prop1 contains some or none
key/value pairs and prop2 contains at least one key with string value with
commas that doesn't already exist in prop1. N.B. This assumes both props were
loaded from property files and the string values in prop2 were escaped in the
file-system properties.

If you call prop1.combine(prop2), the string-value from prop2 that contained
commas is treated as a non-escaped string and thus broken down into a Vector in
prop1. It should put the string-value into prop1 without procing the commas as
there were already processed by prop2 when it was loaded.

I believe that the combine method should merge the existing values from both
props and the putAll method should be overidden to overite existing keys in
prop1 from prop2 while maintaining the keysAsListed List.

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org