From 82b9c8b9ce458c1d2fd2b82a0944ee09b086325c Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Apr 26 2015 09:34:59 +0000 Subject: Let add_user_to_group check if the user to be added exists --- diff --git a/pagure/lib/__init__.py b/pagure/lib/__init__.py index 203d9be..5493d41 100644 --- a/pagure/lib/__init__.py +++ b/pagure/lib/__init__.py @@ -1812,9 +1812,13 @@ def search_groups(session, pattern=None, grp_name=None, grp_type=None): return query.all() -def add_user_to_group(session, new_user, group, user): +def add_user_to_group(session, username, group, user): ''' Add the specified user to the given group. ''' + new_user = search_user(session, username=username) + if not new_user: + raise PagureException('No user `%s` found' % username) + for guser in group.users: if guser.username == new_user.username: return 'User `%s` already in the group, nothing to change.' % (