From 858134df9f75c4693464a6907877ee00f6dfd8af Mon Sep 17 00:00:00 2001
From: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Date: Thu, 12 Dec 2024 13:12:03 +0000
Subject: [PATCH] Bug 38685: Add missing ops to breadcrumbs

Breadcrumbs added to the following op pages:
- confirm
- delete_confirm
- generic_confirm
- edititem
- edit_action
- check_out
- cud-cancel
- availability

Test plan:
1) Enable ILLModule
2) Create a new ILL request:
   <staff_url>/cgi-bin/koha/ill/ill-requests.pl?method=create&backend=Standard
3) Try various actions like 'Edit request', 'Confirm request', 'Place
   request with partners', etc.
4) Notice the breadcrumbs were previously missing. They are now there.
5) If you want to test the breadcrumbs on the batches pages or
   availability check stage let me know and I'll help setting up the
   required plugins to have them work.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
---
 .../prog/en/modules/ill/ill-requests.tt       | 28 ++++++++++++++++---
 1 file changed, 24 insertions(+), 4 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 495a3ab9ea..f2e3aa221d 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
@@ -25,11 +25,15 @@
 
 [% WRAPPER 'sub-header.inc' %]
     [% WRAPPER breadcrumbs %]
-        [% WRAPPER breadcrumb_item bc_active = (op == 'illlist' ? 1 : 0) %]
-            [% IF op == 'illlist' %]
-                <span>ILL requests</span>
+        [% WRAPPER breadcrumb_item bc_active = (op == 'illlist' || op == 'batch_list' ? 1 : 0) %]
+            [% IF op == 'batch_list' %]
+                <a href="/cgi-bin/koha/ill/ill-requests.pl">ILL batch requests</a>
             [% ELSE %]
-                <a href="/cgi-bin/koha/ill/ill-requests.pl">ILL requests</a>
+                [% IF op == 'illlist' %]
+                    <span>ILL requests</span>
+                [% ELSE %]
+                    <a href="/cgi-bin/koha/ill/ill-requests.pl">ILL requests</a>
+                [% END %]
             [% END %]
         [% END %]
         [% WRAPPER breadcrumb_item bc_active= 1 %]
@@ -37,6 +41,22 @@
                 <span>New request</span>
             [% ELSIF op == 'illview' %]
                 <span>Manage request [% request.id_prefix _ request.illrequest_id | html %]</span>
+            [% ELSIF op == 'confirm' %]
+                <span>Confirm request [% request.id_prefix _ request.illrequest_id | html %]</span>
+            [% ELSIF op == 'delete_confirm' %]
+                <span>Delete request [% request.id_prefix _ request.illrequest_id | html %]</span>
+            [% ELSIF op == 'generic_confirm' %]
+                <span>Place request [% request.id_prefix _ request.illrequest_id | html %] with partners</span>
+            [% ELSIF op == 'edititem' %]
+                <span>Edit item metadata for request [% request.id_prefix _ request.illrequest_id | html %]</span>
+            [% ELSIF op == 'edit_action' %]
+                <span>Edit request [% request.id_prefix _ request.illrequest_id | html %]</span>
+            [% ELSIF op == 'check_out' %]
+                <span>Check out request [% request.id_prefix _ request.illrequest_id | html %]</span>
+            [% ELSIF op == 'cud-cancel' %]
+                <span>Cancel request [% request.id_prefix _ request.illrequest_id | html %]</span>
+            [% ELSIF op == 'availability' %]
+                <span>Availability check</span>
             [% ELSIF op == 'typedisclaimer' %]
                 <span>Request type disclaimer</span>
             [% ELSIF op == 'confirmautoill' %]