From eff797d21bcf5f858c1345f63667c212de6291f1 Mon Sep 17 00:00:00 2001
From: leigh-mil <leigh-ctr@friends.dds.mil>
Date: Wed, 9 Jan 2019 11:07:34 -0500
Subject: [PATCH] Refactor pagination macro

---
 templates/components/pagination.html | 29 ++++++++++++++--------------
 1 file changed, 14 insertions(+), 15 deletions(-)

diff --git a/templates/components/pagination.html b/templates/components/pagination.html
index 6874d565..d7674d66 100644
--- a/templates/components/pagination.html
+++ b/templates/components/pagination.html
@@ -18,37 +18,36 @@
   <div class="pagination">
 
     {% if pagination.page == 1 %}
-      {% set max_page = [pagination.pages, 5] | min %}
       {{ Page(pagination, route, 1, label="first", disabled=True, workspace_id=workspace_id) }}
       {{ Page(pagination, route, pagination.page - 1, label="prev", disabled=True, workspace_id=workspace_id) }}
+    {% else %}
+      {{ Page(pagination, route, 1, label="first", workspace_id=workspace_id) }}
+      {{ Page(pagination, route, pagination.page - 1, label="prev", workspace_id=workspace_id) }}
+    {% endif %}
+
+    {% if pagination.page == 1 %}
+      {% set max_page = [pagination.pages, 5] | min %}
       {% for i in range(1, max_page + 1) %}
         {{ Page(pagination, route, i, workspace_id=workspace_id) }}
       {% endfor %}
-      {% if pagination.per_page >= pagination.total %}
-        {{ Page(pagination, route, pagination.page + 1, label="next", disabled=True, workspace_id=workspace_id) }}
-        {{ Page(pagination, route, pagination.pages, label="last", disabled=True, workspace_id=workspace_id) }}
-      {% else %}
-        {{ Page(pagination, route, pagination.page + 1, label="next", workspace_id=workspace_id) }}
-        {{ Page(pagination, route, pagination.pages, label="last", workspace_id=workspace_id) }}
-      {% endif %}
-
     {% elif pagination.page == pagination.pages %}
-      {{ Page(pagination, route, 1, label="first", workspace_id=workspace_id) }}
-      {{ Page(pagination, route, pagination.page - 1, label="prev", workspace_id=workspace_id) }}
       {% for i in range(pagination.pages - 4, pagination.pages + 1) %}
         {{ Page(pagination, route, i, workspace_id=workspace_id) }}
       {% endfor %}
-      {{ Page(pagination, route, pagination.page + 1, label="next", disabled=True, workspace_id=workspace_id) }}
-      {{ Page(pagination, route, pagination.pages, label="last", disabled=True, workspace_id=workspace_id) }}
     {% else %}
       {% set window = pagination | pageWindow %}
-      {{ Page(pagination, route, 1, label="first", workspace_id=workspace_id) }}
-      {{ Page(pagination, route, pagination.page - 1, label="prev", workspace_id=workspace_id) }}
       {% for i in range(window.0, window.1 + 1) %}
         {{ Page(pagination, route, i, workspace_id=workspace_id) }}
       {% endfor %}
+    {% endif %}
+
+    {% if pagination.page == pagination.pages %}
+      {{ Page(pagination, route, pagination.page + 1, label="next", disabled=True, workspace_id=workspace_id) }}
+      {{ Page(pagination, route, pagination.pages, label="last", disabled=True, workspace_id=workspace_id) }}
+    {% else %}
       {{ Page(pagination, route, pagination.page + 1, label="next", workspace_id=workspace_id) }}
       {{ Page(pagination, route, pagination.pages, label="last", workspace_id=workspace_id) }}
     {% endif %}
+
   </div>
 {%- endmacro %}