From 7e200b74a7de9a2d582c73a079ba6ef4a26bcfaa Mon Sep 17 00:00:00 2001
From: mikaelGusse <mikael.gustafsson@aalto.fi>
Date: Thu, 15 Aug 2024 14:45:13 +0300
Subject: [PATCH] Fix tooltip update on removal

---
 .../static/exercise/personalized_points_goal.js     |  4 ++--
 exercise/templates/exercise/_points_progress.html   |  2 +-
 exercise/views.py                                   | 13 ++++---------
 3 files changed, 7 insertions(+), 12 deletions(-)

diff --git a/exercise/static/exercise/personalized_points_goal.js b/exercise/static/exercise/personalized_points_goal.js
index fb338be80..4e44d5ae8 100644
--- a/exercise/static/exercise/personalized_points_goal.js
+++ b/exercise/static/exercise/personalized_points_goal.js
@@ -57,7 +57,7 @@ $(document).ready(function() {
                     spanElement = doc.createElement('span');
                     spanElement.className = 'personalized-points-full-text text-nowrap';
                     doc.body.appendChild(spanElement);
-                    spanElement.innerHTML = "<br>" + $pointsGoalForm.data('personalized-points-goal-tooltip-text') + ": " + response.personalized_points_goal_points;
+                    spanElement.innerHTML = "<br>" + $pointsGoalForm.data('personalized-points-goal-tooltip-text') + ": " + "<span class=\"personalized-points-text text-nowrap\">" + response.personalized_points_goal_points + "</span>";
                 }
                 else {
                     spanElement.textContent = response.personalized_points_goal_points;
@@ -127,7 +127,7 @@ $(document).ready(function() {
 
                 // Update progress-bar style
                 $progressDiv.find('.progress-bar').removeClass('progress-bar-primary');
-
+                
                 $('#remove-success-alert').show();
                 setTimeout(function() {
                     $('#remove-success-alert').hide();
diff --git a/exercise/templates/exercise/_points_progress.html b/exercise/templates/exercise/_points_progress.html
index 154f7987e..eaf8a6378 100644
--- a/exercise/templates/exercise/_points_progress.html
+++ b/exercise/templates/exercise/_points_progress.html
@@ -9,7 +9,7 @@
 <div class="progress" data-toggle="tooltip" data-html="true" data-placement="bottom"
     title="{% translate 'POINTS' %}: &lt;span class='text-nowrap'&gt;{{ formatted_points }} / {{ max }}&lt;/span&gt;
         {% if required %} &lt;br&gt; {% translate 'POINTS_TO_PASS' %}: &lt;span class='text-nowrap'&gt;{{ required }}&lt;/span&gt; {% endif %}
-        {% if personalized_points_module_goal %} &lt;br&gt;&lt;span class='personalized-points-full-text text-nowrap'&gt;{% translate 'PERSONALIZED_POINTS_GOAL' %}: &lt;span class='personalized-points-text text-nowrap'&gt;&lt;/span&gt;{{ personalized_points_module_goal_points|floatformat:"0" }}&lt;/span&gt;{% endif %}"
+        {% if personalized_points_module_goal %} &lt;span class='personalized-points-full-text text-nowrap'&gt; &lt;br&gt;{% translate 'PERSONALIZED_POINTS_GOAL' %}: &lt;span class='personalized-points-text text-nowrap'&gt;{{ personalized_points_module_goal_points|floatformat:"0" }}&lt;/span&gt;&lt;/span&gt;{% endif %}"
 >
     <div class="progress-bar progress-bar-striped progress-bar-{% if full_score %}success{% elif percentage >= personalized_points_module_goal %}primary{% elif passed %}warning{% else %}danger{% endif %}"
         rel="progressbar" aria-valuenow="{{ points }}" aria-valuemin="0" aria-valuemax="{{ max }}"
diff --git a/exercise/views.py b/exercise/views.py
index 5117bf3d7..5b52dcb69 100644
--- a/exercise/views.py
+++ b/exercise/views.py
@@ -620,14 +620,14 @@ def post(self, request: HttpRequest, *args: Any, **kwargs: Any) -> JsonResponse:
         module_slug = self.kwargs['module_slug']
         points_goal = request.POST.get('personalized_points_goal_input')
         delete = request.POST.get('delete')
+        student = UserProfile.objects.get(id=user_id)
+        module = CourseModule.objects.get(url=module_slug)
+        cached_points = CachedPoints(module.course_instance, student, True)
+        cached_module, _, _, _ = cached_points.find(module)
 
         if delete:
-            student = UserProfile.objects.get(id=user_id)
-            module = CourseModule.objects.get(url=module_slug)
             try:
                 StudentModuleGoal.objects.get(student=user_id, module=module.id).delete()
-                cached_points = CachedPoints(module.course_instance, student, True)
-                cached_module, _, _, _ = cached_points.find(module)
                 cached_points.invalidate(module.course_instance, student)
                 return JsonResponse({"success": "deleted"}, status=200)
             except Exception as e:
@@ -636,11 +636,6 @@ def post(self, request: HttpRequest, *args: Any, **kwargs: Any) -> JsonResponse:
         if not points_goal.replace('%', '').isdigit() and not points_goal.isdigit():
             return JsonResponse({"error": "not_a_number"}, status=400)
 
-        # Get the StudentModuleGoal object for the student and module
-        student = UserProfile.objects.get(id=user_id)
-        module = CourseModule.objects.get(url=module_slug)
-        cached_points = CachedPoints(module.course_instance, student, True)
-        cached_module, _, _, _ = cached_points.find(module)
         # Points goal can either be an integer or percentage. If the the points goal is given as a percentage,
         # give it to the points goal directly. Otherwise calculate the points goal by taking the integer
         # and calculating how much that is of the module's max points