[ 
https://issues.apache.org/jira/browse/CONFIGURATION-315?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12572833#action_12572833
 ] 

Andy Yeung commented on CONFIGURATION-315:
------------------------------------------

I also found that the missing checking of isBeforeUpdate in 
CombinedConfiguration.configurationChanged sometimes mess up the detailEvents 
in XMLConfiguration contained in CombinedConfiguration. This would prevent 
!isBeforeUpdate event to sent out to CombinedConfiguration listener, when 
XMLConfiguration receive the event from its delegate in some cases. (this cases 
not always occur but happen occasionally). The detailEvents is -1 in that cases.


> CombinedConfiguration fire EVENT_COMBINED_INVALIDATE even before the 
> contained configuration is updated
> -------------------------------------------------------------------------------------------------------
>
>                 Key: CONFIGURATION-315
>                 URL: https://issues.apache.org/jira/browse/CONFIGURATION-315
>             Project: Commons Configuration
>          Issue Type: Bug
>          Components: Events & Notifications
>    Affects Versions: 1.5
>         Environment: Any
>            Reporter: Andy Yeung
>
> CombinedConfiguration.configurationChanged doesn't check the 
> !event.isBeforeUpdate() and trigger the invalidate call. The invalidate call 
> fire the EVENT_COMBINED_INVALIDATE. so now when a contained config is 
> changed, EVENT_COMBINED_INVALIDATE is fired twice by the 
> CombinedConfiguration. However it should only fire it after the contained 
> FileConfiguration is updated.

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

Reply via email to