hobo_notify: search role only by uuid, not name (#35158)
gitea-wip/wcs/pipeline/head There was a failure building this commit Details
gitea/wcs/pipeline/head Build started... Details

This commit is contained in:
Thomas NOËL 2019-07-31 21:08:40 +02:00
parent a2452f875c
commit db3d4870c1
2 changed files with 6 additions and 1 deletions

View File

@ -534,6 +534,11 @@ def test_process_notification_user_provision(pub):
u'name': u'Service état civil',
u'description': u'etc.',
},
{
u'uuid': u'unknown-uuid',
u'name': u'Service enfance',
u'description': u'',
},
],
}
]

View File

@ -190,7 +190,7 @@ class CmdHoboNotify(Command):
user.is_admin = o.get('is_superuser', False)
user.roles = []
for role_ref in o.get('roles', []):
role = Role.resolve(role_ref['uuid'], name=role_ref.get('name'))
role = Role.resolve(role_ref['uuid'])
if role and role.id not in user.roles:
user.add_roles([role.id])
user.set_attributes_from_formdata(user.form_data)