From ef40adfd8cb3724c2074c65c3a0c5de6e78597f7 Mon Sep 17 00:00:00 2001
From: Nick Clemens <nick@bywatersolutions.com>
Date: Fri, 26 Jan 2024 12:56:39 -0500
Subject: [PATCH] Bug 34478: Manual fix - change to post remove cud from
 confirm step - serials-collection

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

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt
index 2f0c7c75fc..a52c54f19d 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-collection.tt
@@ -50,14 +50,15 @@
                 [% IF ( delete ) %]
                     <div class="dialog alert">
                         <h3>You are about to delete [% serialnumber | html %] serial(s). Do you want to continue?</h3>
-                        <form method="get" action="/cgi-bin/koha/serials/serials-collection.pl">
+                        <form method="post" action="/cgi-bin/koha/serials/serials-collection.pl">
+                            [% INCLUDE 'csrf-token.inc' %]
                             [% IF countitems > 0 %]
                                 <h5>Number of items available: [% countitems | html %]</h5>
                                 <p>
                                 <label>Delete associated items? <input type="checkbox" name="delitems" value="Yes" />Yes</label>
                                 </p>
                             [% END %]
-                            <input type="hidden" name="op" value="delete_confirmed" />
+                            <input type="hidden" name="op" value="cud-delete_confirmed" />
                             <input type="hidden" name="biblionumber" value="[% biblionumber | html %]" />
                             <input type="hidden" name="subscriptionid" value="[% subscriptionid | html %]" />
                             [% FOREACH serialid IN serialsid %]
diff --git a/serials/serials-collection.pl b/serials/serials-collection.pl
index 93db64f105..4a6c8ba2f4 100755
--- a/serials/serials-collection.pl
+++ b/serials/serials-collection.pl
@@ -110,7 +110,7 @@ my $countitems = 0;
 my @serialsid = $query->multi_param('serialid');
 my $subscriptionid = $subscriptionid[0];
 
-if($op eq 'cud-delete_confirm'){
+if($op eq 'delete_confirm'){
     foreach my $serialid (@serialsid){
         $countitems += Koha::Serial::Items->search({serialid => $serialid})->count();
     }
@@ -205,7 +205,7 @@ $template->param(
           callnumber	       => $callnumber,
           uc(C4::Context->preference("marcflavour")) => 1,
           serialsadditems   => $subscriptiondescs->[0]{'serialsadditems'},
-          delete => ($op eq 'cud-delete_confirm'),
+          delete => ($op eq 'delete_confirm'),
           subscriptionid => $subscriptionid,
           countitems => $countitems,
           serialnumber => scalar @serialsid,
-- 
2.39.5