From 5a03fd1334aee342ac7a9ebd7fc6ca91824afc57 Mon Sep 17 00:00:00 2001
From: Anton Khorev <tony29@yandex.ru>
Date: Wed, 17 Apr 2024 01:24:37 +0300
Subject: [PATCH] Introduce pagination item helper

---
 app/helpers/pagination_helper.rb      | 17 +++++++++++++++++
 app/views/shared/_pagination.html.erb | 24 ++----------------------
 2 files changed, 19 insertions(+), 22 deletions(-)
 create mode 100644 app/helpers/pagination_helper.rb

diff --git a/app/helpers/pagination_helper.rb b/app/helpers/pagination_helper.rb
new file mode 100644
index 000000000..b99a2c864
--- /dev/null
+++ b/app/helpers/pagination_helper.rb
@@ -0,0 +1,17 @@
+module PaginationHelper
+  def pagination_item(params, &)
+    link_class = "page-link icon-link text-center"
+    page_link_content = capture(&)
+    if params
+      page_link = link_to page_link_content,
+                          params,
+                          :class => link_class,
+                          :data => { "turbo" => true, "turbo-frame" => "pagination", "turbo-action" => "advance" }
+      tag.li page_link, :class => "page-item d-flex"
+    else
+      page_link = tag.span page_link_content,
+                           :class => link_class
+      tag.li page_link, :class => "page-item d-flex disabled"
+    end
+  end
+end
diff --git a/app/views/shared/_pagination.html.erb b/app/views/shared/_pagination.html.erb
index 547078c51..7808f3d35 100644
--- a/app/views/shared/_pagination.html.erb
+++ b/app/views/shared/_pagination.html.erb
@@ -2,35 +2,15 @@
 
 <% translation_scope ||= "shared.pagination.#{controller.controller_name}" %>
 <nav>
-  <% link_class = "page-link icon-link text-center" %>
   <ul class="pagination">
-    <% newer_link_content = capture do %>
+    <%= pagination_item(newer_id && @params.merge(:before => nil, :after => newer_id)) do %>
       <%= previous_page_svg_tag :class => "flex-shrink-0 d-none d-sm-block" %>
       <%= t :newer, :scope => translation_scope %>
     <% end %>
-    <% if newer_id -%>
-      <li class="page-item d-flex">
-        <%= link_to newer_link_content, @params.merge(:before => nil, :after => newer_id), :class => link_class, :data => { "turbo" => true, "turbo-frame" => "pagination", "turbo-action" => "advance" } %>
-      </li>
-    <% else -%>
-      <li class="page-item d-flex disabled">
-        <%= tag.span newer_link_content, :class => link_class %>
-      </li>
-    <% end -%>
-
-    <% older_link_content = capture do %>
+    <%= pagination_item(older_id && @params.merge(:before => older_id, :after => nil)) do %>
       <%= t :older, :scope => translation_scope %>
       <%= next_page_svg_tag :class => "flex-shrink-0 d-none d-sm-block" %>
     <% end %>
-    <% if older_id -%>
-      <li class="page-item d-flex">
-        <%= link_to older_link_content, @params.merge(:before => older_id, :after => nil), :class => link_class, :data => { "turbo" => true, "turbo-frame" => "pagination", "turbo-action" => "advance" } %>
-      </li>
-    <% else -%>
-      <li class="page-item d-flex disabled">
-        <%= tag.span older_link_content, :class => link_class %>
-      </li>
-    <% end -%>
   </ul>
 </nav>
 
-- 
2.39.5