diff --git a/opentech/apply/funds/forms.py b/opentech/apply/funds/forms.py index ee2d2e838db62c782cfcff35fee89b37c3edc869..1f7fcd2c964be25f59ef39aa08bf4444090ecc0c 100644 --- a/opentech/apply/funds/forms.py +++ b/opentech/apply/funds/forms.py @@ -159,13 +159,14 @@ class UpdateReviewersForm(forms.ModelForm): instance.assigned.filter(role=None).exclude(reviewer__in=current_reviewers).delete() # Add new reviewers + reviewers_no_role_existing = instance.reviewers.filter(assignedreviewers__role=None) AssignedReviewers.objects.bulk_create( AssignedReviewers( submission=instance, role=None, reviewer=reviewer, ) for reviewer in current_reviewers - if reviewer not in instance.reviewers.filter(assignedreviewers__role=None) + if reviewer not in reviewers_no_role_existing ) # Update or create role reviewers