mirrors/django
1
0
Fork 0
mirror of https://github.com/django/django.git synced 2025-07-06 10:49:17 +00:00
Code Issues 32 Releases Wiki Activity

queryset-refactor: Removed another FIXME.

Browse Source 
This time, simplifying the delete() method.


git-svn-id: http://code.djangoproject.com/svn/django/branches/queryset-refactor@7092 bcc190cf-cafb-0310-a4f2-bffc1f526a37
This commit is contained in:
Malcolm Tredinnick 2008-02-06 04:10:58 +00:00
parent 04a55d663f
commit c23436be59
1 changed files with 4 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
django/db/models/query.py
View File

@ -218,7 +218,6 @@ class _QuerySet(object):
qs.query.add_filter(('pk__in', id_list))
return dict([(obj._get_pk_val(), obj) for obj in qs.iterator()])
# XXX Mostly DONE
def delete(self):
"""
Deletes the records in the current QuerySet.
@ -234,22 +233,16 @@ class _QuerySet(object):
# Delete objects in chunks to prevent the list of related objects from
# becoming too long.
more_objects = True
while more_objects:
while 1:
# Collect all the objects to be deleted in this chunk, and all the
# objects that are related to the objects that are to be deleted.
seen_objs = SortedDict()
more_objects = False
for object in del_query[:CHUNK_SIZE]:
more_objects = True
object._collect_sub_objects(seen_objs)
# If one or more objects were found, delete them.
# Otherwise, stop looping.
# FIXME: Does "if seen_objs:.." work here? If so, we can get rid of
# more_objects.
if more_objects:
delete_objects(seen_objs)
if not seen_objs:
break
delete_objects(seen_objs)
# Clear the result cache, in case this QuerySet gets reused.
self._result_cache = None