Dropped Courses (#2262)

Co-authored-by: Jonas Dittrich <58814480+Kakadus@users.noreply.github.com>
Co-authored-by: Richard Ebeling <dev@richardebeling.de>

Author: 3 people

evap/contributor/forms.py

@@ -15,6 +15,12 @@ class EvaluationForm(forms.ModelForm):
     general_questionnaires: "forms.ModelMultipleChoiceField[Questionnaire]" = forms.ModelMultipleChoiceField(
         queryset=None, required=False, widget=CheckboxSelectMultiple, label=_("General questionnaires")
     )
+    dropout_questionnaires: "forms.ModelMultipleChoiceField[Questionnaire]" = forms.ModelMultipleChoiceField(
+        queryset=None,
+        required=False,
+        widget=CheckboxSelectMultiple,
+        label=_("Dropout questionnaires"),
+    )
     course = forms.ModelChoiceField(Course.objects.all(), disabled=True, required=False, widget=forms.HiddenInput())
     name_de_field = forms.CharField(label=_("Name (German)"), disabled=True, required=False)
     name_en_field = forms.CharField(label=_("Name (English)"), disabled=True, required=False)
@@ -28,6 +34,7 @@ class Meta:
             "vote_end_date",
             "participants",
             "general_questionnaires",
+            "dropout_questionnaires",
             "course",
         )
         field_classes = {
@@ -46,6 +53,12 @@ def __init__(self, *args, **kwargs):
             .distinct()
             .prefetch_related("questions")
         )
+        self.fields["dropout_questionnaires"].queryset = (
+            Questionnaire.objects.dropout_questionnaires()
+            .filter(Q(visibility=Questionnaire.Visibility.EDITORS) | Q(contributions__evaluation=self.instance))
+            .distinct()
+            .prefetch_related("questions")
+        )
 
         self.fields["vote_start_datetime"].localize = True
         self.fields["vote_end_date"].localize = True
@@ -55,9 +68,12 @@ def __init__(self, *args, **kwargs):
             queryset = (queryset | self.instance.participants.all()).distinct()
         self.fields["participants"].queryset = queryset
 
-        if self.instance.general_contribution:
+        if general_contribution := self.instance.general_contribution:
             self.fields["general_questionnaires"].initial = [
-                q.pk for q in self.instance.general_contribution.questionnaires.all()
+                q.pk for q in general_contribution.questionnaires.all() if q.is_general
+            ]
+            self.fields["dropout_questionnaires"].initial = [
+                q.pk for q in general_contribution.questionnaires.all() if q.is_dropout
             ]
 
         if not self.instance.allow_editors_to_edit:
@@ -73,6 +89,19 @@ def clean(self):
             self.add_error("vote_start_datetime", "")
             self.add_error("vote_end_date", _("The first day of evaluation must be before the last one."))
 
+        # Ensure locked questionnaires cannot be deselected and only unlocked ones can be added
+        selected_questionnaires = self.instance.general_contribution.questionnaires.filter(is_locked=True).distinct()
+        selected_questionnaires |= self.cleaned_data.get("general_questionnaires").filter(is_locked=False)
+        selected_questionnaires |= self.cleaned_data.get("dropout_questionnaires").filter(is_locked=False)
+
+        self.cleaned_data.update(
+            general_questionnaires=selected_questionnaires.exclude(type=Questionnaire.Type.DROPOUT),
+            dropout_questionnaires=selected_questionnaires.filter(type=Questionnaire.Type.DROPOUT),
+        )
+
+        if not self.cleaned_data.get("general_questionnaires"):
+            self.add_error("general_questionnaires", _("At least one questionnaire must be selected."))
+
     def clean_vote_end_date(self):
         vote_end_date = self.cleaned_data.get("vote_end_date")
 
@@ -83,22 +112,12 @@ def clean_vote_end_date(self):
 
         return vote_end_date
 
-    def clean_general_questionnaires(self):
-        # Ensure all locked questionnaires still have the same status (included or not)
-        not_locked = []
-        if self.cleaned_data.get("general_questionnaires"):
-            not_locked = list(self.cleaned_data.get("general_questionnaires").filter(is_locked=False))
-
-        locked = list(self.instance.general_contribution.questionnaires.filter(is_locked=True))
-
-        if not not_locked + locked:
-            self.add_error("general_questionnaires", _("At least one questionnaire must be selected."))
-
-        return not_locked + locked
-
     def save(self, *args, **kw):
         evaluation = super().save(*args, **kw)
-        evaluation.general_contribution.questionnaires.set(self.cleaned_data.get("general_questionnaires"))
+        selected_questionnaires = self.cleaned_data.get("general_questionnaires") | self.cleaned_data.get(
+            "dropout_questionnaires"
+        )
+        evaluation.general_contribution.questionnaires.set(selected_questionnaires)
         return evaluation
 
 

evap/contributor/templates/contributor_evaluation_form.html

@@ -78,7 +78,7 @@ <h5 class="card-title me-auto">{% translate 'Evaluation data' %}</h5>
                         </div>
                     {% endif %}
                 </div>
-                {% include 'evaluation_form_general_questionnaires.html' with evaluation_form=form %}
+                {% include 'evaluation_form_fields.html' with evaluation_form=form %}
             </div>
         </div>
         <div class="card mb-3">

evap/contributor/tests/test_views.py

@@ -272,6 +272,22 @@ def test_display_request_buttons(self):
         self.assertEqual(page.body.decode().count("Request changes"), 0)
         self.assertEqual(page.body.decode().count("Request creation of new account"), 2)
 
+    def test_questionnaire_input_hidden_if_options_empty(self):
+        Questionnaire.objects.update(visibility=Questionnaire.Visibility.MANAGERS)
+
+        self.evaluation.general_contribution.questionnaires.clear()
+        self.evaluation.save()
+
+        page = self.app.get(self.url, user=self.responsible, status=200)
+        self.assertNotContains(page, "General questionnaires")
+        self.assertNotContains(page, "Dropout questionnaires")
+
+        Questionnaire.objects.update(visibility=Questionnaire.Visibility.EDITORS)
+
+        page = self.app.get(self.url, user=self.responsible, status=200)
+        self.assertContains(page, "General questionnaires")
+        self.assertContains(page, "Dropout questionnaires")
+
 
 class TestContributorResultsExportView(WebTest):
     @classmethod

evap/contributor/views.py

@@ -157,7 +157,9 @@ def render_preview(request, formset, evaluation_form, evaluation):
             request.POST = None  # this prevents errors rendered in the vote form
 
             preview_response = mark_safe(
-                render_vote_page(request, evaluation, preview=True, for_rendering_in_modal=True).content.decode()
+                render_vote_page(
+                    request, evaluation, preview=True, for_rendering_in_modal=True, dropout=False
+                ).content.decode()
             )
             raise IntegrityError  # rollback transaction to discard the database writes
     except IntegrityError:
@@ -241,7 +243,7 @@ def evaluation_preview(request, evaluation_id):
     ):
         raise PermissionDenied
 
-    return render_vote_page(request, evaluation, preview=True)
+    return render_vote_page(request, evaluation, preview=True, dropout=False)
 
 
 @require_POST