jschaufuss/DiceCloud
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Duplicating properties now renews root sub-doc ids

Browse Source
This commit is contained in:
Thaum Rystra
2023-09-18 14:12:30 +02:00
parent 673f187373
commit c274153c79
2 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
app/imports/api/creature/creatureProperties/methods/duplicateProperty.js
View File

@@ -77,7 +77,8 @@ const duplicateProperty = new ValidatedMethod({
});
// Give the docs new IDs without breaking internal references
renewDocIds({ docArray: nodes });
const allNodes = [property, ...nodes];
renewDocIds({ docArray: allNodes });
// Order the root node
property.order += 0.5;
@@ -86,7 +87,7 @@ const duplicateProperty = new ValidatedMethod({
property.dirty = true;
// Insert the properties
CreatureProperties.batchInsert([property, ...nodes]);
CreatureProperties.batchInsert(allNodes);
// Tree structure changed by inserts, reorder the tree
reorderDocs({

5
app/imports/api/library/methods/duplicateLibraryNode.js
View File

@@ -67,12 +67,13 @@ const duplicateLibraryNode = new ValidatedMethod({
});
// Give the docs new IDs without breaking internal references
renewDocIds({ docArray: nodes });
const allNodes = [libraryNode, ...nodes];
renewDocIds({ docArray: allNodes });
// Order the root node
libraryNode.order += 0.5;
LibraryNodes.batchInsert([libraryNode, ...nodes]);
LibraryNodes.batchInsert(allNodes);
// Tree structure changed by inserts, reorder the tree
reorderDocs({