Changes between Initial Version and Version 1 of Ticket #34802
- Timestamp:
- Aug 29, 2023, 7:24:54 AM (15 months ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #34802 – Description
initial v1 1 1 == Status quo 2 2 Currently, the `django.contrib.admin.actions.delete_selected()` Admin action [https://github.com/django/django/blob/4.2.4/django/contrib/admin/actions.py#L52-L57 will display a feedback message] via the Messages framework reporting on how many objects have been deleted. But in reality the number displayed is an estimate based on an earlier query (for the confirmation dialog) rather than the actual number of objects eventually deleted. 3 Usually this is not a problem; it's a matter of the user's flavour of semantics: do they expect to see ''' (a)''' the count of objects that are no more, regardless of whether ''they'' have been the one to delete them (perhaps some got deleted concurrently)? Or do they expect to see '''(b)''' the count of objects deleted by cause of their exact action and by their exact action only?3 Usually this is not a problem; it's a matter of the user's flavour of semantics: do they expect to see '''[=#sem-a (a)]''' the count of objects that are no more, regardless of whether ''they'' have been the one to delete them (perhaps some got deleted concurrently)? Or do they expect to see '''[=#sem-b (b)]''' the count of objects deleted by cause of their exact action and by their exact action only? 4 4 5 5 There is no universally correct answer here, but I think we should assume that the current behaviour of '''(a)''' should not be changed. … … 22 22 23 23 Whether "''not doing some of that deletion process''" falls under "''customization''" is up for debate. 24 Also, what are the alternatives? Ideally the queryset for the interstitial confirmation dialog would be easy to override, so that the developer could raise the warning about excluding the two objects there and then, and as a side effect the interstitial will display the correct candidate objects right there, '''''and''''' in the subsequent "''Successfully deleted [...]''" message.24 Also, what are the alternatives? Ideally the queryset for the interstitial confirmation dialog would be easy to override, so that the developer could raise the warning about excluding the two objects there and then, and as a side effect the interstitial will display the correct candidate objects right there, '''''and''''' display a correct count (conforming to the status quo semantics flavour [#sem-a (a)]) in the subsequent "''Successfully deleted [...]''" message. 25 25 26 26 == Proposed solutions 27 27 1. Make queryset for interstitial overridable as suggested above in '''Discussion'''. 28 28 or 29 2. Let `delete_queryset()` optionally return a count of actually deleted objects. Currently the method doesn't return anything, thus it is straightforward to provide backward compatibility: if nothing (`None`) is returned, the current behaviour persists, but if a count is returned, then that count will go into the message raised. As a side effect, this will flip the count semantics discussed under '''Status quo''' from ''' (b)''' to '''(a)''', which could be held against this solution.29 2. Let `delete_queryset()` optionally return a count of actually deleted objects. Currently the method doesn't return anything, thus it is straightforward to provide backward compatibility: if nothing (`None`) is returned, the current behaviour persists, but if a count is returned, then that count will go into the message raised. As a side effect, this will flip the count semantics discussed under '''Status quo''' from '''[#sem-a (a)]''' to '''[#sem-b (b)]''', which could be held against this solution. 30 30 31 31 I regard solution '''1''' as the superior, neatest solution, yet solution '''2''' is just a quick patch away (patch attached as an example, I'm happy to make a PR for it).