#26333: makemigration on a PointField field with a default value does not use 
the
value's class.
--------------------------+------------------------------------------------
     Reporter:            |      Owner:  nobody
  simondrabble            |
         Type:  Bug       |     Status:  new
    Component:            |    Version:  1.8
  Migrations              |
     Severity:  Normal    |   Keywords:  makemigration gis point pointfield
 Triage Stage:            |  Has patch:  0
  Unreviewed              |
Easy pickings:  0         |      UI/UX:  0
--------------------------+------------------------------------------------
 Given:

 {{{
     from django.contrib.gis.db import models as gis
     from django.contrib.gis.geos import Point
     from django.db import models

     POINT = Point(-104.9903, 39.7392, srid=4326)

     class PagedModel(models.Model):
         objects = gis.GeoManager()
         location = gis.PointField(srid=4326, default=POINT)
 }}}

 then

 {{{
 python manage.py makemigration
 python manage.py migrate
 }}}

 Observed:

 {{{
   File "./manage.py", line 10, in <module>
     execute_from_command_line(sys.argv)
   File "/pyenv/django1.8/lib/python2.7/site-
 packages/django/core/management/__init__.py", line 354, in
 execute_from_command_line
     utility.execute()
   File "/pyenv/django1.8/lib/python2.7/site-
 packages/django/core/management/__init__.py", line 346, in execute
     self.fetch_command(subcommand).run_from_argv(self.argv)
   File "/pyenv/django1.8/lib/python2.7/site-
 packages/django/core/management/base.py", line 394, in run_from_argv
     self.execute(*args, **cmd_options)
   File "/pyenv/django1.8/lib/python2.7/site-
 packages/django/core/management/base.py", line 445, in execute
     output = self.handle(*args, **options)
   File "/pyenv/django1.8/lib/python2.7/site-
 packages/django/core/management/commands/migrate.py", line 222, in handle
     executor.migrate(targets, plan, fake=fake, fake_initial=fake_initial)
   File "/pyenv/django1.8/lib/python2.7/site-
 packages/django/db/migrations/executor.py", line 110, in migrate
     self.apply_migration(states[migration], migration, fake=fake,
 fake_initial=fake_initial)
   File "/pyenv/django1.8/lib/python2.7/site-
 packages/django/db/migrations/executor.py", line 148, in apply_migration
     state = migration.apply(state, schema_editor)
   File "/pyenv/django1.8/lib/python2.7/site-
 packages/django/db/migrations/migration.py", line 115, in apply
     operation.database_forwards(self.app_label, schema_editor, old_state,
 project_state)
   File "/pyenv/django1.8/lib/python2.7/site-
 packages/django/db/migrations/operations/fields.py", line 62, in
 database_forwards
     field,
   File "/pyenv/django1.8/lib/python2.7/site-
 packages/django/contrib/gis/db/backends/postgis/schema.py", line 94, in
 add_field
     super(PostGISSchemaEditor, self).add_field(model, field)
   File "/pyenv/django1.8/lib/python2.7/site-
 packages/django/db/backends/base/schema.py", line 384, in add_field
     definition, params = self.column_sql(model, field,
 include_default=True)
   File "/pyenv/django1.8/lib/python2.7/site-
 packages/django/contrib/gis/db/backends/postgis/schema.py", line 30, in
 column_sql
     column_sql = super(PostGISSchemaEditor, self).column_sql(model, field,
 include_default)
   File "/pyenv/django1.8/lib/python2.7/site-
 packages/django/db/backends/base/schema.py", line 146, in column_sql
     default_value = self.effective_default(field)
   File "/pyenv/django1.8/lib/python2.7/site-
 packages/django/db/backends/base/schema.py", line 211, in
 effective_default
     default = field.get_db_prep_save(default, self.connection)
   File "/pyenv/django1.8/lib/python2.7/site-
 packages/django/contrib/gis/db/models/fields.py", line 307, in
 get_db_prep_save
     return connection.ops.Adapter(self.get_prep_value(value))
   File "/pyenv/django1.8/lib/python2.7/site-
 packages/django/contrib/gis/db/models/fields.py", line 210, in
 get_prep_value
     raise ValueError('Cannot use object with type %s for a geometry lookup
 parameter.' % type(geom).__name__)
 ValueError: Cannot use object with type float for a geometry lookup
 parameter.
 }}}

 Expected:

 Migration completes successfully.

 Hand-editing the generate migration file to use the Point class results in
 success.

 IOW, changing:

 {{{
 field=django.contrib.gis.db.models.fields.PointField(default=(-104.9903,
 39.7392), srid=4326),
 }}}

 to

 {{{
 field=django.contrib.gis.db.models.fields.PointField(default=Point(-104.9903,
 39.7392, srid=4326)),
 }}}

 resolves the issue.

--
Ticket URL: <https://code.djangoproject.com/ticket/26333>
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/055.ffdab849e1acb01aee946b38970918b2%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to