Uploaded image for project: 'Hippo Repository'
  1. Hippo Repository
  2. REPO-1791

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

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Top
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 5.0.1
    • Labels:
      None
    • Similar issues:
    • Story Points:
      1
    • Epic Link:
    • Processed by team:
      Platform
    • Sprint:
      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

          Activity

            People

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

              Dates

              • Created:
                Updated:
                Resolved: