From 79ade1fb2c38997c34e51f4bf495fe1ccf7494e2 Mon Sep 17 00:00:00 2001 From: Patrick Uiterwijk Date: Oct 08 2018 13:53:47 +0000 Subject: Allow providing a custom DB session to is_repo_committer Signed-off-by: Patrick Uiterwijk --- diff --git a/pagure/utils.py b/pagure/utils.py index 3d71822..66e0aec 100644 --- a/pagure/utils.py +++ b/pagure/utils.py @@ -121,7 +121,7 @@ def is_repo_admin(repo_obj, username=None): ) -def is_repo_committer(repo_obj, username=None): +def is_repo_committer(repo_obj, username=None, session=None): """ Return whether the user is a committer of the provided repo. """ if username is None: if not authenticated(): @@ -131,7 +131,9 @@ def is_repo_committer(repo_obj, username=None): username = flask.g.fas_user.username usergroups = set(flask.g.fas_user.groups) else: - user = pagure.lib.get_user(flask.g.session, username) + if not session: + session = flask.g.session + user = pagure.lib.get_user(session, username) usergroups = set(user.groups) # If the user is main admin -> yep