You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Henri Yandell (JIRA)" <ji...@apache.org> on 2008/11/03 23:42:44 UTC

[jira] Closed: (COLLECTIONS-271) org.apache.commons.collections.ExtendedProperties#combine don't import string properly

     [ https://issues.apache.org/jira/browse/COLLECTIONS-271?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Henri Yandell closed COLLECTIONS-271.
-------------------------------------

    Resolution: Fixed

svn ci -m "Applying Nathan Bubna's patch from COLLECTIONS-271 to fix the bug introduced in the last patch where getKeys() breaks after a combine() or subset() call. " src

Sending        src/java/org/apache/commons/collections/ExtendedProperties.java
Sending        src/test/org/apache/commons/collections/TestExtendedProperties.java
Transmitting file data ..
Committed revision 710200.

> org.apache.commons.collections.ExtendedProperties#combine don't import string properly
> --------------------------------------------------------------------------------------
>
>                 Key: COLLECTIONS-271
>                 URL: https://issues.apache.org/jira/browse/COLLECTIONS-271
>             Project: Commons Collections
>          Issue Type: Bug
>          Components: Collection
>    Affects Versions: 3.2
>            Reporter: Alexander Borovsky
>             Fix For: 3.3
>
>         Attachments: COLLECTIONS-271.patch, COLLECTIONS-271.patch, fix.patch
>
>
> When we set property with escaped characters, after combine propertySets we got them unescaped.
> Simple Example
> ExtendedProperties props = new ExtendedProperties();
> props.setProperty("test", "\\\\\\\\192.168.1.91\\\\test");
> props.getProperty("test"); // => \\192.168.1.91\test
> ExtendedProperties props2 = new ExtendedProperties();
> props2.combine(props);
> props.getProperty("test"); // => \192.168.1.91\test -- Wrong!

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.