From 3aafb1cd7e52309d3cac05c2c52e291d127bbec2 Mon Sep 17 00:00:00 2001
From: video-prize-ranch <cb.8a3w5@simplelogin.co>
Date: Tue, 29 Aug 2023 19:08:59 -0400
Subject: [PATCH] Cleanup previous/next page buttons

---
 pages/search.go    | 8 +-------
 pages/tag.go       | 6 ------
 pages/trending.go  | 6 ------
 views/search.hbs   | 6 +++---
 views/tag.hbs      | 7 ++++---
 views/trending.hbs | 4 ++--
 6 files changed, 10 insertions(+), 27 deletions(-)

diff --git a/pages/search.go b/pages/search.go
index ac9cbf9..808144d 100644
--- a/pages/search.go
+++ b/pages/search.go
@@ -29,11 +29,6 @@ func HandleSearch(c *fiber.Ctx) error {
 		pageNumber = 0
 	}
 
-	displayPrevPage := true
-	if page == "0" {
-		displayPrevPage = false
-	}
-
 	results, err := ApiClient.Search(query, page)
 	if err != nil {
 		return err
@@ -42,8 +37,7 @@ func HandleSearch(c *fiber.Ctx) error {
 	return c.Render("search", fiber.Map{
 		"query":			 query,
 		"results":     results,
-		"page":        pageNumber + 1,
-		"displayPrev": displayPrevPage,
+		"page":        pageNumber,
 		"nextPage":    pageNumber + 1,
 		"prevPage":    pageNumber - 1,
 	})
diff --git a/pages/tag.go b/pages/tag.go
index 8682e46..de086ec 100644
--- a/pages/tag.go
+++ b/pages/tag.go
@@ -23,11 +23,6 @@ func HandleTag(c *fiber.Ctx) error {
 		pageNumber = 0
 	}
 
-	displayPrevPage := true
-	if page == "1" {
-		displayPrevPage = false
-	}
-
 	tag, err := ApiClient.FetchTag(c.Params("tag"), c.Query("sort"), page)
 	if err != nil && err.Error() == "ratelimited by imgur" {
 		return c.Status(429).Render("errors/429", fiber.Map{
@@ -44,7 +39,6 @@ func HandleTag(c *fiber.Ctx) error {
 	return c.Render("tag", fiber.Map{
 		"tag":         tag,
 		"page":        page,
-		"displayPrev": displayPrevPage,
 		"nextPage":    pageNumber + 1,
 		"prevPage":    pageNumber - 1,
 	})
diff --git a/pages/trending.go b/pages/trending.go
index 89dd9ab..4225a61 100644
--- a/pages/trending.go
+++ b/pages/trending.go
@@ -36,11 +36,6 @@ func HandleTrending(c *fiber.Ctx) error {
 		sort = "popular"
 	}
 
-	displayPrevPage := true
-	if page == "1" {
-		displayPrevPage = false
-	}
-
 	results, err := ApiClient.FetchTrending(section, sort, page)
 	if err != nil {
 		return err
@@ -51,7 +46,6 @@ func HandleTrending(c *fiber.Ctx) error {
 		"section":     section,
 		"sort":        sort,
 		"page":        pageNumber,
-		"displayPrev": displayPrevPage,
 		"nextPage":    pageNumber + 1,
 		"prevPage":    pageNumber - 1,
 	})
diff --git a/views/search.hbs b/views/search.hbs
index 57ba781..0e516d1 100644
--- a/views/search.hbs
+++ b/views/search.hbs
@@ -22,10 +22,10 @@
     </div>
 
     <div class="flex justify-between mt-4 font-bold">
-      {{#if displayPrev}}
+      {{#noteq page "0"}}
       <a href="/search?q={{query}}&page={{prevPage}}">Previous page</a>
-      {{/if}}
-      <p>Page {{page}}</p>
+      {{/noteq}}
+      <p>Page {{nextPage}}</p>
       <a href="/search?q={{query}}&page={{nextPage}}">Next page</a>
     </div>
   </main>
diff --git a/views/tag.hbs b/views/tag.hbs
index f844169..3d88e2f 100644
--- a/views/tag.hbs
+++ b/views/tag.hbs
@@ -45,10 +45,11 @@
       {{/each}}
     </div>
 
-    <div class="mt-4 font-bold">
-      {{#if displayPrev}}
+    <div class="flex justify-between mt-4 font-bold">
+      {{#noteq page "1"}}
       <a href="{{channel.RelUrl}}?page={{prevPage}}">Previous page</a>
-      {{/if}}
+      {{/noteq}}
+      <p>Page {{nextPage}}</p>
       <a href="{{channel.RelUrl}}?page={{nextPage}}">Next page</a>
     </div>
   </main>
diff --git a/views/trending.hbs b/views/trending.hbs
index f460175..6659c92 100644
--- a/views/trending.hbs
+++ b/views/trending.hbs
@@ -65,9 +65,9 @@
     </div>
 
     <div class="flex justify-between mt-4 font-bold">
-      {{#if displayPrev}}
+      {{#noteq page "1"}}
       <a href="/trending?section={{section}}&sort={{sort}}&page={{prevPage}}">Previous page</a>
-      {{/if}}
+      {{/noteq}}
       <p>Page {{page}}</p>
       <a href="/trending?section={{section}}&sort={{sort}}&page={{nextPage}}">Next page</a>
     </div>