From 43956c35bb52ec8d832eeff55027105b1c09fb0c Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Jan 12 2016 11:17:43 +0000 Subject: Fix resetting the assignee of a PR Up until now we could assign a PR to someone, but not remove it and leave it blank. This commit fixes this. --- diff --git a/pagure/ui/fork.py b/pagure/ui/fork.py index 4eaa812..5e043e0 100644 --- a/pagure/ui/fork.py +++ b/pagure/ui/fork.py @@ -774,14 +774,14 @@ def set_assignee_requests(repo, requestid, username=None): if request.status != 'Open': flask.abort(403, 'Pull-request closed') - form = pagure.forms.AddUserForm() + form = pagure.forms.ConfirmationForm() if form.validate_on_submit(): try: # Assign or update assignee of the ticket message = pagure.lib.add_pull_request_assignee( SESSION, request=request, - assignee=form.user.data or None, + assignee=flask.request.form.get('user', '').strip() or None, user=flask.g.fas_user.username, requestfolder=APP.config['REQUESTS_FOLDER'],) if message: