yuuza/outline
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: Prevent duplicate documents in collection structure

Browse Source
This commit is contained in:
Tom Moor
2024-01-11 22:18:50 -05:00
parent 89931ca2f0
commit 2505fea103
2 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
server/models/Collection.test.ts
View File

@@ -111,6 +111,10 @@ describe("#addDocumentToStructure", () => {
await collection.addDocumentToStructure(newDocument); await collection.addDocumentToStructure(newDocument);
expect(collection.documentStructure!.length).toBe(1); expect(collection.documentStructure!.length).toBe(1);
expect(collection.documentStructure![0].id).toBe(id); expect(collection.documentStructure![0].id).toBe(id);
// should not append multiple times
await collection.addDocumentToStructure(newDocument);
expect(collection.documentStructure!.length).toBe(1);
}); });
test("should add with an index", async () => { test("should add with an index", async () => {

4
server/models/Collection.ts
View File

@@ -647,6 +647,10 @@ class Collection extends ParanoidModel {
this.documentStructure = []; this.documentStructure = [];
} }
if (this.getDocumentTree(document.id)) {
return this;
}
// If moving existing document with children, use existing structure // If moving existing document with children, use existing structure
const documentJson = { const documentJson = {
...(await document.toNavigationNode(options)), ...(await document.toNavigationNode(options)),