From f34aa42e8f8f248ca86b743630c69022c09a6ac3 Mon Sep 17 00:00:00 2001
From: Florian Baumann <flo@noqqe.de>
Date: Mon, 26 Jun 2023 10:29:15 +0200
Subject: [PATCH] fix limit issues

---
 src/serra/web.go     | 17 ++++++++++++-----
 templates/index.tmpl | 14 +++++++-------
 2 files changed, 19 insertions(+), 12 deletions(-)

diff --git a/src/serra/web.go b/src/serra/web.go
index 7138768..5f95eaf 100644
--- a/src/serra/web.go
+++ b/src/serra/web.go
@@ -1,6 +1,7 @@
 package serra
 
 import (
+	"fmt"
 	"net/http"
 	"strconv"
 
@@ -49,10 +50,16 @@ func startWeb() error {
 func landingPage(c *gin.Context) {
 	var query Query
 	if c.ShouldBind(&query) == nil {
-		if query.Limit == 0 {
-			query.Limit = 500
+		strLimit := c.DefaultQuery("limit", "500")
+
+		limit, _ := strconv.ParseInt(strLimit, 10, 64)
+		if limit == 0 {
+			limit = 500
 		}
-		cards := Cards("", query.Set, query.Sort, query.Name, "", "", false, false, query.Page, query.Limit)
+
+		fmt.Println(limit, query.Page, query.Limit)
+
+		cards := Cards("", query.Set, query.Sort, query.Name, "", "", false, false, query.Page*int64(limit), limit)
 		numCards := len(Cards("", query.Set, query.Sort, query.Name, "", "", false, false, 0, 0))
 		sets := Sets("release")
 
@@ -65,9 +72,9 @@ func landingPage(c *gin.Context) {
 			"prevPage": query.Page - 1,
 			"page":     query.Page,
 			"nextPage": query.Page + 1,
-			"limit":    query.Limit,
+			"limit":    limit,
 			"numCards": numCards,
-			"numPages": int64(numCards) / query.Limit,
+			"numPages": int64(numCards) / limit,
 		})
 	}
 }
diff --git a/templates/index.tmpl b/templates/index.tmpl
index 55559a2..8867e7f 100644
--- a/templates/index.tmpl
+++ b/templates/index.tmpl
@@ -195,18 +195,18 @@
     <nav class="pagination" role="navigation" aria-label="pagination">
 
       {{ if ge .prevPage 0 }}
-      <a href="/?set={{.query.Set}}&name={{.query.Name}}&sort={{.query.Sort}}&limit={{.query.Limit}}&page={{.prevPage}}" class="pagination-previous">Previous</a>
+      <a href="/?set={{.query.Set}}&name={{.query.Name}}&sort={{.query.Sort}}&limit={{.limit}}&page={{.prevPage}}" class="pagination-previous">Previous</a>
       {{ end }}
 
       {{ if ( le .nextPage .numPages) }}
-      <a href="/?set={{.query.Set}}&name={{.query.Name}}&sort={{.query.Sort}}&limit={{.query.Limit}}&page={{.nextPage}}" class="pagination-next">Next page</a>
+      <a href="/?set={{.query.Set}}&name={{.query.Name}}&sort={{.query.Sort}}&limit={{.limit}}&page={{.nextPage}}" class="pagination-next">Next page</a>
       {{ end }}
 
       <ul class="pagination-list">
 
         {{ if ne .page 0 }}
         <li>
-          <a class="pagination-link" href="/?set={{.query.Set}}&name={{.query.Name}}&sort={{.query.Sort}}&limit={{.query.Limit}}&page=0" aria-label="Goto page 0">0</a>
+          <a class="pagination-link" href="/?set={{.query.Set}}&name={{.query.Name}}&sort={{.query.Sort}}&limit={{.limit}}&page=0" aria-label="Goto page 0">0</a>
         </li>
 
         <li>
@@ -216,16 +216,16 @@
 
         {{ if gt .prevPage 0 }}
         <li>
-          <a href="/?set={{.query.Set}}&name={{.query.Name}}&sort={{.query.Sort}}&limit={{.query.Limit}}&page={{.prevPage}}" class="pagination-link" aria-label="Goto page {{.prevPage}}">{{.prevPage}}</a>
+          <a href="/?set={{.query.Set}}&name={{.query.Name}}&sort={{.query.Sort}}&limit={{.limit}}&page={{.prevPage}}" class="pagination-link" aria-label="Goto page {{.prevPage}}">{{.prevPage}}</a>
         </li>
         {{end}}
         <li>
-          <a class="pagination-link is-current" href="/?set={{.query.Set}}&name={{.query.Name}}&sort={{.query.Sort}}&limit={{.query.Limit}}&page={{.page}}" aria-label="Page {{ .page }}" aria-current="page">{{.page}}</a>
+          <a class="pagination-link is-current" href="/?set={{.query.Set}}&name={{.query.Name}}&sort={{.query.Sort}}&limit={{.limit}}&page={{.page}}" aria-label="Page {{ .page }}" aria-current="page">{{.page}}</a>
         </li>
 
         {{ if and (ne .nextPage .numPages) ( lt .nextPage .numPages) }}
         <li>
-          <a href="/?set={{.query.Set}}&name={{.query.Name}}&sort={{.query.Sort}}&limit={{.query.Limit}}&page={{.nextPage}}" class="pagination-link" aria-label="Goto page {{.nextPage}} ">{{.nextPage}}</a>
+          <a href="/?set={{.query.Set}}&name={{.query.Name}}&sort={{.query.Sort}}&limit={{.limit}}&page={{.nextPage}}" class="pagination-link" aria-label="Goto page {{.nextPage}} ">{{.nextPage}}</a>
         </li>
         {{ end }}
 
@@ -234,7 +234,7 @@
           <span class="pagination-ellipsis">&hellip;</span>
         </li>
         <li>
-          <a class="pagination-link" href="/?set={{.query.Set}}&name={{.query.Name}}&sort={{.query.Sort}}&limit={{.query.Limit}}&page={{.numPages}}" aria-label="Goto page {{.numPages}}">{{.numPages}}</a>
+          <a class="pagination-link" href="/?set={{.query.Set}}&name={{.query.Name}}&sort={{.query.Sort}}&limit={{.limit}}&page={{.numPages}}" aria-label="Goto page {{.numPages}}">{{.numPages}}</a>
         </li>
         {{end}}