Add a filter option to view teacher submissions for an assignment

Author: krishnans2006

tin/apps/assignments/views.py

@@ -108,6 +108,15 @@ def show_view(request, assignment_id):
                 )
             else:
                 student_list = course.students.all().order_by("last_name", "first_name")
+        elif period == "all":
+            active_period = "all"
+            student_list = course.students.all().order_by("last_name", "first_name")
+        elif period == "none":
+            active_period = "none"
+            student_list = []
+        elif period == "teachers":
+            active_period = "teachers"
+            student_list = course.teacher.all().order_by("last_name", "first_name")
         elif course.period_set.exists():
             if period == "":
                 if request.user in course.teacher.all():
@@ -120,20 +129,14 @@ def show_view(request, assignment_id):
                 else:
                     period = "none"
 
-            if period == "all":
-                active_period = "all"
-                student_list = course.students.all().order_by("last_name", "first_name")
-            elif period == "none":
+            if period == "none":
                 active_period = "none"
                 student_list = []
             else:
                 active_period = get_object_or_404(
                     Period.objects.filter(course=course), id=int(period)
                 )
                 student_list = active_period.students.all().order_by("last_name", "first_name")
-        elif period == "all":
-            active_period = "all"
-            student_list = course.students.all().order_by("last_name", "first_name")
         else:
             active_period = "none"
             student_list = []

tin/templates/assignments/show.html

@@ -95,6 +95,8 @@ <h2 style="border-top:1px solid lightgray;padding-top:15px;">Filter Submissions<
        href="{% url 'assignments:show' assignment.id %}?period=none">None</a>
     <a class="right tin-btn" {% if active_period == "all" %}style="color:#4fab4f;"{% endif %}
        href="{% url 'assignments:show' assignment.id %}?period=all">All</a>
+    <a class="right tin-btn" {% if active_period == "teachers" %}style="color:#4fab4f;"{% endif %}
+       href="{% url 'assignments:show' assignment.id %}?period=teachers">Teachers</a>
     &ensp;
     {% for period in period_set %}
       <a class="right tin-btn" {% if active_period.id == period.id %}style="color:#4fab4f;"{% endif %}
@@ -130,7 +132,7 @@ <h2 style="border-top:1px solid lightgray;padding-top:15px;">Filter Submissions<
         <table id="submission-list" class="has-border">
           <tr>
             <th style="min-width:20px; text-align: center;">#</th>
-            <th style="min-width:125px">Student</th>
+            <th style="min-width:125px">{% if active_period == "teachers" %}Teacher{% else %}Student{% endif %}</th>
             {% if not active_period.name %}
               <th style="min-width:65px;">Period</th>
             {% endif %}
@@ -172,7 +174,7 @@ <h2 style="border-top:1px solid lightgray;padding-top:15px;">Filter Submissions<
         <table id="submission-list" class="has-border">
           <tr>
             <th style="min-width:20px; text-align:center;">#</th>
-            <th style="min-width:125px">Student</th>
+            <th style="min-width:125px">{% if active_period == "teachers" %}Teacher{% else %}Student{% endif %}</th>
             {% if not active_period.name %}
               <th style="min-width:65px;">Period</th>
             {% endif %}