From a2cd16b97a409e4ce4e3ac5cc68c27306453de4c Mon Sep 17 00:00:00 2001
From: Anton M <anton2008m@gmail.com>
Date: Mon, 16 Sep 2024 21:28:05 +0200
Subject: [PATCH] report detail view fixes

---
 .../templates/validity/compliancereport.html  | 24 +++++++++++++++----
 validity/templatetags/validity.py             | 12 ++++------
 2 files changed, 24 insertions(+), 12 deletions(-)

diff --git a/validity/templates/validity/compliancereport.html b/validity/templates/validity/compliancereport.html
index ff08c1f..32ec518 100644
--- a/validity/templates/validity/compliancereport.html
+++ b/validity/templates/validity/compliancereport.html
@@ -29,7 +29,10 @@ <h5 class="card-header">Compliance Report</h5>
             {% with job=object.jobs.first %}
             <tr>
               <th scope="row">Job</th>
-              <td>{% if job %}{{ job | linkify }} | {{ job | colored_choice:"status" }}{% else %}—{% endif %}</td>
+              <td>
+                {% if job %}<a href="{{ job.get_absolute_url }}">{{ job.pk }}</a>
+                | {{ job | colored_choice:"status" }}{% else %}—{% endif %}
+              </td>
             </tr>
             {% if job.error %}
             <tr>
@@ -46,10 +49,21 @@ <h5 class="card-header">Compliance Report</h5>
               <th scope="row">Unique Tests involved</th>
               <td>{{ object.test_count }}</td>
             </tr>
-            {% report_stats_row object "Overall Results" "total" %}
-            {% report_stats_row object "LOW Severity Results" "low" %}
-            {% report_stats_row object "MIDDLE Severity Results" "middle" %}
-            {% report_stats_row object "HIGH Severity Results" "high" %}
+            <tr>
+              <th>Overall Results</th>
+              <td>{% report_stats object "total" %}</td>
+            </tr>
+          </table>
+          <div class="card-header"></div>
+          <table class="table table-hover">
+            <tr>
+              <th>LOW Severity</th><th>MIDDLE Severity</th><th>HIGH Severity</th>
+            </tr>
+            <tr>
+              <td>{% report_stats object "low" %}</td>
+              <td>{% report_stats object "middle" %}</td>
+              <td>{% report_stats object "high" %}</td>
+            </tr>
           </table>
         </div>
       </div>
diff --git a/validity/templatetags/validity.py b/validity/templatetags/validity.py
index 5edec9e..a1d7f57 100644
--- a/validity/templatetags/validity.py
+++ b/validity/templatetags/validity.py
@@ -73,16 +73,14 @@ def urljoin(*parts: str) -> str:
     return "/".join(url_parts)
 
 
-@register.inclusion_tag("validity/inc/report_stats_row.html")
-def report_stats_row(obj, row_name, severity):
-    for i, row_part in enumerate((row_parts := row_name.split())):
-        if row_part.lower() in {"low", "middle", "high"}:
-            row_parts[i] = f"<b>{row_part.upper()}</b>"
-    row_name = mark_safe(" ".join(row_parts))
+@register.simple_tag
+def report_stats(obj, severity):
     count = getattr(obj, f"{severity}_count")
+    if count == 0:
+        return "—"
     passed = getattr(obj, f"{severity}_passed")
     percentage = getattr(obj, f"{severity}_percentage")
-    return {"row_name": row_name, "passed": passed, "count": count, "percentage": percentage}
+    return mark_safe(f"{passed}/{count} ") + colorful_percentage(percentage)
 
 
 @register.simple_tag