Opened 7 years ago

Closed 6 years ago

#12016 closed (fixed)

timefield's are showing as "(None)" in the list_display of the admin interface when time is set to 0:00

Reported by: Tim Kersten Owned by: nobody
Component: contrib.admin Version: 1.1
Severity: Keywords: list_display, timefield
Cc: Tim Kersten Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

A timefield colume containing the time 00:00 would show up in the admin as "(None)" rather than "midnight" when displayed as part of the list_display.

The cause is an if statement that should be checking for None, rather than evaluating to False.

Attachments (1)

timefield_check_for_none_fix.patch (819 bytes) - added by Tim Kersten 7 years ago.

Download all attachments as: .zip

Change History (6)

Changed 7 years ago by Tim Kersten

comment:1 Changed 7 years ago by Tim Kersten

Forgot to mention that the patch is against trunk, specifically r11603, since the bug also exists there. I wasn't sure if the version for this bug report should be set to 1.1 or SVN since it's there in both.

comment:2 Changed 7 years ago by Tim Kersten

Summary: timefield's are showing as "(None)" in the list_display of the admin interfacetimefield's are showing as "(None)" in the list_display of the admin interface when time is set to 0:00

comment:3 Changed 7 years ago by Tim Graham

Triage Stage: UnreviewedReady for checkin

Confirmed this affects 1.1.X and that the fix works. Looks like this code has been refactored a bit on trunk, so this does not affect 1.2.

comment:4 Changed 7 years ago by Tim Kersten

Cc: Tim Kersten added

comment:5 Changed 6 years ago by Luke Plant

Resolution: fixed
Status: newclosed

Since this does not affect trunk (according to timo), and we aren't fixing bugs on 1.1.X anymore, I'm just closing as FIXED.

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