Commit: a3ab5407b549c59559b5b9b6a16066be2015c7ba - allura (git)

allura

Commit MetaInfo

• Revision: a3ab5407b549c59559b5b9b6a16066be2015c7ba (tree)
• Zeit: 2011-04-28 23:48:33
• Autor: Dave Brondsema <dbrondsema@geek...>
• Commiter: Dave Brondsema

Log Message

[#1909] more resilient to already converted data

Signed-off-by: Dave Brondsema <[e-mail]>

Ändern Zusammenfassung

• modified: scripts/update-acls.py

Diff

--- a/scripts/update-acls.py
+++ b/scripts/update-acls.py
@@ -84,6 +84,10 @@ def update_project_acl(project_doc):
 
 def update_neighborhood_acl(neighborhood_doc, init_doc):
     '''Convert nbhd admins users to --init-- project admins'''
+    if 'acl' not in neighborhood_doc:
+        log.warning('Neighborhood %s is already updated' % neighborhood_doc['name'])
+        return
+
     if TEST: log.info('Update nbhd %s', neighborhood_doc['name'])
     if 'acl' not in neighborhood_doc:
         log.warning('Neighborhood %s already updated', neighborhood_doc['name'])
@@ -112,6 +116,10 @@ def update_neighborhood_acl(neighborhood_doc, init_doc):
 
 def simple_acl_update(doc):
     '''Update dict-style to list-style ACL'''
+    if not isinstance(doc['acl'], dict):
+        log.warning('Already upgraded %s' % doc)
+        return
+
     new_acl = []
     for perm, role_ids in sorted(doc['acl'].iteritems()):
         for rid in role_ids: