sourcehut
/
todo.sr.ht
mirror of https://git.sr.ht/~sircmpwn/todo.sr.ht
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Fix error when assigning already assigned user 

Assigning user that is already assigned causes unhandled GraphQLError.
This commit adds validation that the user is not assigned.

References: https://todo.sr.ht/~sircmpwn/todo.sr.ht/275
This commit is contained in:
Krystian Chachuła 2023-11-26 23:29:24 +01:00 committed by Drew DeVault
parent 8ad96d48ce
commit 031383d55f
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
todosrht/blueprints/ticket.py
View File

@ -487,6 +487,8 @@ def ticket_assign(owner, name, ticket_id):
valid = Validation(request)
tracker, ticket = _assignment_get_ticket(owner, name, ticket_id)
user = _assignment_get_user(valid)
valid.expect(user not in ticket.assigned_users,
"This user is already assigned to this ticket.", field="username")
if not valid.ok:
_, access = get_ticket(ticket.tracker, ticket_id)
ctx = get_ticket_context(ticket, ticket.tracker, access)