[ http://issues.apache.org/jira/browse/CONFIGURATION-223?page=all ]
Oliver Heger resolved CONFIGURATION-223.
----------------------------------------
Fix Version/s: Nightly Builds
Resolution: Fixed
The fix was committed. Everything seems to work now. Please double-check.
Thanks.
> PropertiesConfigurationLayout broke the autoSave behaviour of
> PropertiesConfiguration
> -------------------------------------------------------------------------------------
>
> Key: CONFIGURATION-223
> URL: http://issues.apache.org/jira/browse/CONFIGURATION-223
> Project: Commons Configuration
> Issue Type: Bug
> Environment: 1.3RC1
> Reporter: Gabriele Garuglieri
> Assigned To: Oliver Heger
> Priority: Critical
> Fix For: Nightly Builds
>
> Attachments: 223-test.patch
>
>
> When using PropertiesConfiguration with autoSave "true" the
> PropertiesConfigurationLayout causes invalid properties files to be saved.
> When using addProperty("key", "value"); where key does not exist yet in the
> configuration the EVENT_ADD_PROPERTY event that updates the
> PropertiesConfigurationLayout with the new "key" is issued only after the
> possiblySave() method is invoked so the new property will not be saved.
> When using setProperty("key", "value"); where key does not exist yet in the
> configuration the EVENT_SET_PROPERTY event that updates the
> PropertiesConfigurationLayout with the new "key" is issued only after the
> possiblySave() method is invoked so the new property will not be saved.
> The way it is implemented now makes content of the saved file depending on
> PropertiesConfigurationLayout content that depends on an event that will be
> issued only after the file was properly updated. This creates a circular
> dependency that cannot work.
--
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]
