Opened 15 years ago
Closed 15 years ago
#12025 closed (duplicate)
Show generic relation on deleted objects
Reported by: | Owned by: | nobody | |
---|---|---|---|
Component: | Uncategorized | Version: | 1.1 |
Severity: | Keywords: | ||
Cc: | Triage Stage: | Unreviewed | |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | yes |
Easy pickings: | no | UI/UX: | no |
Description
Generic foreign key relations are deleted but not shown in confirmation page.
Attachments (5)
Change History (10)
by , 15 years ago
Attachment: | t12025_r11366.patch added |
---|
by , 15 years ago
Attachment: | t12025_r11366.2.patch added |
---|
comment:1 by , 15 years ago
Patch needs improvement: | set |
---|
This patch uses a mutable default value which is bad practice and will cause it to fail when called more than once.
by , 15 years ago
Attachment: | t12025_r11366.4.patch added |
---|
add delete confirmation for top (deleted) obj
follow-ups: 3 4 comment:2 by , 15 years ago
This patch a) still has the mutable default value, and b) includes django.contrib.contenttypes in the ORM, we want to work to reducing that dependency, not increasing it.
comment:3 by , 15 years ago
Replying to Alex:
This patch a) still has the mutable default value, and b) includes django.contrib.contenttypes in the ORM, we want to work to reducing that dependency, not increasing it.
a) Which mutable default value are you talking about, is it "objs_seen", if yes, you suggest to start this value with what and why ??
b) Humm, ok, i will see what i can do to avoid that. If you have a suggestion i would appreciate.
comment:4 by , 15 years ago
Replying to Alex:
This patch a) still has the mutable default value, and b) includes django.contrib.contenttypes in the ORM, we want to work to reducing that dependency, not increasing it.
Unfortunately the ORM already depends on contrib.contenttypes in at least one other place: source:/django/trunk/django/db/models/sql/subqueries.py#29 - agree that we should be looking to get rid of that, not add more dependency.
This patch would be eclipsed by the patch on #6191, which replaces the admin's related-objects-finding with delegation to the actual ORM code. I'm looking at what it would take to fix this as part of #6191.
comment:5 by , 15 years ago
Resolution: | → duplicate |
---|---|
Status: | new → closed |
Closing as a duplicate of #6191.
added callback to edit confirmation message, merged with ticket #6191