Opened 15 years ago

Last modified 15 years ago

#12251 closed

QuerySet.in_bulk() should accept a set() — at Version 2

Reported by: Johannes Dollinger Owned by: nobody
Component: Database layer (models, ORM) Version: dev
Severity: Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Alex Gaynor) 

--- django/db/models/query.py	(revision 11751)
+++ django/db/models/query.py	(working copy)
@@ -356,7 +356,7 @@
         """
         assert self.query.can_filter(), \
                 "Cannot use 'limit' or 'offset' with in_bulk"
-        assert isinstance(id_list, (tuple, list)), \
+        assert isinstance(id_list, (tuple, list, set, frozenset)), \
                 "in_bulk() must be provided with a list of IDs."
         if not id_list:
             return {}

Change History (3)

by Johannes Dollinger, 15 years ago

Attachment: 12251.queryset_in_bulk_set.diff​ added

comment:1 by anonymous, 15 years ago

Component: Uncategorized → Database layer (models, ORM)
Has patch: set

comment:2 by Alex Gaynor, 15 years ago

Description: modified (diff)
Triage Stage: Unreviewed → Ready for checkin
Version: 1.1 → SVN
Note: See TracTickets for help on using tickets.
Back to Top