ValueError: too many values to unpack during migrate

[tubaman]

On step 2.5 of http://dkerwin.github.com/django-pki/installation/configuration.html, when running migrate I'm getting this error:

(myproject_pinax)pkiplayground@pkiplayground:~/sandboxes/myproject/server2 [master]$ ./manage.py migrate
Running migrations for pki:
 - Migrating forwards to 0013_auto__add_field_certificate_crl_dpoints__del_field_certificateauthorit.
 > pki:0001_initial
 > pki:0002_auto__add_field_certificateauthority_crl_distribution
 > pki:0003_auto__add_pkichangelog
 > pki:0004_auto__add_keyusage__add_x509extension__add_extendedkeyusage__del_field
 > pki:0005_load_eku_and_ku_fixture
Installed 25 object(s) from 1 fixture(s)
 > pki:0006_update_objects_to_x509_extensions
 > pki:0007_auto__del_field_certificateauthority_subcas_allowed
 > pki:0008_auto__del_field_certificate_cert_extension
 > pki:0009_auto__del_field_certificateauthority_type
 > pki:0010_auto__del_field_certificate_pem_encoded__del_field_certificateauthorit
 > pki:0011_add_pki_download_permission
Adding content type 'pki | certificateauthority'
Adding content type 'pki | certificate'
Adding content type 'pki | pkichangelog'
Adding content type 'pki | x509extension'
Adding content type 'pki | keyusage'
Adding content type 'pki | extendedkeyusage'
 > pki:0012_auto__add_unique_keyusage_name__add_unique_extendedkeyusage_name
 > pki:0013_auto__add_field_certificate_crl_dpoints__del_field_certificateauthorit
Traceback (most recent call last):
  File "./manage.py", line 20, in <module>
    execute_from_command_line()
  File "/home/pkiplayground/python_envs/myproject_pinax/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 429, in execute_from_command_line
    utility.execute()
  File "/home/pkiplayground/python_envs/myproject_pinax/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 379, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/home/pkiplayground/python_envs/myproject_pinax/local/lib/python2.7/site-packages/django/core/management/base.py", line 191, in run_from_argv
    self.execute(*args, **options.__dict__)
  File "/home/pkiplayground/python_envs/myproject_pinax/local/lib/python2.7/site-packages/django/core/management/base.py", line 220, in execute
    output = self.handle(*args, **options)
  File "/home/pkiplayground/python_envs/myproject_pinax/local/lib/python2.7/site-packages/south/management/commands/migrate.py", line 105, in handle
    ignore_ghosts = ignore_ghosts,
  File "/home/pkiplayground/python_envs/myproject_pinax/local/lib/python2.7/site-packages/south/migration/__init__.py", line 191, in migrate_app
    success = migrator.migrate_many(target, workplan, database)
  File "/home/pkiplayground/python_envs/myproject_pinax/local/lib/python2.7/site-packages/south/migration/migrators.py", line 221, in migrate_many
    result = migrator.__class__.migrate_many(migrator, target, migrations, database)
  File "/home/pkiplayground/python_envs/myproject_pinax/local/lib/python2.7/site-packages/south/migration/migrators.py", line 298, in migrate_many
    interactive=self.interactive)
  File "/home/pkiplayground/python_envs/myproject_pinax/local/lib/python2.7/site-packages/south/db/generic.py", line 808, in send_pending_create_signals
    interactive=interactive)
  File "/home/pkiplayground/python_envs/myproject_pinax/local/lib/python2.7/site-packages/south/db/generic.py", line 855, in really_send_create_signal
    db=self.db_alias,
  File "/home/pkiplayground/python_envs/myproject_pinax/local/lib/python2.7/site-packages/django/dispatch/dispatcher.py", line 172, in send
    response = receiver(signal=self, sender=sender, **named)
  File "/home/pkiplayground/python_envs/myproject_pinax/local/lib/python2.7/site-packages/django/contrib/auth/management/__init__.py", line 44, in create_permissions
    for ctype, (codename, name) in searched_perms:
ValueError: too many values to unpack
(myproject_pinax)pkiplayground@pkiplayground:~/sandboxes/myproject/server2 [master]$

[tubaman]

FYI, This is using r18275734586793c2c0043662cda33b3ee6cecf4f (next branch)

[dkerwin]

Thanks for reporting this. I'll have a look at your issue asap. Thanks!

[Tegereo]

This is a pretty easy one.

diff --git pki/models.py pki/models.py
index 65d76d3..0a03e0b 100644
--- pki/models.py
+++ pki/models.py
@@ -524,7 +524,7 @@ class CertificateAuthority(CertificateBase):
     class Meta:
         db_table = 'pki_certificateauthority'
         verbose_name_plural = 'Certificate Authorities'
-        permissions = (("can_download", "Can download"))
+        permissions = (("can_download", "Can download"),)

     def __unicode__(self):
         return self.common_name
@@ -919,7 +919,7 @@ class Certificate(CertificateBase):
     class Meta:
         db_table = 'pki_certificate'
         verbose_name_plural = 'Certificates'
-        permissions = (("can_download", "Can download"))
+        permissions = (("can_download", "Can download"),)
         unique_together = (("name", "parent"), ("common_name", "parent"))

     def __unicode__(self):

@Tegereo just stumbled upon this when using depends_on, and it would have been clearer if it was stated that you need an extra comma to indicate it's a pack of tuples:

doesn't work, shows that 'too many values to unpack' error:

(
    ('myapp', '0001')  # no comma
)

works correctly:

(
    ('myapp', '0001'),  # has comma
)