Skip to content
Snippets Groups Projects
Commit 45b9034d authored by Fredrik Jonsson's avatar Fredrik Jonsson
Browse files

Revert: Allow staff reviewers to be deleted.

parent fc03cfb4
No related branches found
No related tags found
No related merge requests found
...@@ -6,6 +6,7 @@ from django import forms ...@@ -6,6 +6,7 @@ from django import forms
from django.utils.safestring import mark_safe from django.utils.safestring import mark_safe
from django.utils.text import slugify from django.utils.text import slugify
from django.utils.translation import ugettext_lazy as _ from django.utils.translation import ugettext_lazy as _
from django_select2.forms import Select2Widget
from hypha.apply.categories.models import MetaTerm from hypha.apply.categories.models import MetaTerm
from hypha.apply.users.models import User from hypha.apply.users.models import User
...@@ -238,8 +239,6 @@ class UpdateReviewersForm(ApplicationSubmissionModelForm): ...@@ -238,8 +239,6 @@ class UpdateReviewersForm(ApplicationSubmissionModelForm):
for role, reviewer in assigned_roles.items(): for role, reviewer in assigned_roles.items():
if reviewer: if reviewer:
AssignedReviewers.objects.update_role(role, reviewer, instance) AssignedReviewers.objects.update_role(role, reviewer, instance)
else:
AssignedReviewers.objects.filter(role=role, submission=instance).delete()
# 2. Update non-role reviewers # 2. Update non-role reviewers
# 2a. Remove those not on form # 2a. Remove those not on form
...@@ -303,9 +302,6 @@ class BatchUpdateReviewersForm(forms.Form): ...@@ -303,9 +302,6 @@ class BatchUpdateReviewersForm(forms.Form):
for role, reviewer in assigned_roles.items(): for role, reviewer in assigned_roles.items():
if reviewer: if reviewer:
AssignedReviewers.objects.update_role(role, reviewer, *submissions) AssignedReviewers.objects.update_role(role, reviewer, *submissions)
else:
for submission in submissions:
AssignedReviewers.objects.filter(role=role, submission=submission).delete()
return None return None
...@@ -318,6 +314,9 @@ def make_role_reviewer_fields(): ...@@ -318,6 +314,9 @@ def make_role_reviewer_fields():
field_name = 'role_reviewer_' + slugify(str(role)) field_name = 'role_reviewer_' + slugify(str(role))
field = forms.ModelChoiceField( field = forms.ModelChoiceField(
queryset=staff_reviewers, queryset=staff_reviewers,
widget=Select2Widget(attrs={
'data-placeholder': 'Select a reviewer',
}),
required=False, required=False,
label=mark_safe(render_icon(role.icon) + f'{role.name} Reviewer'), label=mark_safe(render_icon(role.icon) + f'{role.name} Reviewer'),
) )
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment