datetime.time default value on migrations

[Oscar Ramirez]

When trying to use datetime.time(10) value for a django.db.models.fields.TimeField model default value on the default value questioner the system raises this exception:

The datetime module is available, so you can do e.g. datetime.date.today()
>>> datetime.time(10)
Traceback (most recent call last):
File "manage.py", line 10, in <module>
  execute_from_command_line(sys.argv)
[....]
File "/Users/tuxskar/.virtualenvs/caluny/lib/python2.7/site-packages/django/db/migrations/questioner.py", line 126, in ask_not_null_addition
    return eval(code, {}, {"datetime": datetime_safe})
  File "<string>", line 1, in <module>
TypeError: 'module' object is not callable

I have made a patch and attach here. (Github Pull Request)

[Jayadeep Karnati]

Can you provide the steps to reproduce the bug, more clearly? I don't get any error when I enter the following in the shell of django 1.7:

>>> import datetime
>>> datetime.time(10)

Is that what you meant?

Also, you have mentioned that a patch has been attached but I couldn't find any attachments.

[Tim Graham]

The patch is linked in the "Pull Requests:" field on the ticket. The problem is with the migration questioner, not the plain Django shell.

[Markus Holtermann]

Link to PR: ​https://github.com/django/django/pull/3739

[Tim Graham <timograham@…>]

In 54085b0f9ba7d9f705f9b9c90d3433b0ef6aa042:

Fixed #23998 -- Added datetime.time support to migrations questioner.

[Tim Graham <timograham@…>]

In 1ad5deedd4c51729be680e3c2730c2ef5ab4f831:

[1.7.x] Fixed #23998 -- Added datetime.time support to migrations questioner.

Backport of 54085b0f9ba7d9f705f9b9c90d3433b0ef6aa042 from master