Opened 8 months ago
Last modified 8 months ago
#36234 closed Bug
Restore single_object argument to LogEntry.objects.log_actions() — at Initial Version
| Reported by: | Adam Johnson | Owned by: | |
|---|---|---|---|
| Component: | contrib.admin | Version: | 5.1 |
| Severity: | Release blocker | Keywords: | |
| Cc: | Triage Stage: | Ready for checkin | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
#36217 dropped the single_argument argument in a bug fix release (5.1.7). Consequently, the method has two signatures in different Django 5.1 versions. I found real-world breakage from this in:
- a client project which had calls to
log_actions() - django-import-export - https://github.com/django-import-export/django-import-export/pull/2045
- django-stubs - https://github.com/typeddjango/django-stubs/blob/master/django-stubs/contrib/admin/models.pyi#L33 would need changing
I propose that we restore the argument in Django 5.1 and deprecate it from Django 5.2. It doesn’t make sense that we’d go through the deprecation pathway for the log_action() -> log_actions() migration, but drop an argument in a point release.
Note:
See TracTickets
for help on using tickets.