Opened 16 years ago
Closed 16 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 , 16 years ago
| Attachment: | t12025_r11366.patch added |
|---|
by , 16 years ago
| Attachment: | t12025_r11366.2.patch added |
|---|
comment:1 by , 16 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 , 16 years ago
| Attachment: | t12025_r11366.4.patch added |
|---|
add delete confirmation for top (deleted) obj
follow-ups: 3 4 comment:2 by , 16 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 , 16 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 , 16 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 , 16 years ago
| Resolution: | → duplicate |
|---|---|
| Status: | new → closed |
Closing as a duplicate of #6191.
added callback to edit confirmation message, merged with ticket #6191