Opened 10 years ago

Closed 8 years ago

#6372 closed (fixed)

archive_week generic views should default allow_empty to False

Reported by: David Paccoud Owned by: nobody
Component: Generic views Version: master
Severity: Keywords:
Cc: Triage Stage: Design decision needed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:


In ticket #685, it has been decided that allow_empty parameter should default to True for index generic views. archive_week in not considered an index view here (as archive_year, archive_month or archive_day) but it is the only generic view where allow_empty default to True from the beginning.

I think it would make sens, for better consistency, that the allow_empty parameter default to False in archive_week. This would break backwards compatibility.

Attachments (1)

fix_allow_empty.diff (1.3 KB) - added by David Paccoud 10 years ago.
Fix allow_empty in archive_week generic view

Download all attachments as: .zip

Change History (3)

Changed 10 years ago by David Paccoud

Attachment: fix_allow_empty.diff added

Fix allow_empty in archive_week generic view

comment:1 Changed 10 years ago by Simon Greenhill <dev@…>

Triage Stage: UnreviewedDesign decision needed

comment:2 Changed 8 years ago by Russell Keith-Magee

Resolution: fixed
Status: newclosed

This has been addressed as part of the introduction of class-based views in [14254].

Note: See TracTickets for help on using tickets.
Back to Top