From 0a6cc8912c6a40ec81644c365dfe105771047864 Mon Sep 17 00:00:00 2001
From: Mike Cantelon <mcantelon@gmail.com>
Date: Wed, 27 Sep 2023 00:54:41 -0700
Subject: [PATCH] Use server-side paging in aips page. (#209)

To avoid slow page responses when there are a large amount of AIPs use
server-side paging to limit the number of AIP HTML sent to the browser.
---
 AIPscan/Reporter/helpers.py          | 10 +++++++++
 AIPscan/Reporter/templates/aips.html | 31 ++++++++++++++++++++++++----
 AIPscan/Reporter/views.py            | 19 +++++++++++++----
 3 files changed, 52 insertions(+), 8 deletions(-)

diff --git a/AIPscan/Reporter/helpers.py b/AIPscan/Reporter/helpers.py
index 17c38048..4fa1bb36 100644
--- a/AIPscan/Reporter/helpers.py
+++ b/AIPscan/Reporter/helpers.py
@@ -145,3 +145,13 @@ def get_premis_xml_lines(file_object):
         premis_xml_lines = file_object.premis_object.split("\n")
 
     return premis_xml_lines
+
+
+def calculate_paging_window(page, pagination):
+    first_item = ((page - 1) * pagination.per_page) + 1
+    last_item = page * pagination.per_page
+
+    if last_item > pagination.total:
+        last_item = pagination.total
+
+    return first_item, last_item
diff --git a/AIPscan/Reporter/templates/aips.html b/AIPscan/Reporter/templates/aips.html
index c456f527..45f05689 100644
--- a/AIPscan/Reporter/templates/aips.html
+++ b/AIPscan/Reporter/templates/aips.html
@@ -2,7 +2,7 @@
 
 {% block content %}
 
-<div class="alert alert-header"><strong>AIPs ({{ aips|length }})</strong></div>
+<div class="alert alert-header"><strong>AIPs ({{ pagination.items|length }})</strong></div>
 
 {% if storage_services %}
 
@@ -42,9 +42,9 @@
     </div>
   </div>
 
-  {% if aips %}
+  {% if pagination.items %}
 
-    <table id="aiptable" class="table table-striped table-bordered">
+    <table id="aipstable" class="table table-striped table-bordered">
       <thead>
       <tr>
       <th><strong>Transfer name</strong></th>
@@ -56,7 +56,7 @@
       </tr>
       </thead>
       {% if aips %}
-        {% for aip in aips %}
+        {% for aip in pagination.items %}
           <tr>
             <td>{{ aip.transfer_name }}</td>
             <td>{{ aip.uuid }}</td>
@@ -85,6 +85,29 @@
 
 {% endif %}
 
+{% if pagination.total %}
+
+  <div class="dataTables_wrapper">
+    <div class="row">
+      <div class="col-sm-12 col-md-5">
+        <div class="dataTables_info" id="aiptable_info" role="status" aria-live="polite">Showing {{ first_item }} to {{ last_item }} of {{ pagination.total }} entries</div>
+      </div>
+      <div class="col-sm-12 col-md-7">
+        <div class="dataTables_paginate paging_full_numbers" id="aiptable_paginate">
+          <ul class="pagination">
+            <li class="paginate_button page-item first"><a href="{{ url_for('reporter.view_aips') }}?page=1" class="page-link">First</a></li>
+            <li class="paginate_button page-item previous"><a href="{{ url_for('reporter.view_aips') }}?page={{ pagination.prev_num }}" class="page-link {% if pagination.prev_num is none %}disabled{% endif %}">Previous</a></li>
+            <li class="paginate_button page-item active"><a href="#" class="page-link">{{ pagination.page }}</a></li>
+            <li class="paginate_button page-item next {% if pagination.next_num is none %}disabled{% endif %}"><a href="{{ url_for('reporter.view_aips') }}?page={{ pagination.next_num }}" class="page-link">Next</a></li>
+            <li class="paginate_button page-item last"><a href="{{ url_for('reporter.view_aips') }}?page={{ pagination.pages }}" class="page-link">Last</a></li>
+          </ul>
+        </div>
+      </div>
+    </div>
+  </div>
+
+{% endif %}
+
 <script>
 $(document).ready(function(){
   var storageServiceId = $('#ss').val();
diff --git a/AIPscan/Reporter/views.py b/AIPscan/Reporter/views.py
index 66b4273d..009f03ea 100644
--- a/AIPscan/Reporter/views.py
+++ b/AIPscan/Reporter/views.py
@@ -39,7 +39,7 @@
     request_params,
     sort_puids,
 )
-from AIPscan.Reporter.helpers import get_premis_xml_lines
+from AIPscan.Reporter.helpers import calculate_paging_window, get_premis_xml_lines
 
 
 def _get_storage_service(storage_service_id):
@@ -85,10 +85,18 @@ def view_aips():
     except Exception as e:
         print(e)
 
-    aips = AIP.query.filter_by(storage_service_id=storage_service.id)
+    page = int(request.args.get("page", default="1"))
+    per_page = 5
+
+    pagination = AIP.query.paginate(page=page, per_page=per_page, error_out=False)
+
     if storage_location:
-        aips = aips.filter_by(storage_location_id=storage_location_id)
-    aips = aips.all()
+        pagination = AIP.query.filter_by(
+            storage_location_id=storage_location_id
+        ).paginate(page=page, per_page=per_page, error_out=False)
+    aips = pagination.items
+
+    first_item, last_item = calculate_paging_window(page, pagination)
 
     return render_template(
         "aips.html",
@@ -99,6 +107,9 @@ def view_aips():
         ),
         storage_location=storage_location,
         aips=aips,
+        pagination=pagination,
+        first_item=first_item,
+        last_item=last_item,
     )