Uploaded image for project: '[Read Only] - Hippo Repository'
  1. [Read Only] - Hippo Repository
  2. REPO-1791

EventJournalProcessor$Changes#addCurrentChanges(Changes) must also accomodate for overlapping paths

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Top
    • Resolution: Fixed
    • None
    • 5.0.1
    • None
    • None
    • 1
    • Platform164

    Description

      The EventJournalProcessor$Changes class and event recording has been overhauled:

      • it now encapsulates all the logic of event path processing internally.
      • it now only handles the paths from the event itself
      • merging of currentChanges onto pendingChanges now uses the same logic as for recording single events (which required to also add tracking of deleted config Paths internally)

      Postprocessing of the EventJournal based Changes (pendingChanges) now use/create separate paths (for added/deleted) as these might need to be adjusted to:

      • record added content paths within added config paths (AutoExportConfigExporter)
      • record deleted content paths within deleted config paths (AutoExportConfigExporter)
        note: this is a bugfix as we didn't detect this use-case before
      • add all child paths of all added content paths (#getAllAddedContentPaths(paths)

      Finally, the #injectResidualCategoryOverrides (REPO-1730) logic now has moved to AutoExportConfigExporter as an 'in-process' enhancement instead of a post-processing rewrite of the export diff.

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              adouma Ate Douma
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: