basic form for editing question types and weightings

Different form from options as it makes the logic much easier and they
are separate tasks.

Author: struan

ceuk-marking/urls.py

@@ -318,6 +318,11 @@
         questions.OptionsView.as_view(),
         name="edit_options",
     ),
+    path(
+        "questions/weightings/<section_name>/",
+        questions.WeightingsView.as_view(),
+        name="edit_weightings",
+    ),
     path(
         "questions/sections/",
         questions.SectionList.as_view(),

crowdsourcer/forms.py

@@ -31,6 +31,7 @@
     MarkingSession,
     Option,
     PublicAuthority,
+    Question,
     Response,
     ResponseType,
     Section,
@@ -579,3 +580,7 @@ def __init__(self, properties: {}, **kwargs):
 OptionFormset = modelformset_factory(
     Option, fields=["score"], extra=0, can_delete=False
 )
+
+QuestionFormset = modelformset_factory(
+    Question, fields=["question_type", "weighting"], extra=0, can_delete=False
+)

crowdsourcer/templates/crowdsourcer/questions/sections.html

@@ -13,7 +13,7 @@ <h1 class="mb-md-0 me-md-auto">Sections</h1>
 
     <ul>
         {% for section in sections %}
-        <li><a href="{% session_url 'edit_options' section.title %}">{{ section.title }}</li>
+        <li>{{section.title}} - <a href="{% session_url 'edit_weightings' section.title %}">weightings and type</a> : <a href="{% session_url 'edit_options' section.title %}">scores</a></li>
         {% endfor %}
     </ul>
 {% endif %}

crowdsourcer/templates/crowdsourcer/questions/weightings.html

@@ -0,0 +1,56 @@
+{% extends 'crowdsourcer/base.html' %}
+
+{% load crowdsourcer_tags django_bootstrap5 static %}
+
+{% block content %}
+{% if show_login %}
+    <h1 class="mb-4">Sign in</h1>
+    <a href="{% url 'login' %}">Sign in</a>
+{% else %}
+    <h3 class="mb-4">Types and Weightings for {{ section.title }} questions</h3>
+
+
+    <form action="" method="post">
+        {% csrf_token %}
+        {{ form.management_form }}
+        <div class="container">
+            <div class="row border-bottom fw-bold">
+                <div class="col-8">
+                    Question
+                </div>
+
+                <div class="col-2">
+                    Type
+                </div>
+
+                <div class="col-2">
+                    Weighting
+                </div>
+            </div>
+            {% for question_form in form %}
+                <fieldset class="pt-1 mb-3">
+                <div class="row py-0 border-bottom">
+                    <div class="col-8">
+                        <div class="mb-dd-4">
+                        <span class="fw-bold">{{ question_form.instance.number_and_part }}</span>
+                        {{ question_form.instance.description }}
+                        </div>
+                    </div>
+
+                    <div class="col-2">
+                        {% bootstrap_field question_form.question_type show_label=False %}
+                    </div>
+
+                    <div class="col-2">
+                        {% bootstrap_field question_form.weighting show_label=False %}
+                    </div>
+                </div>
+                {{ question_form.id }}
+                </fieldset>
+            {% endfor %}
+        </div>
+        <input type="submit" class="btn btn-primary" value="Update">
+    </form>
+
+{% endif %}
+{% endblock %}

crowdsourcer/views/questions.py

@@ -3,8 +3,8 @@
 from django.urls import reverse
 from django.views.generic import FormView, ListView
 
-from crowdsourcer.forms import OptionFormset
-from crowdsourcer.models import Option, Section
+from crowdsourcer.forms import OptionFormset, QuestionFormset
+from crowdsourcer.models import Option, Question, Section
 
 
 class SectionList(ListView):
@@ -57,3 +57,43 @@ def get_context_data(self, **kwargs):
     def form_valid(self, form):
         form.save()
         return super().form_valid(form)
+
+
+class WeightingsView(UserPassesTestMixin, FormView):
+    template_name = "crowdsourcer/questions/weightings.html"
+    form_class = QuestionFormset
+
+    def test_func(self):
+        return self.request.user.has_perm("crowdsourcer.can_manage_users")
+
+    def get_success_url(self):
+        return reverse(
+            "session_urls:edit_weightings",
+            kwargs={
+                "marking_session": self.request.current_session.label,
+                "section_name": "Buildings & Heating",
+            },
+        )
+
+    def get_form(self):
+        self.section = get_object_or_404(
+            Section,
+            title=self.kwargs["section_name"],
+            marking_session=self.request.current_session,
+        )
+
+        questions = Question.objects.filter(
+            section=self.section,
+        ).order_by("number", "number_part")
+        return self.form_class(queryset=questions, **self.get_form_kwargs())
+
+    def get_context_data(self, **kwargs):
+        context = super().get_context_data(**kwargs)
+
+        context["section"] = self.section
+
+        return context
+
+    def form_valid(self, form):
+        form.save()
+        return super().form_valid(form)