From 118ecf950a1dd1186b77214918a619077a45beeb Mon Sep 17 00:00:00 2001 From: shamoon <4887959+shamoon@users.noreply.github.com> Date: Tue, 3 Mar 2026 11:28:14 -0800 Subject: [PATCH] This is all unused now --- src/documents/bulk_edit.py | 31 --------------------------- src/documents/tests/test_bulk_edit.py | 25 --------------------- 2 files changed, 56 deletions(-) diff --git a/src/documents/bulk_edit.py b/src/documents/bulk_edit.py index 5cf150ad3..a9d4894d6 100644 --- a/src/documents/bulk_edit.py +++ b/src/documents/bulk_edit.py @@ -81,37 +81,6 @@ def restore_archive_serial_numbers(backup: dict[int, int | None]) -> None: logger.info(f"Restored archive serial numbers for documents {list(backup.keys())}") -def _get_root_ids_by_doc_id(doc_ids: list[int]) -> dict[int, int]: - """ - Resolve each provided document id to its root document id. - - - If the id is already a root document: root id is itself. - - If the id is a version document: root id is its `root_document_id`. - """ - qs = Document.objects.filter(id__in=doc_ids).only("id", "root_document_id") - return {doc.id: doc.root_document_id or doc.id for doc in qs} - - -def _get_root_and_current_docs_by_root_id( - root_ids: set[int], -) -> tuple[dict[int, Document], dict[int, Document]]: - """ - Returns: - - root_docs: root_id -> root Document - - current_docs: root_id -> newest version Document (or root if none) - """ - root_docs = { - doc.id: doc - for doc in Document.objects.filter(id__in=root_ids).select_related( - "owner", - ) - } - current_docs: dict[int, Document] = {} - for root_id, root_doc in root_docs.items(): - current_docs[root_id] = get_latest_version_for_root(root_doc) - return root_docs, current_docs - - def _resolve_root_and_source_doc( doc: Document, *, diff --git a/src/documents/tests/test_bulk_edit.py b/src/documents/tests/test_bulk_edit.py index fa4fa3449..4676ff44b 100644 --- a/src/documents/tests/test_bulk_edit.py +++ b/src/documents/tests/test_bulk_edit.py @@ -405,31 +405,6 @@ class TestBulkEdit(DirectoriesMixin, TestCase): self.assertTrue(Document.objects.filter(id=self.doc1.id).exists()) self.assertFalse(Document.objects.filter(id=version.id).exists()) - def test_get_root_and_current_doc_mapping(self) -> None: - version1 = Document.objects.create( - checksum="B-v1", - title="B version 1", - root_document=self.doc2, - ) - version2 = Document.objects.create( - checksum="B-v2", - title="B version 2", - root_document=self.doc2, - ) - - root_ids_by_doc_id = bulk_edit._get_root_ids_by_doc_id( - [self.doc2.id, version1.id, version2.id], - ) - self.assertEqual(root_ids_by_doc_id[self.doc2.id], self.doc2.id) - self.assertEqual(root_ids_by_doc_id[version1.id], self.doc2.id) - self.assertEqual(root_ids_by_doc_id[version2.id], self.doc2.id) - - root_docs, current_docs = bulk_edit._get_root_and_current_docs_by_root_id( - {self.doc2.id}, - ) - self.assertEqual(root_docs[self.doc2.id].id, self.doc2.id) - self.assertEqual(current_docs[self.doc2.id].id, version2.id) - def test_resolve_root_and_source_doc_latest_version_prefers_newest_version( self, ) -> None: