Opened 17 years ago

Last modified 13 years ago

#6749 closed

Generic week view displays Sunday objects in both the current and previous week — at Initial Version

Reported by: Jim Garrison Owned by: nobody
Component: Generic views Version: dev
Severity: Normal Keywords: archive_week
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: yes Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

Objects on a Sunday are mistakenly displayed in both the correct week and the previous week.

The relevant code from django/views/generic/date_based.py (function archive_week):

    # Calculate first and last day of week, for use in a date-range lookup.
    first_day = date
    last_day = date + datetime.timedelta(days=7)
    lookup_kwargs = {'%s__range' % date_field: (first_day, last_day)}

first_day} is set from {{{date, which is the Sunday that begins the week. However, last_day is then set to be the day 7 days after that day, which is the next Sunday. The final line above sets the range to be from one Sunday to the next Sunday, but the range test is inclusive so both Sundays are included by mistake.

This bug can be fixed by changing the 7 to a 6. Trivial patch is attached.

Change History (1)

by Jim Garrison, 17 years ago

Attachment: week_view.diff added
Note: See TracTickets for help on using tickets.
Back to Top