Make course_id compulsory argument in download_assignment_file view function and format error message

1 files changed, 3 insertions, 3 deletions

diff --git a/yaksh/views.py b/yaksh/views.py
index 521096e..444b324 100644
--- a/yaksh/views.py
+++ b/yaksh/views.py
@@ -2098,14 +2098,14 @@ def update_email(request):
 
 @login_required
 @email_verified
-def download_assignment_file(request, quiz_id, course_id=None,
+def download_assignment_file(request, quiz_id, course_id,
                              question_id=None, user_id=None):
     user = request.user
     course = Course.objects.get(id=course_id)
     if (not course.is_creator(user) and not course.is_teacher(user) and
             not course.is_student(user)):
-        raise Http404("You are not allowed to download {}".format(
-            course.name.replace(" ", "_"))
+        raise Http404("You are not allowed to download files for {0}".format(
+            course.name)
         )
     qp = QuestionPaper.objects.get(quiz_id=quiz_id)
     assignment_files, file_name = AssignmentUpload.objects.get_assignments(