From 7f15421d5c76338d1218dbe5e95005c38750bd9e Mon Sep 17 00:00:00 2001 From: Andrii Veremeienko Date: Sun, 17 Oct 2021 14:43:34 +0300 Subject: [PATCH] Bug 29257: Redundant slash in the URL appear during item level hold placing The solution was to get rid of adding slash to $biblionumbers directly and join slash only in place when it's needed. How to test: 1. Search for an item; 2. Select single item, click 'Place hold'; 3. Enter a patron card number; 4. Complete hold placing by clicking 'Place hold'; 5. Note that after redirection to the page with hold list, URL doesn't contain slash after biblionumber parameter; E.g.: request.pl?biblionumbers=233 --- reserve/placerequest.pl | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/reserve/placerequest.pl b/reserve/placerequest.pl index 5ee8dce2f6..2cf13a8893 100755 --- a/reserve/placerequest.pl +++ b/reserve/placerequest.pl @@ -53,7 +53,7 @@ my $borrower = Koha::Patrons->find( $borrowernumber ); $borrower = $borrower->unblessed if $borrower; my $biblionumbers = $input->param('biblionumbers'); -$biblionumbers ||= $biblionumber . '/'; +$biblionumbers ||= $biblionumber; my $bad_bibs = $input->param('bad_bibs'); my $holds_to_place_count = $input->param('holds_to_place_count') || 1; @@ -164,10 +164,12 @@ if ( $type eq 'str8' && $borrower ) { } } + my $url = "request.pl?biblionumbers=$biblionumbers"; if ($bad_bibs) { - $biblionumbers .= $bad_bibs; + $url .= '/' . $bad_bibs; } - print $input->redirect("request.pl?biblionumbers=$biblionumbers"); + + print $input->redirect($url); } elsif ( $borrowernumber eq '' ) { print $input->header(); -- 2.17.1