From 3a8ada1e0fbb14c2c353f93166f8befa55a40896 Mon Sep 17 00:00:00 2001
From: Nick Clemens <nick@bywatersolutions.com>
Date: Thu, 22 Feb 2024 07:39:06 -0500
Subject: [PATCH] Bug 34478: Manual fix - add cud- op for checkouts -
 circulation.tt

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
---
 circ/circulation.pl                                         | 5 +++--
 koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt | 5 +++++
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/circ/circulation.pl b/circ/circulation.pl
index a1a264a201..a6a30ce288 100755
--- a/circ/circulation.pl
+++ b/circ/circulation.pl
@@ -111,6 +111,7 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user (
     }
 );
 
+my $op                      = $query->param('op');
 my $override_high_holds     = $query->param('override_high_holds');
 my $override_high_holds_tmp = $query->param('override_high_holds_tmp');
 
@@ -137,7 +138,7 @@ if ( C4::Context->preference("AutoSwitchPatron") && $barcode ) {
 $findborrower ||= $query->param('findborrower') || q{};
 $findborrower =~ s|,| |g;
 
-if ( $query->param('confirm_hold') ) {
+if ( $op eq 'cud-confirm_hold' && $query->param('confirm_hold') ) {
     my $reserve_id          = $query->param('confirm_hold');
     my $hold_branch         = $query->param('hold_branch');
     my $hold_itemnumber     = $query->param('hold_itemnumber');
@@ -310,7 +311,7 @@ if ($patron) {
 # STEP 3 : ISSUING
 #
 #
-if (@$barcodes) {
+if (@$barcodes && $op eq 'cud-checkout') {
     my $checkout_infos;
     for my $barcode ( @$barcodes ) {
 
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
index 654e910852..3eb1a55434 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
@@ -277,6 +277,7 @@
                                     [% INCLUDE 'csrf-token.inc' %]
                                     <fieldset>
                                     <input type="hidden" name="restoreduedatespec" />
+                                    <input type="hidden" name="op" value="cud-checkout" />
 
                                     [% IF (forceallow) %]<input type="hidden" name="forceallow" value="1">[% END %]
 
@@ -383,6 +384,7 @@
                             [% IF ADDITIONAL_MATERIALS && !CAN_user_circulate_force_checkout %]
                                 <form method="post" action="/cgi-bin/koha/circ/circulation.pl" autocomplete="off">
                                     [% INCLUDE 'csrf-token.inc' %]
+                                    <input type="hidden" name="op" value="cud-checkout" />
                                     <input type="hidden" name="restoreduedatespec" />
 
                                     [% IF (forceallow) %]<input type="hidden" name="forceallow" value="1">[% END %]
@@ -408,6 +410,7 @@
                             [% IF ( RESERVED or RESERVE_WAITING or TRANSFERRED or PROCESSING ) %]
                                 <form method="post" action="/cgi-bin/koha/circ/circulation.pl">
                                     [% INCLUDE 'csrf-token.inc' %]
+                                    <input type="hidden" name="op" value="cud-confirm_hold" />
                                     <input type="hidden" name="restoreduedatespec" />
                                     <input type="hidden" name="borrowernumber" value="[% patron.borrowernumber | html %]" />
                                     <input type="hidden" name="duedatespec" value="[% duedatespec | html %]" />
@@ -668,6 +671,7 @@
                                                                         <input type="hidden" name="forceallow" value="1">
                                                                     [% END %]
                                                                     <input type="hidden" name="restoreduedatespec" />
+                                                                    <input type="hidden" name="op" value="cud-checkout" />
                                                                     <input type="hidden" name="borrowernumber" value="[% patron.borrowernumber | html %]" />
                                                                     <input type="hidden" name="duedatespec" value="[% duedatespec | html %]" />
                                                                     <input type="hidden" name="stickyduedate" value="[% stickyduedate | html %]" />
@@ -724,6 +728,7 @@
                                 <div class="col-sm-6">
                                     <form method="post" action="/cgi-bin/koha/circ/circulation.pl" id="mainform" name="mainform" autocomplete="off">
                                         [% INCLUDE 'csrf-token.inc' %]
+                                        <input type="hidden" name="op" value="cud-checkout" />
                                         <input type="hidden" name="restoreduedatespec" />
                                         [% IF ( issue ) %]
                                             <fieldset id="circ_circulation_issue" class="lastchecked">
-- 
2.39.5