From e475273cd33c9e36f3dad3a5219a5f5619feabcd Mon Sep 17 00:00:00 2001
From: Bartuccio Antoine <klmp200@klmp200.net>
Date: Fri, 5 Jul 2019 12:31:49 +0200
Subject: [PATCH] pedagogy: enhance StarList widget

---
 pedagogy/forms.py                           | 20 ++++++++++----
 pedagogy/templates/pedagogy/starlist.jinja  | 29 ++++++++++++++-------
 pedagogy/templates/pedagogy/uv_detail.jinja | 22 ++++++++--------
 3 files changed, 45 insertions(+), 26 deletions(-)

diff --git a/pedagogy/forms.py b/pedagogy/forms.py
index 00984cf7..98a90438 100644
--- a/pedagogy/forms.py
+++ b/pedagogy/forms.py
@@ -77,6 +77,16 @@ class UVForm(forms.ModelForm):
 class StarList(forms.NumberInput):
     template_name = "pedagogy/starlist.jinja"
 
+    def __init__(self, nubmer_of_stars=0):
+        super(StarList, self).__init__(None)
+        self.number_of_stars = nubmer_of_stars
+
+    def get_context(self, name, value, attrs):
+        context = super(StarList, self).get_context(name, value, attrs)
+        context["number_of_stars"] = range(0, self.number_of_stars)
+        context["translations"] = {"do_not_vote": _("Do not vote")}
+        return context
+
 
 class UVCommentForm(forms.ModelForm):
     """
@@ -99,11 +109,11 @@ class UVCommentForm(forms.ModelForm):
             "comment": MarkdownInput,
             "author": forms.HiddenInput,
             "uv": forms.HiddenInput,
-            "grade_global": StarList,
-            "grade_utility": StarList,
-            "grade_interest": StarList,
-            "grade_teaching": StarList,
-            "grade_work_load": StarList,
+            "grade_global": StarList(5),
+            "grade_utility": StarList(5),
+            "grade_interest": StarList(5),
+            "grade_teaching": StarList(5),
+            "grade_work_load": StarList(5),
         }
 
     def __init__(self, author_id, uv_id, *args, **kwargs):
diff --git a/pedagogy/templates/pedagogy/starlist.jinja b/pedagogy/templates/pedagogy/starlist.jinja
index 53dc4290..e8f7db47 100644
--- a/pedagogy/templates/pedagogy/starlist.jinja
+++ b/pedagogy/templates/pedagogy/starlist.jinja
@@ -1,29 +1,38 @@
 <div>
-    {# <input name="{{ widget.name }}" id="{{ widget.name }}" hidden type="number" {% include "django/forms/widgets/attrs.html" %}></input> #}
     <style>
-        .checked{
+        .checked {
             color : orange;
         }
-        .unchecked{
+        .unchecked {
             color : gray;
         }
-        .star input[type="radio"]{
+        .star input[type="radio"] {
             display : none;
         }
-        .star{
+        .star {
             display: inline;
         }
-        
+
     </style>
 
-    {% for i in i|rjust:5 %}
+    <label class="star">
+        <input type="radio" name="{{ widget.name }}" value="-1" onclick='
+            var stars = document.getElementsByClassName("{{ widget.name }}");
+            for (var i = 0; i < stars.length; i++){
+                var attrs = stars[i].getAttribute("class");
+                attrs = attrs.replace("unchecked", "");
+                attrs = attrs.replace("checked", "");
+                stars[i].setAttribute("class", attrs + " unchecked");
+            }
+        ' checked>
+        <span class="fa fa-times-circle"> {{ translations.do_not_vote }}</span>
+    </label>
+    {% for i in number_of_stars %}
     <label class="star">
         <input type="radio" name="{{ widget.name }}" value="{{ forloop.counter0 }}" onclick='
             var stars = document.getElementsByClassName("{{ widget.name }}");
-            // console.log(Array.from(this.parentNode.parentNode.children).filter(el => el.className == "star"));
 
-            for (var i = 0; i < 5; i++){
-                console.log(i);
+            for (var i = 0; i < stars.length; i++){
                 var attrs = stars[i].getAttribute("class");
                 attrs = attrs.replace("unchecked", "");
                 attrs = attrs.replace("checked", "");
diff --git a/pedagogy/templates/pedagogy/uv_detail.jinja b/pedagogy/templates/pedagogy/uv_detail.jinja
index 56bb7385..8d65c1a2 100644
--- a/pedagogy/templates/pedagogy/uv_detail.jinja
+++ b/pedagogy/templates/pedagogy/uv_detail.jinja
@@ -1,5 +1,5 @@
 {% extends "core/base.jinja" %}
-{% import "pedagogy/macros.jinja" as macros%}
+{% from "pedagogy/macros.jinja" import display_star %}
 
 {% block title %}
 {% trans %}UV Details{% endtrans %}
@@ -15,20 +15,20 @@
 	<p>{{ object.key_concepts|markdown }}</p>
 	<p>{% trans %}UV manager: {% endtrans %}{{ object.manager }}</p>
 
-	<p>{{ object.grade_global_average }}</p>
-	<p>{{ object.grade_utility_average }}</p>
-	<p>{{ object.grade_interest_average }}</p>
-	<p>{{ object.grade_teaching_average }}</p>
-	<p>{{ object.grade_work_load_average }}</p>
+	<p>{{ display_star(object.grade_global_average) }}</p>
+	<p>{{ display_star(object.grade_utility_average) }}</p>
+	<p>{{ display_star(object.grade_interest_average) }}</p>
+	<p>{{ display_star(object.grade_teaching_average) }}</p>
+	<p>{{ display_star(object.grade_work_load_average) }}</p>
 
 	{% if object.comments.exists() %}
 		<h2>{% trans %}Comments{% endtrans %}</h2>
 		{% for comment in object.comments.all() %}
-			<p>{{ macros.display_star(comment.grade_global) }}</p>
-			<p>{{ macros.display_star(comment.grade_utility) }}</p>
-			<p>{{ macros.display_star(comment.grade_interest) }}</p>
-			<p>{{ macros.display_star(comment.grade_teaching) }}</p>
-			<p>{{ macros.display_star(comment.grade_work_load) }}</p>
+			<p>{{ display_star(comment.grade_global) }}</p>
+			<p>{{ display_star(comment.grade_utility) }}</p>
+			<p>{{ display_star(comment.grade_interest) }}</p>
+			<p>{{ display_star(comment.grade_teaching) }}</p>
+			<p>{{ display_star(comment.grade_work_load) }}</p>
 			<p>{{ comment.comment|markdown }}</p>
 			<p>{% trans %}Published: {% endtrans %}{{ comment.publish_date }}</p>
 			<p>{% trans %}Author: {% endtrans %}{{ comment.author }}</p>