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

Reloading initialize items should order the reloaded node at the position of the delta insert location

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Normal
    • Resolution: Fixed
    • 2.24.09.1
    • 2.24.13
    • None
    • None

    Description

      Steps to reproduce:
      1. Initialize the attached XML (my-documents-tree-loader.xml) at the content root: /hippo:configuration/hippo:frontend/cms/cms-pickers/documents
      2. In the console reorder the myDocumentsTreeLoader to the last position (see also order before the reload):
      3. Reload my-documents-tree-loader.xml

      Order before the reload:
      1. navigator
      2. breadcrumbs
      3. documentsTreeLoader
      4. imagesTreeLoader
      5. assetsTreeLoader
      6. myDocumentsTreeLoader

      Order after the reload:
      1. navigator
      2. breadcrumbs
      3. documentsTreeLoader
      4. imagesTreeLoader
      5. myDocumentsTreeLoader ????? at position 5 instead of 4
      6. assetsTreeLoader

      Expected order:
      1. navigator
      2. breadcrumbs
      3. documentsTreeLoader
      4. myDocumentsTreeLoader
      5. imagesTreeLoader
      6. assetsTreeLoader

      Cause:
      In the fix of REPO-636 the location index of the node is stored before calling initializeNodecontent and after initializeNodecontent the node is reorder using the stored index.
      With a delta merge insert the stored index cannot be guarantee to have the correct location after initializeNodecontent.

      Possible fix:
      If the reloaded node is a delta merge insert then don't call reorderNode method.

      Attachments

        Activity

          People

            jsheriff Junaidh Kadhar Sheriff
            rgerritsen Rutger Gerritsen (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: