Browse files

Fixed #17435 - Clarified that QuerySet.update returns the number of r…

…ows matched
  • Loading branch information...
1 parent cc337a7 commit 6d46c740d80b0c7f75064bc6bb4d99b15b106ba4 @timgraham timgraham committed Oct 6, 2012
Showing with 4 additions and 2 deletions.
  1. +2 −1 docs/ref/models/querysets.txt
  2. +2 −1 docs/topics/db/queries.txt
3 docs/ref/models/querysets.txt
@@ -1564,7 +1564,8 @@ update
.. method:: update(**kwargs)
Performs an SQL update query for the specified fields, and returns
-the number of rows affected.
+the number of rows matched (which may not be equal to the number of rows
+updated if some rows already have the new value).
For example, to turn comments off for all blog entries published in 2010,
you could do this::
3 docs/topics/db/queries.txt
@@ -959,7 +959,8 @@ new value to be the new model instance you want to point to. For example::
>>> Entry.objects.all().update(blog=b)
The ``update()`` method is applied instantly and returns the number of rows
-affected by the query. The only restriction on the
+matched by the query (which may not be equal to the number of rows updated if
+some rows already have the new value). The only restriction on the
:class:`~django.db.models.query.QuerySet` that is updated is that it can only
access one database table, the model's main table. You can filter based on
related fields, but you can only update columns in the model's main

0 comments on commit 6d46c74

Please sign in to comment.