#16818: Regression introduced by r16739 -- `ManyRelatedManager.add()` doesn't
commit to database
-------------------------------------+-------------------------------------
Reporter: pressureman | Owner: nobody
Type: Bug | Status: new
Component: Database layer | Version: SVN
(models, ORM) | Resolution:
Severity: Release blocker | Triage Stage: Accepted
Keywords: | Needs documentation: 0
Has patch: 1 | Patch needs improvement: 0
Needs tests: 0 | UI/UX: 0
Easy pickings: 0 |
-------------------------------------+-------------------------------------
Changes (by kmtracey):
* owner: kmtracey => nobody
* has_patch: 0 => 1
Comment:
Attached patch makes the test run successfully on the 3 backends I have
available to test where it was failing before (sqlite3, MySQL/InnoDB,
PostgreSQL). Possibly it would be a good idea to factor out that force-
managed stuff into a reusable bit that could be shared by different
operations that require it...in a brief scan I see it in the insert (where
I stole it from) and now bulk_insert and django/db/models/deletion.py has
something very similar. But first I'd like some confirmation from someone
with more ORM knowledge than me that that's the right thing to do to
correct the error here.
--
Ticket URL: <https://code.djangoproject.com/ticket/16818#comment:19>
Django <https://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 [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en.
