Opened 12 years ago

Closed 12 years ago

Last modified 11 years ago

#18691 closed Bug (wontfix)

No timezone support for TimeField

Reported by: tejinderss@… Owned by: nobody
Component: Database layer (models, ORM) Version: 1.4
Severity: Normal Keywords: timezone, datetime, datetimefield, timefield
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Django 1.4 does not store aware times for TimeField, it only does for DateTimeField. Its really annoying, as python datetime.time object does support TZINFO just like datetime.datetime objects.
Steps to reproduce:
In the view i have set the timezone to timezone.activate(timezone) and after that i used form's TimeInput field to get the time. Now when i store it, it does not make the conversion from the current timezone to default timezone (which is set to UTC), it just naive time object. If i follow the same steps with DateTimeField, it works as expected, the datetime is first converted from current timezone to default one and then stored to the db.

Db used:
Postgres 9.1 with psycopg2 adapter

Change History (6)

comment:1 by Aymeric Augustin, 12 years ago

Resolution: wontfix
Status: newclosed

This is a design decision and it's explained in the documentation. In short aware time objects are pathological creatures :)

It is fairly easy to create a custom AwareTimeField if you need it.

comment:2 by tejinderss@…, 12 years ago

Thanks for the quick response, could you please point me to the direction where i could get a help with AwareTimeField

comment:3 by Aymeric Augustin, 12 years ago

Here's how aware-datetime support was added DateTimeField: https://github.com/django/django/commit/9b1cb755a2#L14L735

You must do similar changes in a custom TimeField. You can either inherit Django's TimeField and override a few methods or copy its definition and edit it -- I don't know which way will look more clean.

Version 0, edited 12 years ago by Aymeric Augustin (next)

comment:4 by tejinderss@…, 12 years ago

Ok i will have a look, thanks a lot.

comment:5 by anonymous, 11 years ago

I think this can be handled with just a custom form field. For those looking for a solution, you might try this: https://gist.github.com/jfialkoff/7311019

comment:6 by joshua.fialkoff@…, 11 years ago

Sorry, I posted that last comment in haste and eventually realized that the proposed solution was not a good way to handle this issue. Here is a better-thought-out solution: http://stackoverflow.com/questions/19780082/how-to-make-timefield-timezone-aware/19780083#19780083. Feel free to delete the last comment.

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