#13240: Modify RelatedManager and ManyRelatedManager add() and remove() to
accept
QuerySets and iterables
---------------------------------------------------+------------------------
Reporter: gabrielhurley | Owner:
gabrielhurley
Status: assigned | Milestone: 1.3
Component: Database layer (models, ORM) | Version: SVN
Resolution: | Keywords:
Stage: Accepted | Has_patch: 1
Needs_docs: 0 | Needs_tests: 0
Needs_better_patch: 1 |
---------------------------------------------------+------------------------
Comment (by gabrielhurley):
My limiting factor here is backwards compatibility. I don't love this API
either, but I don't see a simple way of improving it without breaking
existing code. I'm all ears for suggestions.
However, the fact that the current system 1) naively assumes anything
passed in that's not an *args of model instances is instead a set of pk's;
and 2) that a queryset is not a valid input both seem quite broken to me.
Number one there regularly produces confusing and frustrating errors as
well.
So please, point me in the right direction. I'd like to see this fixed.
But saying "this isn't it" doesn't go anywhere.
--
Ticket URL: <http://code.djangoproject.com/ticket/13240#comment:9>
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.
