Opened 5 years ago

Closed 4 years ago

#14257 closed (fixed)

Remove isValidIDList from ManyToManyField in django/db/models/fields/related.py

Reported by: andrewsk Owned by: andrewsk
Component: Database layer (models, ORM) Version: master
Severity: Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

method isValidIDList seems to be an internal method used in Django before MultiDb.
It is not used anywhere in Django code now.
Also see #13771

Attachments (1)

rm_isValidIDList.diff (1.3 KB) - added by andrewsk 5 years ago.

Download all attachments as: .zip

Change History (3)

Changed 5 years ago by andrewsk

comment:1 Changed 5 years ago by andrewsk

  • Needs documentation unset
  • Needs tests unset
  • Owner changed from nobody to andrewsk
  • Patch needs improvement unset
  • Status changed from new to assigned

comment:2 Changed 4 years ago by Alex

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [14602]) Fixed #14257 -- removed ManyToManyField.isValidIDList which was completely dead code (unused since magic-removal). Thanks to andrewsk for the patch.

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