Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#20654 closed Bug (fixed)

Type mismatch in the docs for m2m_changed's pk_set argument

Reported by: Daniel Izquierdo Owned by: nobody
Component: Documentation Version: 1.5
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

The docs state that m2m_changed's pk_set argument is a list of primary key values: ​https://docs.djangoproject.com/en/dev/ref/signals/

But it's actually a set. See: ​https://github.com/django/django/blob/master/django/db/models/fields/related.py

Pull request: ​https://github.com/django/django/pull/1302

Change History (3)

comment:1 by Claude Paroz, 11 years ago

Easy pickings: set
Has patch: set
Triage Stage: Unreviewed → Ready for checkin
Type: Uncategorized → Bug

comment:2 by Baptiste Mispelon <bmispelon@…>, 11 years ago

Resolution: → fixed
Status: new → closed

In 9a2b07f1b45741da39a7606474aec3548780032b:

Fixed #20654 -- Fixed type of m2m_changed's pk_set arg in docs

comment:3 by Tim Graham <timograham@…>, 11 years ago

In 030ef4d1afe8c49da24ae7e6a1b8205041db8c61:

[1.5.x] Fixed #20654 -- Fixed type of m2m_changed's pk_set arg in docs

Backport of 9a2b07f1b4 from master

Note: See TracTickets for help on using tickets.
Back to Top