From 106c859e6418ef20ca66d77db2001236d1b89572 Mon Sep 17 00:00:00 2001
From: barnold <barnold@tilde.club>
Date: Sun, 18 Sep 2022 14:46:57 +0100
Subject: [PATCH] Switched to url_with for page navigation...

since it inherits query parameters for me, simplifying things.
Thanks to kraih[m] on #mojo for help.
---
 lib/MyApp/Controller/Book.pm       |  2 --
 templates/_page-navigation.html.ep | 12 ++++--------
 2 files changed, 4 insertions(+), 10 deletions(-)

diff --git a/lib/MyApp/Controller/Book.pm b/lib/MyApp/Controller/Book.pm
index 94301d7..b669070 100644
--- a/lib/MyApp/Controller/Book.pm
+++ b/lib/MyApp/Controller/Book.pm
@@ -10,7 +10,6 @@ sub index ($self) {
    $self->render(
       book_page => $book_page,
       pager => $book_page->pager,
-      query_params => $self->req->query_params,
    );
 }
 
@@ -24,7 +23,6 @@ sub author ($self) {
    $self->render(
       author_page => $author_page,
       pager => $author_page->pager,
-      query_params => $self->req->query_params,
    );
 }
 
diff --git a/templates/_page-navigation.html.ep b/templates/_page-navigation.html.ep
index d6ff7cf..b0a1dff 100644
--- a/templates/_page-navigation.html.ep
+++ b/templates/_page-navigation.html.ep
@@ -2,26 +2,22 @@
 Page <%= $pager->current_page %> of <%= $pager->last_page %>
 
 &nbsp;&nbsp;&nbsp;
-<%= link_to url_for(page_number => 1)
-->query($query_params)
+<%= link_to url_with(page_number => 1)
 ->to_abs => begin %>◄ First<% end %>
 
 % if (my $prev = $pager->previous_page) {
-   <%= link_to url_for(page_number => $prev)
-   ->query($query_params)
+   <%= link_to url_with(page_number => $prev)
    ->to_abs => begin %>◄ Prev<% end %>
 % } else {
 ◄ Prev
 % }
 
 % if (my $next = $pager->next_page) {
-   <%= link_to url_for(page_number => $next)
-   ->query($query_params)
+   <%= link_to url_with(page_number => $next)
    ->to_abs => begin %>Next ►<% end %>
 % } else {
 Next ►
 % }
 
-<%= link_to url_for(page_number => $pager->last_page)
-->query($query_params)
+<%= link_to url_with(page_number => $pager->last_page)
 ->to_abs => begin %>Last ►<% end %>