Uploaded image for project: 'Commons Configuration'
  1. Commons Configuration
  2. CONFIGURATION-315

CombinedConfiguration fire EVENT_COMBINED_INVALIDATE even before the contained configuration is updated

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.5
    • Fix Version/s: 1.6
    • Component/s: Events & Notifications
    • Labels:
      None
    • Environment:

      Any

      Description

      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.

        Attachments

          Activity

            People

            • Assignee:
              oheger Oliver Heger
              Reporter:
              andyyeung Andy Yeung
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: