Opened 17 years ago

Closed 17 years ago

#6779 closed (fixed)

Don't show delete checkbox for "extra" inline items

Reported by: Chris Beaven Owned by: Brian Rosner
Component: contrib.admin Version: newforms-admin
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Easy fix, here's the patch.

Attachments (1)

inline_delete_only_existing.diff​ (2.6 KB ) - added by Chris Beaven 17 years ago.

Download all attachments as: .zip

Change History (6)

by Chris Beaven, 17 years ago

Attachment: inline_delete_only_existing.diff​ added

comment:1 by Brian Rosner, 17 years ago

Owner: changed from nobody to Brian Rosner
Status: new → assigned
Triage Stage: Unreviewed → Ready for checkin

comment:2 by Brian Rosner, 17 years ago

Triage Stage: Ready for checkin → Accepted

New hooks provide more places to do this nicely.

comment:3 by Chris Beaven, 17 years ago

Doh, I actually did that initially (at the formset level). I thought you'd prefer the more sedated version ;)

comment:4 by Brian Rosner, 17 years ago

Okay, actually thought about this and your patch is the most correct way. Will apply shortly.

comment:5 by Brian Rosner, 17 years ago

Resolution: → fixed
Status: assigned → closed

(In [7367]) newforms-admin: Fixed #6779 -- Removed the delete checkbox for "extra" inlines. Thanks, SmileyChris.

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