mirrors/django
1
0
Fork 0
mirror of https://github.com/django/django.git synced 2025-06-06 20:19:13 +00:00
Code Issues 32 Releases Wiki Activity

magic-removal: Restored __getitem__() on QuerySet and fixed bug in QuerySet._clone()

Browse Source 
git-svn-id: http://code.djangoproject.com/svn/django/branches/magic-removal@2167 bcc190cf-cafb-0310-a4f2-bffc1f526a37
This commit is contained in:
Adrian Holovaty 2006-01-30 03:33:40 +00:00
parent f771586deb
commit 9708aa51c0
1 changed files with 15 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -90,6 +90,20 @@ class QuerySet(object):
def __iter__(self): def __iter__(self):
return iter(self._get_data()) return iter(self._get_data())
def __getitem__(self, k):
"Retrieve an item or slice from the set of results."
# __getitem__ can't return QuerySet instances, because filter() and
# order_by() on the result would break badly. This means we don't have
# to worry about arithmetic with self._limit or self._offset -- they'll
# both be None at this point.
if self._result_cache is None:
if isinstance(k, slice):
return list(self._clone(_offset=k.start, _limit=k.stop))[::k.step]
else:
return self._clone(_offset=k, _limit=1).get()
else:
return self._result_cache[k]
#################################### ####################################
# METHODS THAT DO DATABASE QUERIES # # METHODS THAT DO DATABASE QUERIES #
#################################### ####################################
@ -205,6 +219,7 @@ class QuerySet(object):
c._tables = self._tables c._tables = self._tables
c._offset = self._offset c._offset = self._offset
c._limit = self._limit c._limit = self._limit
c.__dict__.update(kwargs)
return c return c
def _get_data(self): def _get_data(self):
@ -319,30 +334,6 @@ class QuerySet(object):
# combined = self._combine(other) # combined = self._combine(other)
# combined._filter = self._filter | other._filter # combined._filter = self._filter | other._filter
# return combined # return combined
#
# def __getitem__(self, k):
# """Retrieve an item or slice from the set of results"""
# # getitem can't return query instances, because .filter()
# # and .order_by() methods on the result would break badly.
# # This means we don't have to worry about arithmetic with
# # self._limit or self._offset - they will both be None
# # at this point
# if isinstance(k, slice):
# # Get a new query if we haven't already got data from db
# if self._result_cache is None:
# # slice.stop and slice.start
# clone = self._clone(_offset=k.start, _limit=k.stop)
# return list(clone)[::k.step]
# # TODO - we are throwing away this retrieved data.
# # We could cache it if we had some kind of sparse
# # list structure we could put it in.
# else:
# return self._result_cache[k]
#
# else:
# # TODO: possibly use a new query which just gets one item
# # if we haven't already got them all?
# return self._fetch_data()[k]
class QOperator: class QOperator:
"Base class for QAnd and QOr" "Base class for QAnd and QOr"