.

longs/templates/longs/index.html

@@ -4,7 +4,7 @@
 {% for exam in exams %}
 <div class="longs">
     <h1><a href="{% url 'longs:exam_overview' pk=exam.pk %}">Exam: {{ exam.name }} </a></h1>
-    {% if request.user.is_staff %}<a href="{% url 'longs:mark_question_overview' pk=exam.pk sk=0 %}">Mark answers</a>{% endif %}
+    {% if request.user.is_staff %}<a href="{% url 'longs:mark_question_overview' exam_id=exam.pk sk=0 %}">Mark answers</a>{% endif %}
     {% if request.user.is_staff %}<a href="{% url 'longs:exam_scores_cid' pk=exam.pk %}">Scores</a>{% endif %}
 </div>
 {% endfor %}

longs/templates/longs/mark_overview.html

@@ -8,13 +8,13 @@
     <div>
         <button class="show-all-button">Show all</button><button class="show-unmarked-button">Show unmarked</button>
     </div>
-    <div id="stark-marking-button"><a href="{% url 'longs:mark_question_overview' pk=exam.pk sk=0 %}"><button>Click here to start
+    <div id="stark-marking-button"><a href="{% url 'longs:mark_question_overview' exam_id=exam.pk sk=0 %}"><button>Click here to start
                 marking</button></a></div>
 
     <ul id="question-mark-list">
         {% for question, unmarked_count in question_unmarked_map %}
         <li data-markcount={{unmarked_count}} {% if unmarked_count %}class="unmarked" {% endif %}><a
-                href="{% url 'longs:mark_question_overview' pk=exam.pk sk=forloop.counter0 %}">Question {{forloop.counter }}:
+                href="{% url 'longs:mark_question_overview' exam_id=exam.pk sk=forloop.counter0 %}">Question {{forloop.counter }}:
                 {{ question }}</a><br /> Answers to mark: {{unmarked_count}}</li>
         {% endfor %}
     </ul>

longs/urls.py

@@ -67,7 +67,7 @@ urlpatterns = [
         name="question_review",
     ),
     path("exam/<int:exam_id>/<int:question_number>/<int:cid>/mark", views.mark_answer, name="mark_answer"),
-    path("exam/<int:pk>/<int:sk>/mark", views.mark_question_overview, name="mark_question_overview"),
+    path("exam/<int:exam_id>/<int:sk>/mark", views.mark_question_overview, name="mark_question_overview"),
     path("exam/<int:pk>/mark", views.LongExamViews.mark_overview, name="mark_overview"),
     #     path("exam/<int:pk>/<int:sk>/", views.exam_take, name="exam_take"),
     path(

longs/views.py

@@ -722,8 +722,8 @@ def mark_answer(request, exam_id, question_number, cid):
 # @user_passes_test(user_is_admin, login_url="/accounts/login")
 @login_required
 @user_is_long_marker
-def mark_question_overview(request, pk, sk):
-    exam = get_object_or_404(Exam, pk=pk)
+def mark_question_overview(request, exam_id, sk):
+    exam = get_object_or_404(Exam, pk=exam_id)
 
     questions = exam.exam_questions.all()
 
@@ -740,7 +740,7 @@ def mark_question_overview(request, pk, sk):
         raise Http404("Exam question does not exist")
 
 
-    user_answers = question.cid_user_answers.filter(exam__id=pk)
+    user_answers = question.cid_user_answers.filter(exam__id=exam_id)
 
     unmarked_count = user_answers.filter(score=CidUserAnswer.ScoreOptions.UNMARKED).count()