Details
-
Bug
-
Status: Closed
-
Normal
-
Resolution: Fixed
-
2.24.09.1
-
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.