From 98c93947097f2ec1721cf5b31873aa7cbfdf1f1e Mon Sep 17 00:00:00 2001
From: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Date: Thu, 12 Dec 2024 13:11:21 +0000
Subject: [PATCH] Bug 38685: Preparation: DRY code

This is in preparation of the next patch, to avoid big copy + pastes

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
---
 .../prog/en/modules/ill/ill-requests.tt       | 35 ++++++-------------
 1 file changed, 10 insertions(+), 25 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt
index 556659181c8..2299ed243e2 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt
@@ -23,38 +23,23 @@
 
 [% WRAPPER 'sub-header.inc' %]
     [% WRAPPER breadcrumbs %]
-        [% IF op == 'cud-create' %]
-            [% WRAPPER breadcrumb_item %]
+        [% WRAPPER breadcrumb_item bc_active = (op == 'illlist' ? 1 : 0) %]
+            [% IF op == 'illlist' %]
+                <span>ILL requests</span>
+            [% ELSE %]
                 <a href="/cgi-bin/koha/ill/ill-requests.pl">ILL requests</a>
             [% END %]
-            [% WRAPPER breadcrumb_item bc_active= 1 %]
+        [% END %]
+        [% WRAPPER breadcrumb_item bc_active= 1 %]
+            [% IF op == 'cud-create' %]
                 <span>New request</span>
-            [% END %]
-        [% ELSIF op == 'illview' %]
-            [% WRAPPER breadcrumb_item %]
-                <a href="/cgi-bin/koha/ill/ill-requests.pl">ILL requests</a>
-            [% END %]
-            [% WRAPPER breadcrumb_item bc_active= 1 %]
+            [% ELSIF op == 'illview' %]
                 <span>Manage request [% request.id_prefix _ request.illrequest_id | html %]</span>
-            [% END %]
-        [% ELSIF op == 'typedisclaimer' %]
-            [% WRAPPER breadcrumb_item %]
-                <a href="/cgi-bin/koha/ill/ill-requests.pl">ILL requests</a>
-            [% END %]
-            [% WRAPPER breadcrumb_item bc_active= 1 %]
+            [% ELSIF op == 'typedisclaimer' %]
                 <span>Request type disclaimer</span>
-            [% END %]
-        [% ELSIF op == 'confirmautoill' %]
-            [% WRAPPER breadcrumb_item %]
-                <a href="/cgi-bin/koha/ill/ill-requests.pl">ILL requests</a>
-            [% END %]
-            [% WRAPPER breadcrumb_item bc_active= 1 %]
+            [% ELSIF op == 'confirmautoill' %]
                 <span>Confirm automatic request</span>
             [% END %]
-        [% ELSE %]
-            [% WRAPPER breadcrumb_item bc_active= 1 %]
-                <span>ILL requests</span>
-            [% END %]
         [% END %]
     [% END #/ WRAPPER breadcrumbs %]
 [% END #/ WRAPPER sub-header.inc %]
-- 
2.47.1