Opened 6 years ago
Last modified 6 years ago
#29447 closed Bug
RelatedManager.set() doesn't pass bulk keyword argument to clear() — at Version 1
Reported by: | Jeremy Lainé | Owned by: | nobody |
---|---|---|---|
Component: | Database layer (models, ORM) | Version: | 2.0 |
Severity: | Normal | Keywords: | |
Cc: | Triage Stage: | Accepted | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description (last modified by )
For reverse many-to-one relations, the set
method's bulk
keyword argument is passed down to the add()
and remove()
methods. It isn't however passed down to the clear()
method:
This looks like a bug, as passing bulk=False, clear=True results in the following behavior:
- clear() is invoked in "bulk" mode, not triggering signals
- add() is invoked in "non-bulk" mode, signals are triggered
Admitedly, I'm not too sure why you would want to do this, but at the very least the docs should mention if this combination is unsupported.
I spotted this while reading the related_descriptors.py
code to fix #29440.