From 670139492d1a8c7f70aeb715d78f7f22d00f2d9b Mon Sep 17 00:00:00 2001 From: Jeremy Stretch Date: Fri, 29 May 2020 13:47:19 -0400 Subject: [PATCH] Fix permission action evaluation --- netbox/users/migrations/0009_replicate_permissions.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/netbox/users/migrations/0009_replicate_permissions.py b/netbox/users/migrations/0009_replicate_permissions.py index ba0663a0c..c5e4d364c 100644 --- a/netbox/users/migrations/0009_replicate_permissions.py +++ b/netbox/users/migrations/0009_replicate_permissions.py @@ -14,8 +14,11 @@ def replicate_permissions(apps, schema_editor): # TODO: Optimize this iteration so that ObjectPermissions with identical sets of users and groups # are combined into a single ObjectPermission instance. for perm in Permission.objects.all(): - print(f'Replicating permission {perm.codename}') - action, model_name = perm.codename.split('_') + # Account for non-standard permission names; e.g. napalm_read + if perm.codename.split('_')[0] in ACTIONS: + action = perm.codename.split('_')[0] + else: + action = perm.codename if perm.group_set.exists() or perm.user_set.exists(): obj_perm = ObjectPermission(actions=[action])