Document that RelatedManager.set() doesn't accept bulk keyword argument for many-to-many relations

[Jeremy Lainé]

Since Django 2.0, trying to pass bulk=True to a many-to-many RelatedManager results in an error:

TypeError: set() got an unexpected keyword argument 'bulk'

The documentation however still states that the "bulk" argument exists, so I'm not sure whether the documentation or the code is wrong.

Looking at the django code, the set() method for reverse many-to-one, and forward many-to-many relations don't have the same signature.

In create_reverse_many_to_one_manager:

def set(self, objs, *, bulk=True, clear=False):

In create_forward_many_to_many_manager:

def set(self, objs, *, clear=False):

[Tim Graham]

I believe the documentation should be updated similar to abe6c5defefc7057e7fb5f47b79643f7b89f7d90.

[Jeremy Lainé]

I have added a PR, which tries to make the "bulk" argument more consistently documented:

• the signatures for clear() and remove() did not reference "bulk" at all

• I added a note in clear(), remove() and set() to the effect that the "bulk" argument does not exist for many-to-many relations.

​https://github.com/django/django/pull/9982

[Jeremy Lainé]

In the process, I discovered that for one-to-many relations, the "bulk" argument is correctly passed down to add() and remove(), but not to clear(). This looks like a bug, I will open another issue.

[Tim Graham <timograham@…>]

In 74179296:

Fixed #29440 -- Doc'd where the bulk argument applies in RelatedManager.

[Tim Graham <timograham@…>]

In 36c35aba:

[2.1.x] Fixed #29440 -- Doc'd where the bulk argument applies in RelatedManager.

Backport of 741792961815cf4a95c5ce8ab590dfc7700c6153 from master