[ http://issues.apache.org/jira/browse/CONFIGURATION-224?page=all ]

Oliver Heger resolved CONFIGURATION-224.
----------------------------------------

    Fix Version/s: Nightly Builds
       Resolution: Fixed

A comment describing the behavior of getProperty() was added to the 
Configuration interface. So I am closing this ticket now. The problems with 
ConfigurationDynaBean are now dealt with in ticket CONFIGURATION-227.

> BaseConfiguration.getProperty() returns reference to intenal store collection 
> object instead of a clone
> -------------------------------------------------------------------------------------------------------
>
>                 Key: CONFIGURATION-224
>                 URL: http://issues.apache.org/jira/browse/CONFIGURATION-224
>             Project: Commons Configuration
>          Issue Type: Bug
>         Environment: 1.3RC2
>            Reporter: Gabriele Garuglieri
>            Priority: Minor
>             Fix For: Nightly Builds
>
>         Attachments: list.patch
>
>
> When the property object is a List or an Array, 
> BaseConfiguration.getProperty() returns a reference to the object contained 
> in the store.
> This means that changing that object, changes the content of the properties 
> store without an explicit setProperty() or addProperty(). 
> Also any change to the object in the store will change the object that the 
> code has reference to, but the user is not aware that the property has 
> changed unless he has added a ConfigurationListener.
> Attached a patch to make the object returned by 
> BaseConfiguration.getProperty() a clone of store content, to avoid side 
> effects when changing Collection properties.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to