#17962: Django Admin delete_view could benefit from extension points
-------------------------------+--------------------
     Reporter:  gcc            |      Owner:  nobody
         Type:  Uncategorized  |     Status:  new
    Component:  Uncategorized  |    Version:  SVN
     Severity:  Normal         |   Keywords:
 Triage Stage:  Unreviewed     |  Has patch:  1
Easy pickings:  0              |      UI/UX:  0
-------------------------------+--------------------
 I want to increase control over who can delete what objects using the
 admin interface.

 `delete_view` calls `has_delete_permission()`, which is fine, but then it
 calls `get_deleted_objects()` from `django.contrib.admin.util`.

 This is very hard to override without monkey-patching
 `get_deleted_objects()`, because it requires replacing the whole of
 `delete_view` just to change one line.

 I propose calling a member method `get_deleted_objects()`, which by
 default just calls the one from `util`, to make it easy to override this
 behaviour.

 I've attached a patch without tests, because the existing tests should
 check that this doesn't break anything.

-- 
Ticket URL: <https://code.djangoproject.com/ticket/17962>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.

Reply via email to