summaryrefslogtreecommitdiff
path: root/docs/topics
diff options
context:
space:
mode:
authorNick Pope <nick@nickpope.me.uk>2022-02-22 09:29:38 +0000
committerGitHub <noreply@github.com>2022-02-22 10:29:38 +0100
commit847f46e9bf88964484c8b76a10af753ea1018311 (patch)
tree13aced534cbae373f47cce8fce1444ad0e8e01d3 /docs/topics
parent7ba6ebe9149ae38257d70100e8bfbfd0da189862 (diff)
Removed redundant QuerySet.all() calls in docs and tests.
Most QuerySet methods are mapped onto the Manager and, in general, it isn't necessary to call .all() on the manager.
Diffstat (limited to 'docs/topics')
-rw-r--r--docs/topics/db/aggregation.txt6
-rw-r--r--docs/topics/db/multi-db.txt4
-rw-r--r--docs/topics/db/queries.txt4
3 files changed, 7 insertions, 7 deletions
diff --git a/docs/topics/db/aggregation.txt b/docs/topics/db/aggregation.txt
index 772de0abdb..42be569cb4 100644
--- a/docs/topics/db/aggregation.txt
+++ b/docs/topics/db/aggregation.txt
@@ -56,12 +56,12 @@ above::
# Average price across all books.
>>> from django.db.models import Avg
- >>> Book.objects.all().aggregate(Avg('price'))
+ >>> Book.objects.aggregate(Avg('price'))
{'price__avg': 34.35}
# Max price across all books.
>>> from django.db.models import Max
- >>> Book.objects.all().aggregate(Max('price'))
+ >>> Book.objects.aggregate(Max('price'))
{'price__max': Decimal('81.20')}
# Difference between the highest priced book and the average price of all books.
@@ -111,7 +111,7 @@ belong to this ``QuerySet``. This is done by appending an ``aggregate()``
clause onto the ``QuerySet``::
>>> from django.db.models import Avg
- >>> Book.objects.all().aggregate(Avg('price'))
+ >>> Book.objects.aggregate(Avg('price'))
{'price__avg': 34.35}
The ``all()`` is redundant in this example, so this could be simplified to::
diff --git a/docs/topics/db/multi-db.txt b/docs/topics/db/multi-db.txt
index d74972b0e0..cf7d6fef4f 100644
--- a/docs/topics/db/multi-db.txt
+++ b/docs/topics/db/multi-db.txt
@@ -459,10 +459,10 @@ which you want to run the query. For example::
>>> Author.objects.all()
>>> # So will this.
- >>> Author.objects.using('default').all()
+ >>> Author.objects.using('default')
>>> # This will run on the 'other' database.
- >>> Author.objects.using('other').all()
+ >>> Author.objects.using('other')
Selecting a database for ``save()``
-----------------------------------
diff --git a/docs/topics/db/queries.txt b/docs/topics/db/queries.txt
index 23326fb778..9e45f66bb5 100644
--- a/docs/topics/db/queries.txt
+++ b/docs/topics/db/queries.txt
@@ -1327,7 +1327,7 @@ new value to be the new model instance you want to point to. For example::
>>> b = Blog.objects.get(pk=1)
# Change every Entry so that it belongs to this Blog.
- >>> Entry.objects.all().update(blog=b)
+ >>> Entry.objects.update(blog=b)
The ``update()`` method is applied instantly and returns the number of rows
matched by the query (which may not be equal to the number of rows updated if
@@ -1361,7 +1361,7 @@ update one field based on the value of another field in the model. This is
especially useful for incrementing counters based upon their current value. For
example, to increment the pingback count for every entry in the blog::
- >>> Entry.objects.all().update(number_of_pingbacks=F('number_of_pingbacks') + 1)
+ >>> Entry.objects.update(number_of_pingbacks=F('number_of_pingbacks') + 1)
However, unlike ``F()`` objects in filter and exclude clauses, you can't
introduce joins when you use ``F()`` objects in an update -- you can only