From 9ef89266c4c6f544c3489be851dcb8ab3b73d08f Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Apr 02 2015 10:52:37 +0000 Subject: Prevent admins from adding to a project a user that is already present --- diff --git a/pagure/lib/__init__.py b/pagure/lib/__init__.py index 4292299..1ce09f9 100644 --- a/pagure/lib/__init__.py +++ b/pagure/lib/__init__.py @@ -495,6 +495,13 @@ def add_user_to_project(session, project, new_user, user): new_user_obj = __get_user(session, new_user) user_obj = __get_user(session, user) + users = set([user.user for user in project.users]) + users.add(project.user.user) + if new_user in users: + raise pagure.exceptions.PagureException( + 'This user is already listed on this project.' + ) + project_user = model.ProjectUser( project_id=project.id, user_id=new_user_obj.id,