[ http://issues.apache.org/jira/browse/CONFIGURATION-223?page=comments#action_12431060 ] Oliver Heger commented on CONFIGURATION-223: --------------------------------------------
Will have a closer look at this issue. Perhaps the problem could be solved when AbstractFileConfiguration did not override addPropertyDirect(), but addProperty() and setProperty() (for calling the possiblySave() method). This would also have the advantage that adding of properties with multiple values would cause only one save() operation instead of one per property value. > 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 > > 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]
