Add search functionality in assignment view

This reduces database queries when teachers are specifically looking for one student (a surprisingly common occurrence)

Author: krishnans2006

tin/apps/assignments/views.py

@@ -78,10 +78,17 @@ def show_view(request, assignment_id):
         teacher_last_login = request.user.last_login
         time_24_hours_ago = now() - datetime.timedelta(days=1)
 
+        query = request.GET.get("query", "")
+
         period = request.GET.get("period", "")
         period_set = course.period_set.order_by("teacher", "name")
 
-        if course.period_set.exists():
+        if query:
+            active_period = "query"
+            student_list = course.students.filter(full_name__icontains=query).order_by(
+                "periods", "last_name"
+            )
+        elif course.period_set.exists():
             if period == "":
                 if request.user in course.teacher.all():
                     try:
@@ -157,6 +164,7 @@ def show_view(request, assignment_id):
             ),
             "is_student": course.is_student_in_course(request.user),
             "is_teacher": request.user in course.teacher.all(),
+            "query": query,
             "period_set": period_set,
             "active_period": active_period,
             "quiz_accessible": quiz_accessible,

tin/static/css/base.css

@@ -216,7 +216,7 @@ a:hover {
     text-align: center;
 }
 
-a.tin-btn {
+.tin-btn {
     border: 1px solid #cfcfcf;
 
     border-radius: 4px;

tin/templates/assignments/show.html

@@ -96,6 +96,16 @@ <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 !important"{% endif %}
        href="{% url 'assignments:show' assignment.id %}?period=all">All</a>
+    &ensp;
+    <form method="GET" action="{% url 'assignments:show' assignment.id %}" style="display: inline-block;">
+      <label>
+        <b>Matches:</b>
+        <input type="text" name="query" value="{{ query }}" placeholder="part of a name">
+      </label>
+      <button type="submit" class="tin-btn" style="cursor: pointer;{% if active_period == "query" %}color:#4fab4f !important;{% endif %}">
+        <i class="fa fa-search"></i>
+      </button>
+    </form>
     <br><br>
     {% for period in period_set %}
       <a class="right tin-btn" {% if active_period.id == period.id %}style="color:#4fab4f !important"{% endif %}