Fix: remove outdated admin logentry handler (#8580)

diff --git a/src/documents/apps.py b/src/documents/apps.py
index c00b23ff2477799a5c4fb5aeb3a76c436eec96b8..ac1bb21ebb59c123290a05758eb5750b999a8de3 100644 (file)
--- a/src/documents/apps.py
+++ b/src/documents/apps.py
@@ -16,7 +16,6 @@ class DocumentsConfig(AppConfig):
         from documents.signals.handlers import run_workflows_updated
         from documents.signals.handlers import set_correspondent
         from documents.signals.handlers import set_document_type
-        from documents.signals.handlers import set_log_entry
         from documents.signals.handlers import set_storage_path
         from documents.signals.handlers import set_tags
 
@@ -25,7 +24,6 @@ class DocumentsConfig(AppConfig):
         document_consumption_finished.connect(set_document_type)
         document_consumption_finished.connect(set_tags)
         document_consumption_finished.connect(set_storage_path)
-        document_consumption_finished.connect(set_log_entry)
         document_consumption_finished.connect(add_to_index)
         document_consumption_finished.connect(run_workflows_added)
         document_updated.connect(run_workflows_updated)

diff --git a/src/documents/signals/handlers.py b/src/documents/signals/handlers.py
index d669e3a0f3df2a403c266970149244472f11fee4..d3fa11b3b1d67ae6285ffe8451bca4023c9c47fa 100644 (file)
--- a/src/documents/signals/handlers.py
+++ b/src/documents/signals/handlers.py
@@ -10,10 +10,6 @@ from celery.signals import task_failure
 from celery.signals import task_postrun
 from celery.signals import task_prerun
 from django.conf import settings
-from django.contrib.admin.models import ADDITION
-from django.contrib.admin.models import LogEntry
-from django.contrib.auth.models import User
-from django.contrib.contenttypes.models import ContentType
 from django.core.mail import EmailMessage
 from django.db import DatabaseError
 from django.db import close_old_connections
@@ -537,20 +533,6 @@ def check_paths_and_prune_custom_fields(sender, instance: CustomField, **kwargs)
             update_filename_and_move_files(sender, cf_instance)
 
 
-def set_log_entry(sender, document: Document, logging_group=None, **kwargs):
-    ct = ContentType.objects.get(model="document")
-    user = User.objects.get(username="consumer")
-
-    LogEntry.objects.create(
-        action_flag=ADDITION,
-        action_time=timezone.now(),
-        content_type=ct,
-        object_id=document.pk,
-        user=user,
-        object_repr=document.__str__(),
-    )
-
-
 def add_to_index(sender, document, **kwargs):
     from documents import index
 

diff --git a/src/documents/tests/test_matchables.py b/src/documents/tests/test_matchables.py
index 4626e2c080a56e960bf030191ac6ea3562a564d2..9ca23e53d28864664f8869e58b22e8a0afa17d87 100644 (file)
--- a/src/documents/tests/test_matchables.py
+++ b/src/documents/tests/test_matchables.py
@@ -4,7 +4,6 @@ from collections.abc import Iterable
 from pathlib import Path
 from random import randint
 
-from django.contrib.admin.models import LogEntry
 from django.contrib.auth.models import User
 from django.test import TestCase
 from django.test import override_settings
@@ -451,11 +450,3 @@ class TestDocumentConsumptionFinishedSignal(TestCase):
             document=self.doc_contains,
         )
         self.assertEqual(self.doc_contains.correspondent, None)
-
-    def test_logentry_created(self):
-        document_consumption_finished.send(
-            sender=self.__class__,
-            document=self.doc_contains,
-        )
-
-        self.assertEqual(LogEntry.objects.count(), 1)