#24424: Removing a model's last field results in SQL syntax error on SQLite
----------------------------+------------------------------------
     Reporter:  adnam       |                    Owner:
         Type:  Bug         |                   Status:  new
    Component:  Migrations  |                  Version:  1.7
     Severity:  Normal      |               Resolution:
     Keywords:              |             Triage Stage:  Accepted
    Has patch:  1           |      Needs documentation:  0
  Needs tests:  0           |  Patch needs improvement:  1
Easy pickings:  0           |                    UI/UX:  0
----------------------------+------------------------------------

Comment (by jwineinger):

 The case below is nearly exactly the same one I ran into.

 {{{#!python
 # models.py
 from django.db import models

 class Thing(models.Model):
     pass

 class Blob(Thing):
     pass


 # 0001_initial.py
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals

 from django.db import migrations, models


 class Migration(migrations.Migration):

     dependencies = [
     ]

     operations = [
         migrations.CreateModel(
             name='Thing',
             fields=[
                 ('id', models.AutoField(auto_created=True,
 primary_key=True, serialize=False, verbose_name='ID')),
             ],
         ),
         migrations.CreateModel(
             name='Blob',
             fields=[
                 ('thing_ptr', models.OneToOneField(auto_created=True,
 parent_link=True, primary_key=True, serialize=False, to='app_a.Thing')),
             ],
             bases=('app_a.thing',),
         ),
     ]
 }}}

 Making `Thing` inherit a new model `Bar`:

 {{{#!python
 # models.py
 from django.db import models

 class Bar(models.Model):
     pass


 # 0002_auto.py
 # -*- coding: utf-8 -*-
 from __future__ import unicode_literals

 from django.db import models, migrations


 class Migration(migrations.Migration):

     dependencies = [
         ('thing', '0001_initial'),
     ]

     operations = [
         migrations.RemoveField(
             model_name='blob',
             name='thing_ptr',
         ),
         migrations.DeleteModel(
             name='Blob',
         ),
     ]
 }}}

 {{{
 ./manage.py migrate
 Operations to perform:
   Synchronize unmigrated apps: staticfiles, messages
   Apply all migrations: auth, admin, thing, sessions, contenttypes
 Synchronizing apps without migrations:
   Creating tables...
     Running deferred SQL...
   Installing custom SQL...
 Running migrations:
   Rendering model states... DONE
   Applying contenttypes.0001_initial... OK
   Applying auth.0001_initial... OK
   Applying admin.0001_initial... OK
   Applying contenttypes.0002_remove_content_type_name... OK
   Applying auth.0002_alter_permission_name_max_length... OK
   Applying auth.0003_alter_user_email_max_length... OK
   Applying auth.0004_alter_user_username_opts... OK
   Applying auth.0005_alter_user_last_login_null... OK
   Applying auth.0006_require_contenttypes_0002... OK
   Applying sessions.0001_initial... OK
   Applying thing.0001_initial... OK
   Applying thing.0002_auto_20150415_0113...Traceback (most recent call
 last):
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/db/backends/utils.py", line 64, in execute
     return self.cursor.execute(sql, params)
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/db/backends/sqlite3/base.py", line 318, in execute
     return Database.Cursor.execute(self, query, params)
 sqlite3.OperationalError: near ")": syntax error

 The above exception was the direct cause of the following exception:

 Traceback (most recent call last):
   File "./manage.py", line 10, in <module>
     execute_from_command_line(sys.argv)
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/core/management/__init__.py", line 338, in
 execute_from_command_line
     utility.execute()
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/core/management/__init__.py", line 330, in execute
     self.fetch_command(subcommand).run_from_argv(self.argv)
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/core/management/base.py", line 390, in run_from_argv
     self.execute(*args, **cmd_options)
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/core/management/base.py", line 441, in execute
     output = self.handle(*args, **options)
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/core/management/commands/migrate.py", line 221, in handle
     executor.migrate(targets, plan, fake=fake, fake_initial=fake_initial)
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/db/migrations/executor.py", line 110, in migrate
     self.apply_migration(states[migration], migration, fake=fake,
 fake_initial=fake_initial)
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/db/migrations/executor.py", line 147, in apply_migration
     state = migration.apply(state, schema_editor)
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/db/migrations/migration.py", line 115, in apply
     operation.database_forwards(self.app_label, schema_editor, old_state,
 project_state)
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/db/migrations/operations/fields.py", line 121, in
 database_forwards
     schema_editor.remove_field(from_model,
 from_model._meta.get_field(self.name))
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/db/backends/sqlite3/schema.py", line 194, in remove_field
     self._remake_table(model, delete_fields=[field])
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/db/backends/sqlite3/schema.py", line 144, in _remake_table
     self.quote_name(model._meta.db_table),
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/db/backends/base/schema.py", line 107, in execute
     cursor.execute(sql, params)
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/db/backends/utils.py", line 79, in execute
     return super(CursorDebugWrapper, self).execute(sql, params)
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/db/backends/utils.py", line 64, in execute
     return self.cursor.execute(sql, params)
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/db/utils.py", line 97, in __exit__
     six.reraise(dj_exc_type, dj_exc_value, traceback)
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/utils/six.py", line 658, in reraise
     raise value.with_traceback(tb)
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/db/backends/utils.py", line 64, in execute
     return self.cursor.execute(sql, params)
   File "/Users/jawineinger/venvs/test/lib/python3.4/site-
 packages/django/db/backends/sqlite3/base.py", line 318, in execute
     return Database.Cursor.execute(self, query, params)
 django.db.utils.OperationalError: near ")": syntax error
 }}}

--
Ticket URL: <https://code.djangoproject.com/ticket/24424#comment:23>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.30f9d07e031525bcc24fd1019c10b5b6%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to