From a94e5d2e4706b7c6b107a0f6d5d57c26a4f76ca2 Mon Sep 17 00:00:00 2001 From: Trenton H <797416+stumpylog@users.noreply.github.com> Date: Fri, 17 Nov 2023 12:24:49 -0800 Subject: [PATCH] Only delete the ContentTypes and Permissions once, not the loop (#4623) --- src/documents/management/commands/document_importer.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/documents/management/commands/document_importer.py b/src/documents/management/commands/document_importer.py index 7a3e40f28..9895104c3 100644 --- a/src/documents/management/commands/document_importer.py +++ b/src/documents/management/commands/document_importer.py @@ -118,10 +118,10 @@ class Command(BaseCommand): # Fill up the database with whatever is in the manifest try: with transaction.atomic(): + # delete these since pk can change, re-created from import + ContentType.objects.all().delete() + Permission.objects.all().delete() for manifest_path in manifest_paths: - # delete these since pk can change, re-created from import - ContentType.objects.all().delete() - Permission.objects.all().delete() call_command("loaddata", manifest_path) except (FieldDoesNotExist, DeserializationError, IntegrityError) as e: self.stdout.write(self.style.ERROR("Database import failed"))