Opened 12 years ago
Closed 12 years ago
#21958 closed Bug (fixed)
Migrations fail on adding ForeignKey field to User model
| Reported by: | code22 | Owned by: | |
|---|---|---|---|
| Component: | Migrations | Version: | 1.7-alpha-1 |
| Severity: | Release blocker | Keywords: | |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
It seems that migrations have problems with adding columns with foreign key to User tables. This happens both if I use settings.AUTH_USER_MODEL and django.contrib.auth.model.User.
My model looked like this:
class TestModel(models.Model): text = models.CharField(max_length=10)
so if I add User field
class TestModel(models.Model): text = models.CharField(max_length=10) user = models.ForeignKey(settings.AUTH_USER_MODEL, null=True)
then run manage.py makemigrations and then manage.py migrate I get this:
Operations to perform:
Synchronize unmigrated apps: admin, contenttypes, auth, sessions
Apply all migrations: migr_test
Synchronizing apps without migrations:
Creating tables...
Installing custom SQL...
Installing indexes...
Installed 0 object(s) from 0 fixture(s)
Running migrations:
Applying migr_test.0002_testmodel_user...Traceback (most recent call last):
File "./manage.py", line 10, in <module>
execute_from_command_line(sys.argv)
File "/private/tmp/test_env/src/django/django/core/management/__init__.py", line 427, in execute_from_command_line
utility.execute()
File "/private/tmp/test_env/src/django/django/core/management/__init__.py", line 419, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/private/tmp/test_env/src/django/django/core/management/base.py", line 288, in run_from_argv
self.execute(*args, **options.__dict__)
File "/private/tmp/test_env/src/django/django/core/management/base.py", line 337, in execute
output = self.handle(*args, **options)
File "/private/tmp/test_env/src/django/django/core/management/commands/migrate.py", line 145, in handle
executor.migrate(targets, plan, fake=options.get("fake", False))
File "/private/tmp/test_env/src/django/django/db/migrations/executor.py", line 60, in migrate
self.apply_migration(migration, fake=fake)
File "/private/tmp/test_env/src/django/django/db/migrations/executor.py", line 94, in apply_migration
migration.apply(project_state, schema_editor)
File "/private/tmp/test_env/src/django/django/db/migrations/migration.py", line 97, in apply
operation.database_forwards(self.app_label, schema_editor, project_state, new_state)
File "/private/tmp/test_env/src/django/django/db/migrations/operations/fields.py", line 35, in database_forwards
field,
File "/private/tmp/test_env/src/django/django/db/backends/sqlite3/schema.py", line 96, in add_field
self._remake_table(model, create_fields=[field])
File "/private/tmp/test_env/src/django/django/db/backends/sqlite3/schema.py", line 33, in _remake_table
if field.get_default():
File "/private/tmp/test_env/src/django/django/db/models/fields/related.py", line 1649, in get_default
if isinstance(field_default, self.rel.to):
TypeError: isinstance() arg 2 must be a class, type, or tuple of classes and types
Used Python3.3.3 and Python2.7.6 with the same result
Change History (4)
comment:1 by , 12 years ago
| Severity: | Normal → Release blocker |
|---|---|
| Triage Stage: | Unreviewed → Accepted |
comment:2 by , 12 years ago
| Owner: | set to |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
comment:3 by , 12 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → new |
Dependency handling should also be done for "NOT NULL fields with no default"
comment:4 by , 12 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Please open a new ticket and explain the bug more fully; it's the not the same as this bug, so please don't re-open it unless you've found the same problem as before.
This looks like a bug with swappable models.