From 2478de6b98f731ed1caa4786c07471de50154133 Mon Sep 17 00:00:00 2001
From: Ross <xkjq@ymail.com>
Date: Wed, 29 Apr 2026 22:37:13 +0100
Subject: [PATCH] feat: Enhance logs view with support for displaying all
 entries and improved pagination

Co-authored-by: Copilot <copilot@github.com>
---
 rad/views.py               | 12 +++++++--
 templates/logs_viewer.html | 52 ++++++++++++++++++--------------------
 2 files changed, 34 insertions(+), 30 deletions(-)

diff --git a/rad/views.py b/rad/views.py
index 2b2a3f89..6b1bd456 100644
--- a/rad/views.py
+++ b/rad/views.py
@@ -243,10 +243,12 @@ def logs_view(request):
     
     Scope: superusers only.
     Functionality: reads log.txt, displays entries with optional text search filter.
+    Supports ?all=1 to show all entries without pagination limit.
     """
     log_file_path = os.path.join(settings.BASE_DIR, "log.txt")
     entries = []
     search_query = request.GET.get("q", "").strip()
+    show_all = request.GET.get("all") == "1"
     
     if os.path.exists(log_file_path):
         try:
@@ -267,14 +269,20 @@ def logs_view(request):
     else:
         entries = [f"Log file not found at {log_file_path}"]
     
+    total = len(entries)
+    limit = None if show_all else 500
+    displayed_entries = entries[:limit] if limit else entries
+
     return render(
         request,
         "logs_viewer.html",
         {
-            "entries": entries[:500],
-            "total_entries": len(entries),
+            "entries": displayed_entries,
+            "total_entries": total,
             "search_query": search_query,
             "log_file_path": log_file_path,
+            "show_all": show_all,
+            "limit": limit,
         },
     )
 
diff --git a/templates/logs_viewer.html b/templates/logs_viewer.html
index 15c257d9..50ac74cb 100644
--- a/templates/logs_viewer.html
+++ b/templates/logs_viewer.html
@@ -11,7 +11,11 @@
             </div>
             <div class="text-end">
                 <span class="badge bg-secondary rounded-pill px-3 py-2">
+                    {% if show_all %}
+                        Viewing all {{ total_entries }} entries
+                    {% else %}
                     Showing {{ entries|length }}/{{ total_entries }} entries
+                    {% endif %}
                 </span>
             </div>
         </div>
@@ -19,7 +23,7 @@
         <div class="card border-0 shadow-sm mb-4">
             <div class="card-body">
                 <form method="get" class="row g-2">
-                    <div class="col-12 col-md-8">
+                    <div class="col-12 col-md-6">
                         <input type="search"
                                name="q"
                                class="form-control"
@@ -32,44 +36,36 @@
                             <a href="{% url 'logs_view' %}" class="btn btn-outline-secondary ms-1">Clear</a>
                         {% endif %}
                     </div>
+                    <div class="col-12 col-md-auto">
+                        {% if show_all %}
+                            <a href="{% url 'logs_view' %}?q={{ search_query }}" class="btn btn-outline-secondary">Show recent only</a>
+                        {% else %}
+                            <a href="{% url 'logs_view' %}?q={{ search_query }}&all=1" class="btn btn-outline-info">View all logs</a>
+                        {% endif %}
+                    </div>
                 </form>
             </div>
         </div>
 
         {% if entries %}
             <div class="card border-0 shadow-sm">
-                <div class="card-body p-0">
-                    <div style="max-height: 70vh; overflow-y: auto;">
-                        <table class="table table-sm table-striped mb-0" style="font-family: monospace; font-size: 0.875rem;">
-                            <tbody>
-                                {% for entry in entries %}
-                                    <tr>
-                                        <td class="text-muted align-top" style="width: 30px; white-space: nowrap;">
-                                            {{ forloop.counter }}
-                                        </td>
-                                        <td style="word-break: break-all; white-space: pre-wrap;">
-                                            {% if 'ERROR' in entry %}
-                                                <span class="badge bg-danger me-1">ERROR</span>
-                                            {% elif 'WARNING' in entry or 'WARN' in entry %}
-                                                <span class="badge bg-warning me-1">WARN</span>
-                                            {% elif 'DEBUG' in entry %}
-                                                <span class="badge bg-secondary me-1">DEBUG</span>
-                                            {% elif 'INFO' in entry %}
-                                                <span class="badge bg-info me-1">INFO</span>
-                                            {% endif %}
-                                            {{ entry }}
-                                        </td>
-                                    </tr>
-                                {% endfor %}
-                            </tbody>
-                        </table>
-                    </div>
+                <div class="card-body p-0" style="max-height: 75vh; overflow-y: auto;">
+                    <pre style="margin: 0; padding: 1rem; font-size: 0.85rem; line-height: 1.5;">{% for entry in entries %}
+{{ entry }}
+{% endfor %}</pre>
                 </div>
             </div>
 
             <div class="alert alert-secondary mt-3 small">
                 <i class="bi bi-info-circle"></i>
-                Showing most recent entries first. Filtered display limited to 500 entries. 
+                {% if show_all %}
+                    Displaying all {{ total_entries }} log entries (newest first).
+                {% else %}
+                    Displaying most recent 500 entries (newest first).
+                    {% if total_entries > 500 %}
+                        <a href="{% url 'logs_view' %}?q={{ search_query }}&all=1" class="alert-link">View all {{ total_entries }} entries</a>.
+                    {% endif %}
+                {% endif %}
                 {% if search_query %}
                     Search found {{ entries|length }} matching entr{{ entries|length|pluralize:"y,ies" }}.
                 {% endif %}