Only delete the ContentTypes and Permissions once, not the loop (#4623)

paperless-ngx · Nov 26, 2023 · 97f69ee · 97f69ee
1 parent 2959ed9
commit 97f69ee
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/documents/management/commands/document_importer.py b/src/documents/management/commands/document_importer.py
@@ -118,10 +118,10 @@ def handle(self, *args, **options):
             # 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"))