From 88f257ba55e4f0a4c58c0f64815b9995b7072080 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Joonas=20Kylm=C3=A4l=C3=A4?= <joonas.kylmala@iki.fi>
Date: Sat, 21 May 2022 19:23:32 +0000
Subject: [PATCH] Bug 30828: Remove useless 'type' parameter in placerequest.pl

To test:
 1) Grep for placerequest.pl in the source code and check that
    request.tt is the only place where the type parameter is passed.

Signed-off-by: David Nind <david@davidnind.com>

JD amended patch: adjust commit title
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit 54a6c73c6d6e94554b7e4c15554e08d79e3eb578)
---
 koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt | 2 --
 reserve/placerequest.pl                                    | 3 +--
 2 files changed, 1 insertion(+), 4 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
index f5184e85b3..396afc7964 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
@@ -446,8 +446,6 @@
                     <form action="placerequest.pl" method="post" name="form" id="hold-request-form">
 
                         <input type="hidden" name="borrowernumber" value="[% patron.borrowernumber | html %]" />
-                        <input type="hidden" name="type" value="str8" />
-
                         [% FOREACH biblionumber IN biblionumbers %]
                             <input type="hidden" name="biblionumber" value="[% biblionumber | html %]"/>
                         [% END %]
diff --git a/reserve/placerequest.pl b/reserve/placerequest.pl
index 7b8f7f84ac..4f255fdcc5 100755
--- a/reserve/placerequest.pl
+++ b/reserve/placerequest.pl
@@ -43,7 +43,6 @@ my $notes          = $input->param('notes');
 my $branch         = $input->param('pickup');
 my $startdate      = $input->param('reserve_date') || '';
 my @rank           = $input->multi_param('rank-request');
-my $type           = $input->param('type');
 my $title          = $input->param('title');
 my $checkitem      = $input->param('checkitem');
 my $expirationdate = $input->param('expiration_date');
@@ -65,7 +64,7 @@ foreach my $bibnum ( @holdable_bibs ) {
 
 my $found;
 
-if ( $type eq 'str8' && $patron ) {
+if ( $patron ) {
 
     foreach my $biblionumber ( keys %bibinfos ) {
 
-- 
2.20.1