Merge lp:~ricardokirkner/django-openid-auth/compatible-sqlite-migration into lp:~django-openid-auth/django-openid-auth/trunk

[Ricardo Kirkner]

make migration more generic and compatible with sqlite

Ensures that all Permission objects for models are up to date.

Given this is Django a version updgrade, and there's a South
bug[0] that won't let it track the permissions Meta for
models, we need to ensure all of them get synced correctly for
all installed apps.

[0] http://south.aeracode.org/ticket/211

[lp:~ricardokirkner/django-openid-auth/compatible-sqlite-migration]

113. By Ricardo Kirkner on 2014-09-01

fix extra whitespace

[Fabián Ezequiel Gallina (fgallina)]

LGTM

review: Approve

=== modified file 'django_openid_auth/migrations/0002_add_perm_account_verified.py'
--- django_openid_auth/migrations/0002_add_perm_account_verified.py	2014-08-29 16:50:41 +0000
+++ django_openid_auth/migrations/0002_add_perm_account_verified.py	2014-09-01 11:42:01 +0000
@@ -2,18 +2,20 @@
 from south.utils import datetime_utils as datetime
 from south.db import db
 from south.v2 import DataMigration
-from django.db import models
+from django.contrib.auth.management import create_permissions
+from django.db import transaction
+from django.db.models import get_app, get_models
 
 class Migration(DataMigration):
 
     def forwards(self, orm):
         "Write your forwards methods here."
-        ct, created = orm['contenttypes.ContentType'].objects.get_or_create(
-            model='useropenid', app_label='django_openid_auth',
-            defaults=dict(name='user open id'))
-        perm, created = orm['auth.permission'].objects.get_or_create(
-            content_type=ct, codename='account_verified',
-            defaults=dict(name=u'The OpenID account has been verified'))
+        models = get_models()
+        apps = [get_app(model._meta.app_label) for model in models]
+        for app in apps:
+            # Safeguard for sqlite. See http://bit.ly/1rRHC9Y
+            with transaction.autocommit():
+                create_permissions(app, models, verbosity=0)
 
     def backwards(self, orm):
         "Write your backwards methods here."