From 8cde63946f0b5bf8f27e77a0d4bc8fcf9d92f305 Mon Sep 17 00:00:00 2001
From: Adil Mohak <adilmohak123@gmail.com>
Date: Mon, 30 Sep 2024 23:11:54 +0300
Subject: [PATCH] Restrict access to all programs page for student

---
 course/views.py                           | 2 +-
 templates/aside.html                      | 2 ++
 templates/course/course_registration.html | 3 +--
 templates/course/course_single.html       | 4 ++++
 templates/course/program_single.html      | 4 ++++
 templates/quiz/quiz_list.html             | 4 ++++
 templates/result/assessment_results.html  | 2 +-
 7 files changed, 17 insertions(+), 4 deletions(-)

diff --git a/course/views.py b/course/views.py
index 80fc39a..1679294 100644
--- a/course/views.py
+++ b/course/views.py
@@ -25,7 +25,7 @@ from .filters import ProgramFilter, CourseAllocationFilter
 from .models import Program, Course, CourseAllocation, Upload, UploadVideo
 
 
-@method_decorator([login_required], name="dispatch")
+@method_decorator([login_required, lecturer_required], name="dispatch")
 class ProgramFilterView(FilterView):
     filterset_class = ProgramFilter
     template_name = "course/program_list.html"
diff --git a/templates/aside.html b/templates/aside.html
index 12ba2bb..f216350 100644
--- a/templates/aside.html
+++ b/templates/aside.html
@@ -68,9 +68,11 @@
 			</li>
 			{% endif %}
 
+			{% if request.user.is_superuser or request.user.is_lecturer %}
 			<li class="{% if request.path == pro %}active{% endif %}">
 				<a href="{% url 'programs' %}"><i class="fas fa-book-open"></i>{% trans 'Programs & Courses' %}</a>
 			</li>
+			{% endif %}
 
 			{% if request.user.is_superuser or request.user.is_lecturer %}
 			<li class="{% if request.path == qce %}active{% endif %}">
diff --git a/templates/course/course_registration.html b/templates/course/course_registration.html
index 518eb6a..fed678e 100644
--- a/templates/course/course_registration.html
+++ b/templates/course/course_registration.html
@@ -35,8 +35,7 @@
         <p class="form-title fw-bold">{% trans 'Course Add' %}</p>
         <div class="container">
             <div class="d-flex justify-content-between mb-3">
-                <button title="{% trans 'Save Score' %}" type="submit" class="btn btn-primary"><i class="fa fa-plus"></i> {% trans 'Add
-                    Selected' %}</button>
+                <button title="{% trans 'Save Score' %}" type="submit" class="btn btn-primary"><i class="fa fa-plus"></i> {% trans 'Add Selected' %}</button>
             </div>
 
             <div class="table-responsive p-0 px-2 mt-2">
diff --git a/templates/course/course_single.html b/templates/course/course_single.html
index 3aca9a2..27ff2c9 100644
--- a/templates/course/course_single.html
+++ b/templates/course/course_single.html
@@ -8,7 +8,11 @@
 <nav style="--bs-breadcrumb-divider: '>';" aria-label="breadcrumb">
     <ol class="breadcrumb">
         <li class="breadcrumb-item"><a href="/">{% trans 'Home' %}</a></li>
+        {% if request.user.is_student %}
+        <li class="breadcrumb-item"><a href="{% url 'user_course_list' %}">{% trans 'My courses' %}</a></li>
+        {% else %}
         <li class="breadcrumb-item"><a href="{% url 'programs' %}">{% trans 'Programs' %}</a></li>
+        {% endif %}
         <li class="breadcrumb-item"><a href="{% url 'program_detail' course.program.id %}">{{ course.program }}</a></li>
         <li class="breadcrumb-item active" aria-current="page">{{ course }}</li>
     </ol>
diff --git a/templates/course/program_single.html b/templates/course/program_single.html
index 8b1eee1..c1ae053 100644
--- a/templates/course/program_single.html
+++ b/templates/course/program_single.html
@@ -8,7 +8,11 @@
 <nav style="--bs-breadcrumb-divider: '>';" aria-label="breadcrumb">
     <ol class="breadcrumb">
         <li class="breadcrumb-item"><a href="/">{% trans 'Home' %}</a></li>
+        {% if request.user.is_student %}
+        <li class="breadcrumb-item"><a href="{% url 'user_course_list' %}">{% trans 'My courses' %}</a></li>
+        {% else %}
         <li class="breadcrumb-item"><a href="{% url 'programs' %}">{% trans 'Programs' %}</a></li>
+        {% endif %}
         <li class="breadcrumb-item active" aria-current="page">{{ program.title }}</li>
     </ol>
 </nav>
diff --git a/templates/quiz/quiz_list.html b/templates/quiz/quiz_list.html
index 1ae5dc7..a36c0d1 100644
--- a/templates/quiz/quiz_list.html
+++ b/templates/quiz/quiz_list.html
@@ -10,7 +10,11 @@
 <nav style="--bs-breadcrumb-divider: '>';" aria-label="breadcrumb">
     <ol class="breadcrumb">
         <li class="breadcrumb-item"><a href="/">{% trans 'Home' %}</a></li>
+        {% if request.user.is_student %}
+        <li class="breadcrumb-item"><a href="{% url 'user_course_list' %}">{% trans 'My courses' %}</a></li>
+        {% else %}
         <li class="breadcrumb-item"><a href="{% url 'programs' %}">{% trans 'Programs' %}</a></li>
+        {% endif %}
         <li class="breadcrumb-item"><a href="{% url 'program_detail' pk=1 %}">{{ course.program }}</a></li>
         <li class="breadcrumb-item"><a href="{{ course.get_absolute_url }}">{{ course }}</a></li>
         <li class="breadcrumb-item active" aria-current="page">{% trans 'Quizzes' %}</li>
diff --git a/templates/result/assessment_results.html b/templates/result/assessment_results.html
index 4440c38..f71e7ab 100644
--- a/templates/result/assessment_results.html
+++ b/templates/result/assessment_results.html
@@ -77,7 +77,7 @@
       <tr>
         <th>#</th>
         <th>{% trans 'Course Title' %}</th>
-        <th>{% trans 'Course Code %}</th>
+        <th>{% trans 'Course Code' %}</th>
         <th>{% trans 'Cr.Hr(s)' %}</th>
         <th>{% trans 'Assignment' %}</th>
         <th>{% trans 'Mid exam' %}</th>