Opened 7 years ago

Closed 7 years ago

Last modified 4 years ago

#11022 closed Uncategorized (fixed)

Document that the admin bulk delete doesn't call Model.delete()

Reported by: Jacob Owned by: Jacob
Component: Documentation Version: master
Severity: Normal 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

As per #10751 the admin delete action doesn't call Model.delete(). That ticket is DDN and could go either way, but for now we need to explain this behavior in the bulk admin docs. We should indicate that the workaround is easy: define a custom bulk delete action that does call Model.delete().

Attachments (1)

11022-bulk-delete-warning-r10680.diff (1023 bytes) - added by Idan Gazit 7 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 7 years ago by Idan Gazit

Owner: changed from nobody to Idan Gazit

Changed 7 years ago by Idan Gazit

comment:2 Changed 7 years ago by Idan Gazit

Has patch: set
Needs documentation: unset
Version: 1.0SVN

Attached patch warns and provides link to QuerySet.delete docs. Needs review but otherwise complete.

comment:3 Changed 7 years ago by Alex Gaynor

Triage Stage: AcceptedReady for checkin

comment:4 Changed 7 years ago by Jacob

Owner: changed from Idan Gazit to Jacob
Status: newassigned

comment:5 Changed 7 years ago by Alex Gaynor

Resolution: fixed
Status: assignedclosed

Fixed in r10780.

comment:6 Changed 5 years ago by Jacob

milestone: 1.1

Milestone 1.1 deleted

comment:7 Changed 4 years ago by anonymous

Easy pickings: unset
Severity: Normal
Type: Uncategorized
UI/UX: unset

I'm having trouble figuring out what release this fix is included in. I'm in 1.2 and I still seem to be hitting this problem.

Note: See TracTickets for help on using tickets.
Back to Top