[
https://issues.apache.org/jira/browse/CONFIGURATION-272?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12499126
]
Emmanuel Bourg commented on CONFIGURATION-272:
----------------------------------------------
The append() function is also affected by this issue.
I have two solutions in mind :
- Implement the workaround mentioned above in the copy and append functions,
but this is not thread safe, if the target configuration is saved at this
moment the list delimiters are lost. It's certain to happen if the target
configuration is a file configuration with auto save set to true.
- for copy(), use clearProperty + addPropertyDirect. Unfortunately this will
trigger 2 events per key, and since fireEvent() is protected it's not possible
to use the setDetail() trick to fire the right event.
> ConfigurationUtils.copy results in escaped "," characters no longer being
> escaped
> ---------------------------------------------------------------------------------
>
> Key: CONFIGURATION-272
> URL: https://issues.apache.org/jira/browse/CONFIGURATION-272
> Project: Commons Configuration
> Issue Type: Bug
> Affects Versions: 1.4
> Environment: commons 1.4
> Reporter: John Meagher
> Assigned To: Emmanuel Bourg
> Fix For: 1.5
>
>
> When using ConfigurationUtils.copy to go between configuration formats any
> escaped out "," characters are put into the output configuration without the
> escape character. This results in a different value being returned from
> getString calls as it now things the property is a list. If 4 escape
> characters are used instead of the expected 1 then the copy will end up with
> the correct value, but then the original will end up with the escape
> character as part of the property value.
> Example:
> SomeLongProperty=This is a test\, it is only a test
> # Ends up right in the copy, but not right in the original
> SomeOtherProperty=This is also a test\\\\, it is also only a test
> After copy:
> # Missing "\" before ","
> SomeLongProperty=This is a test, it is only a test
> SomeOtherProperty=This is also a test\, it is also only a test
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
