#6191: Admin delete view doesn't show all items in some circumstances
-------------------------------------------+--------------------------------
Reporter: nicklane | Owner: carljm
Status: new | Milestone:
Component: django.contrib.admin | Version: SVN
Resolution: | Keywords: nfa-someday
Stage: Accepted | Has_patch: 1
Needs_docs: 0 | Needs_tests: 0
Needs_better_patch: 1 |
-------------------------------------------+--------------------------------
Comment (by carljm):
Discovered two more differences between this patch and current behavior.
1. Currently the list of "related objects to be deleted" includes entries
for "one or more foo in bar: my_bar", where foo is a M2M field on the bar
model pointing to the object about to be deleted. With the patch, m2m
intermediary relationship objects are only listed if you have an explicit
"through" model, in which case each "through" object is listed separately.
IMO, an m2m without an explicit through model is conceptually just a
relationship; the fact that an entry in a table will be deleted is just a
relationship implementation detail that doesn't require notification. If
the previous behavior is preferred, however, I am willing to add it to the
patch.
2. In general, this patch only ever lists a given object once, even if
there are multiple relationship paths to it. Current behavior can
potentially list the same object many times if there are several paths to
it.
--
Ticket URL: <http://code.djangoproject.com/ticket/6191#comment:18>
Django <http://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 django-upda...@googlegroups.com.
To unsubscribe from this group, send email to
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en.
