From 37c046010308d412310a2047f082e5af227fc223 Mon Sep 17 00:00:00 2001
From: idugan100 <isaacd4444@gmail.com>
Date: Fri, 29 Dec 2023 23:43:52 -0500
Subject: [PATCH 1/3] updating template to have a professor filter

---
 home/templates/course_reviews.html | 23 +++++++++++++++++++----
 1 file changed, 19 insertions(+), 4 deletions(-)

diff --git a/home/templates/course_reviews.html b/home/templates/course_reviews.html
index a0cad0e5..97fcddd6 100644
--- a/home/templates/course_reviews.html
+++ b/home/templates/course_reviews.html
@@ -31,6 +31,19 @@ <h1 class="text-center">Reviews for <strong><a href="{{ course.get_absolute_url
 					<option value="asc">Rating</option>
 				</select>
 			</div>
+
+			<div class="input-group-prepend ml-3">
+				<label class="input-group-text" for="professor-filter" style="border-bottom-left-radius: 0;">Professor Filter:</label>
+			</div>
+			
+			<div class="dropdown">
+				<select class="form-control text-white custom-select" name="professor-filter" id="professor-filter">
+					<option selected value="">All Professors</option>
+					{% for professor in professors %} 
+						<option value={{professor.id}}>{{professor.name}}</option>
+					{% endfor%}
+				  </select>
+			  </div>
 		</div>
 	</div>
 {% endif %}
@@ -49,10 +62,11 @@ <h1 class="text-center">Reviews for <strong><a href="{{ course.get_absolute_url
 		color_2: "secondary",
 	});
 
-	function sortReviews(direction) {
+	function sortReviews(direction, professor) {
 		var data = {
 			"csrfmiddlewaretoken": "{{ csrf_token }}",
 			"direction": direction,
+			"professor": professor,
 			"type": "course",
 			"obj_id": "{{ course.pk }}"
 		}
@@ -67,9 +81,10 @@ <h1 class="text-center">Reviews for <strong><a href="{{ course.get_absolute_url
 		});
 	}
 
-	$("#rating-select").on("change", function() {
-		var selectedRating = $(this).val();
-		sortReviews(selectedRating);
+	$("#rating-select, #professor-filter").on("change", function() {
+		var selectedRating = $("#rating-select").val();
+		var professorId = $("#professor-filter").val();
+		sortReviews(selectedRating, professorId);
 	});
 </script>
 {% endblock %}

From f285af18d082922ae07311786861362924425e2d Mon Sep 17 00:00:00 2001
From: idugan100 <isaacd4444@gmail.com>
Date: Fri, 29 Dec 2023 23:45:11 -0500
Subject: [PATCH 2/3] updating course review logic to give template professor
 names and filter on professor name when appropriate

---
 home/views/basic.py | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/home/views/basic.py b/home/views/basic.py
index 73bb5a85..795f18d4 100644
--- a/home/views/basic.py
+++ b/home/views/basic.py
@@ -63,9 +63,11 @@ def get(self, request, name):
             raise Http404()
 
         reviews = course.review_set(manager="verified").order_by("-created_at")
-
+        professors = { review.professor for review in reviews }
+        
         context = {
             "course": course,
+            "professors": professors,
             "num_reviews": reviews.count(),
             "reviews_table": VerifiedReviewsTable(reviews, request)
         }
@@ -116,7 +118,10 @@ def post(self, request):
         if data_type == "professor":
             reviews = Review.verified.filter(professor__id=obj_id)
         elif data_type == "course":
-            reviews = Review.verified.filter(course__id=obj_id)
+            if(data["professor"]==""):
+                reviews = Review.verified.filter(course__id=obj_id)
+            else:
+                reviews = Review.verified.filter(course__id=obj_id).filter(professor__id=int(data["professor"]))
         else:
             raise ValueError(f"Unknown type: {data_type}")
 

From db817f5b5d0ae67dd128d2ce7ee2f3a81100b3f3 Mon Sep 17 00:00:00 2001
From: idugan100 <isaacd4444@gmail.com>
Date: Fri, 29 Dec 2023 23:46:52 -0500
Subject: [PATCH 3/3] adding styling so professor filter matches rating filter

---
 planetterp/static/css/stylesheet.css | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/planetterp/static/css/stylesheet.css b/planetterp/static/css/stylesheet.css
index dea46ce1..d74f5689 100644
--- a/planetterp/static/css/stylesheet.css
+++ b/planetterp/static/css/stylesheet.css
@@ -224,6 +224,14 @@ body {
 	font-size: 18px;
 }
 
+#professor-filter {
+	background: #6c757d; 
+	border-bottom-left-radius: 0;
+	border-top-left-radius: 0;
+	border-bottom-right-radius: 0; 
+	border:1px solid #6c757d; 
+}
+
 .card-header-gened {
 	text-align: center;
 }