Table creation for models with only a single local field fails

[tommi@…]

Hi,

I noticed that South creates erroneous migration code in case of only one local field.

E.g. for model in app

buggy

class Target(models.Model):
    pass

following migration is created:

# encoding: utf-8
import datetime
from south.db import db
from south.v2 import SchemaMigration
from django.db import models

class Migration(SchemaMigration):
    
    def forwards(self, orm):
        
        # Adding model 'Target'
        db.create_table('buggy_target', (
            ('id', self.gf('django.db.models.fields.AutoField')(primary_key=True))
        ))
        db.send_create_signal('buggy', ['Target'])
    
    
    def backwards(self, orm):
        
        # Deleting model 'Target'
        db.delete_table('buggy_target')
    
    
    models = {
        'buggy.target': {
            'Meta': {'object_name': 'Target', 'managed': 'True'},
            'id': ('django.db.models.fields.AutoField', [], {'primary_key': 'True'})
        }
    }
    
    complete_apps = ['buggy']

And there we're missing a comma after the

('id', self.gf('django.db.models.fields.AutoField')(primary_key=True))

and thus unpacking in

create_table

fails.

Attached is a patch (diffed against 626:3c55f23ca089).

-Tommi Penttinen-

[anonymous]

Bugger, mispressed the submit button. But for completeness, I meant 'south.db.generic.create_table' with 'create_table'.

And although 'Target' as itself would be kind of useless, models with only many-to-many relationships would not contain any other local fields either.

-Tommi Penttinen-

[andrew]

Fixed in [d2cb8cb7060a].