GrouperAPI: dict formatting #42

return boolean
UAL-RE · Sep 18, 2020 · 168c006 · 168c006
1 parent 6a8b600
commit 168c006
Showing 1 changed file with 9 additions and 8 deletions.
diff --git a/requiam/grouper_admin.py b/requiam/grouper_admin.py
@@ -160,14 +160,13 @@ def add_privilege(self, access_group, target_group, target_group_type, privilege
                 raise Exception(f"Could NOT find access_group: {access_group}")
 
             # initialize
-            params = {
-                'WsRestAssignGrouperPrivilegesLiteRequest': {
-                    'allowed': 'T',
-                    'subjectId': access_group_detail['uuid'],
-                    'privilegeName': '',
-                    'groupName': target_groupname,
-                    'privilegeType': 'access'
-                }
+            params = dict()
+            params['WsRestAssignGrouperPrivilegesLiteRequest'] = {
+                'allowed': 'T',
+                'subjectId': access_group_detail['uuid'],
+                'privilegeName': '',
+                'groupName': target_groupname,
+                'privilegeType': 'access'
             }
 
             for privilege in privileges:
@@ -178,3 +177,5 @@ def add_privilege(self, access_group, target_group, target_group_type, privilege
 
                 if metadata['resultCode'] not in ['SUCCESS_ALLOWED', 'SUCCESS_ALLOWED_ALREADY_EXISTED']:
                     raise ValueError(f"Unexpected result received: {metadata['resultCode']}")
+
+        return True