Browse files

Merge pull request #318 from Matt-Stevens/master

Small doc change: clarify day and month
  • Loading branch information...
2 parents e5983af + a7d831a commit 3f53cac520d6a4f87b7b8e45eeb06921a58c515f @adrianholovaty adrianholovaty committed Sep 5, 2012
Showing with 2 additions and 2 deletions.
  1. +2 −2 docs/topics/db/queries.txt
4 docs/topics/db/queries.txt
@@ -235,14 +235,14 @@ refinements together. For example::
... ).exclude(
... ).filter(
- ... pub_date__gte=datetime(2005, 1, 1)
+ ... pub_date__gte=datetime(2005, 1, 30)
Django member

It would have been absolutely perfect to switch this example from datetimes to dates while we were there.

Django member
mjtamlyn added a note Sep 6, 2012

Hmm, it's a blog post model, so I guess pub_date is more likely to be a datetime than a date. I know you can compare the two, and this is a date, but it's not that clear.

Then again if it's not a date, it should probably be called published or pub_time not pub_date...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
... )
This takes the initial :class:`~django.db.models.query.QuerySet` of all entries
in the database, adds a filter, then an exclusion, then another filter. The
final result is a :class:`~django.db.models.query.QuerySet` containing all
entries with a headline that starts with "What", that were published between
-January 1, 2005, and the current day.
+January 30, 2005, and the current day.
.. _filtered-querysets-are-unique:

0 comments on commit 3f53cac

Please sign in to comment.