mirrors/django
1
0
Fork 0
mirror of https://github.com/django/django.git synced 2025-10-24 06:06:09 +00:00
Code Issues 32 Releases Wiki Activity

Fixed #36217 -- Restored pre_save/post_save signal emission via LogEntry.save() for single-object deletion in the admin.

Browse Source 
Regression in 40b3975e7d.

Thanks smiling-watermelon for the report.

Co-authored-by: Sarah Boyce <42296566+sarahboyce@users.noreply.github.com>
This commit is contained in:
antoliny0919
2025-02-28 20:17:17 +09:00
committed by Sarah Boyce
parent 1759c1dbd1
commit c09bceef68
8 changed files with 41 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

32
tests/admin_utils/test_logentry.py
View File

@@ -5,6 +5,7 @@ from django.contrib.admin.models import ADDITION, CHANGE, DELETION, LogEntry
from django.contrib.admin.utils import quote
from django.contrib.auth.models import User
from django.contrib.contenttypes.models import ContentType
from django.db.models.signals import post_save, pre_save
from django.test import TestCase, override_settings
from django.urls import reverse
from django.utils import translation
@@ -41,11 +42,23 @@ class LogEntryTests(TestCase):
[cls.a1],
CHANGE,
change_message="Changed something",
single_object=True,
)
def setUp(self):
self.client.force_login(self.user)
self.signals = []
pre_save.connect(self.pre_save_listener, sender=LogEntry)
self.addCleanup(pre_save.disconnect, self.pre_save_listener, sender=LogEntry)
post_save.connect(self.post_save_listener, sender=LogEntry)
self.addCleanup(post_save.disconnect, self.post_save_listener, sender=LogEntry)
def pre_save_listener(self, instance, **kwargs):
self.signals.append(("pre_save", instance))
def post_save_listener(self, instance, created, **kwargs):
self.signals.append(("post_save", instance, created))
def test_logentry_save(self):
"""
@@ -271,6 +284,7 @@ class LogEntryTests(TestCase):
for obj in queryset
]
self.assertSequenceEqual(logs, expected_log_values)
self.assertEqual(self.signals, [])
def test_recentactions_without_content_type(self):
"""
@@ -314,6 +328,8 @@ class LogEntryTests(TestCase):
"created_1": "00:00",
}
changelist_url = reverse("admin:admin_utils_articleproxy_changelist")
expected_signals = []
self.assertEqual(self.signals, expected_signals)
# add
proxy_add_url = reverse("admin:admin_utils_articleproxy_add")
@@ -322,6 +338,10 @@ class LogEntryTests(TestCase):
proxy_addition_log = LogEntry.objects.latest("id")
self.assertEqual(proxy_addition_log.action_flag, ADDITION)
self.assertEqual(proxy_addition_log.content_type, proxy_content_type)
expected_signals.extend(
[("pre_save", proxy_addition_log), ("post_save", proxy_addition_log, True)]
)
self.assertEqual(self.signals, expected_signals)
# change
article_id = proxy_addition_log.object_id
@@ -334,6 +354,10 @@ class LogEntryTests(TestCase):
proxy_change_log = LogEntry.objects.latest("id")
self.assertEqual(proxy_change_log.action_flag, CHANGE)
self.assertEqual(proxy_change_log.content_type, proxy_content_type)
expected_signals.extend(
[("pre_save", proxy_change_log), ("post_save", proxy_change_log, True)]
)
self.assertEqual(self.signals, expected_signals)
# delete
proxy_delete_url = reverse(
@@ -344,6 +368,10 @@ class LogEntryTests(TestCase):
proxy_delete_log = LogEntry.objects.latest("id")
self.assertEqual(proxy_delete_log.action_flag, DELETION)
self.assertEqual(proxy_delete_log.content_type, proxy_content_type)
expected_signals.extend(
[("pre_save", proxy_delete_log), ("post_save", proxy_delete_log, True)]
)
self.assertEqual(self.signals, expected_signals)
def test_action_flag_choices(self):
tests = ((1, "Addition"), (2, "Change"), (3, "Deletion"))
@@ -358,7 +386,6 @@ class LogEntryTests(TestCase):
[self.a1],
CHANGE,
change_message="Article changed message",
single_object=True,
)
c1 = Car.objects.create()
LogEntry.objects.log_actions(
@@ -366,7 +393,6 @@ class LogEntryTests(TestCase):
[c1],
ADDITION,
change_message="Car created message",
single_object=True,
)
exp_str_article = escape(str(self.a1))
exp_str_car = escape(str(c1))