From cdd1e04550bef7794c7498d71f65f986cf1aa666 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Javi=20Mart=C3=ADn?= <javim@elretirao.net>
Date: Sun, 27 Jun 2021 17:01:18 +0200
Subject: [PATCH] Extract methods in order links component

---
 .../shared/order_links_component.html.erb     |  6 ++---
 .../shared/order_links_component.rb           | 22 +++++++++++++++++++
 2 files changed, 24 insertions(+), 4 deletions(-)

diff --git a/app/components/shared/order_links_component.html.erb b/app/components/shared/order_links_component.html.erb
index 2c7a36695..d0cdb431d 100644
--- a/app/components/shared/order_links_component.html.erb
+++ b/app/components/shared/order_links_component.html.erb
@@ -1,10 +1,8 @@
 <ul class="no-bullet submenu order-links">
   <% valid_orders.each do |order| %>
     <li class="inline-block">
-      <%= link_to current_path_with_query_params(order: order, page: 1), class: order == current_order ? "is-active" : "" do %>
-        <%= content_tag(order == current_order ? :h2 : :span) do %>
-          <%= t("#{i18n_namespace}.orders.#{order}") %>
-        <% end %>
+      <%= link_to link_path(order), class: html_class(order) do %>
+        <%= content_tag tag_name(order), link_text(order) %>
       <% end %>
     </li>
   <% end %>
diff --git a/app/components/shared/order_links_component.rb b/app/components/shared/order_links_component.rb
index 9849214fd..7baeb20ff 100644
--- a/app/components/shared/order_links_component.rb
+++ b/app/components/shared/order_links_component.rb
@@ -5,4 +5,26 @@ class Shared::OrderLinksComponent < ApplicationComponent
   def initialize(i18n_namespace)
     @i18n_namespace = i18n_namespace
   end
+
+  private
+
+    def html_class(order)
+      "is-active" if order == current_order
+    end
+
+    def tag_name(order)
+      if order == current_order
+        :h2
+      else
+        :span
+      end
+    end
+
+    def link_path(order)
+      current_path_with_query_params(order: order, page: 1)
+    end
+
+    def link_text(order)
+      t("#{i18n_namespace}.orders.#{order}")
+    end
 end