Jonathan Druart
93244efa3c
If a hold is selected from the result list, we should let the ability to select an item-level hold. Test plan: I. Detail page 1/ Go to a bibliographic record detail page 2/ Click "Place hold" 3/ Select a patron => No change expected, you can select an item II. Search result, multiple holds 1/ Search for an item with more than one search result 2/ Select several items, click 'Place hold' 3/ Enter a patron card number => No change expected, item level holds are not available. III. Search result, single hold 1/ Search for an item with more than one search result 2/ Select only one item, click 'Place hold' 3/ Enter a patron card number => With this patch applied, item level hold is available. The screen is the same as when you place a hold from the bibliographic record detail page => Without this patch you cannot place an item-level hold QA notes: We could go a bit further and remove the 2 biblionumbers and biblionumber from hold script, as well as remove the checkMultiHold in request.tt. We should not have a biblionumbers param that contain a list of biblionumber separated by '/' but several biblionumber parameters instead. QA notes 2: About placerequest.pl, see bug 19618 comment 27. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
102 lines
3.6 KiB
Perl
Executable file
102 lines
3.6 KiB
Perl
Executable file
#!/usr/bin/perl
|
|
|
|
#script to modify reserves/requests
|
|
#written 2/1/00 by chris@katipo.oc.nz
|
|
#last update 27/1/2000 by chris@katipo.co.nz
|
|
|
|
|
|
# Copyright 2000-2002 Katipo Communications
|
|
#
|
|
# This file is part of Koha.
|
|
#
|
|
# Koha is free software; you can redistribute it and/or modify it
|
|
# under the terms of the GNU General Public License as published by
|
|
# the Free Software Foundation; either version 3 of the License, or
|
|
# (at your option) any later version.
|
|
#
|
|
# Koha is distributed in the hope that it will be useful, but
|
|
# WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
# GNU General Public License for more details.
|
|
#
|
|
# You should have received a copy of the GNU General Public License
|
|
# along with Koha; if not, see <http://www.gnu.org/licenses>.
|
|
|
|
use Modern::Perl;
|
|
use CGI qw ( -utf8 );
|
|
use C4::Output;
|
|
use C4::Reserves;
|
|
use C4::Auth;
|
|
use Koha::DateUtils qw( dt_from_string );
|
|
|
|
my $query = new CGI;
|
|
my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
|
|
{
|
|
template_name => "about.tt",
|
|
query => $query,
|
|
type => "intranet",
|
|
authnotrequired => 0,
|
|
flagsrequired => { catalogue => 1 },
|
|
debug => 1,
|
|
}
|
|
);
|
|
|
|
my @reserve_id = $query->multi_param('reserve_id');
|
|
my @rank = $query->multi_param('rank-request');
|
|
my @biblionumber = $query->multi_param('biblionumber');
|
|
my @borrower = $query->multi_param('borrowernumber');
|
|
my @reservedates = $query->multi_param('reservedate');
|
|
my @expirationdates = $query->multi_param('expirationdate');
|
|
my @branch = $query->multi_param('pickup');
|
|
my @itemnumber = $query->multi_param('itemnumber');
|
|
my $biblionumbers = $query->param('biblionumbers');
|
|
my $count=@rank;
|
|
|
|
my $CancelBiblioNumber = $query->param('CancelBiblioNumber');
|
|
my $CancelBorrowerNumber = $query->param('CancelBorrowerNumber');
|
|
my $CancelItemnumber = $query->param('CancelItemnumber');
|
|
|
|
# 2 possibilitys : cancel an item reservation, or modify or cancel the queded list
|
|
|
|
# 1) cancel an item reservation by function ModReserveCancelAll (in reserves.pm)
|
|
if ($CancelBorrowerNumber) {
|
|
ModReserveCancelAll($CancelItemnumber, $CancelBorrowerNumber);
|
|
$biblionumber[0] = $CancelBiblioNumber,
|
|
}
|
|
|
|
# 2) Cancel or modify the queue list of reserves (without item linked)
|
|
else {
|
|
for (my $i=0;$i<$count;$i++){
|
|
undef $itemnumber[$i] if !$itemnumber[$i];
|
|
my $suspend_until = $query->param( "suspend_until_" . $reserve_id[$i] );
|
|
my $params = {
|
|
rank => $rank[$i],
|
|
reserve_id => $reserve_id[$i],
|
|
expirationdate => $expirationdates[$i] ? dt_from_string($expirationdates[$i]) : undef,
|
|
branchcode => $branch[$i],
|
|
itemnumber => $itemnumber[$i],
|
|
defined $suspend_until ? ( suspend_until => $suspend_until ) : (),
|
|
};
|
|
if (C4::Context->preference('AllowHoldDateInFuture')) {
|
|
$params->{reservedate} = $reservedates[$i] ? dt_from_string($reservedates[$i]) : undef;
|
|
}
|
|
|
|
ModReserve($params);
|
|
}
|
|
}
|
|
|
|
my $from=$query->param('from');
|
|
$from ||= q{};
|
|
if ( $from eq 'borrower'){
|
|
print $query->redirect("/cgi-bin/koha/members/moremember.pl?borrowernumber=$borrower[0]");
|
|
} elsif ( $from eq 'circ'){
|
|
print $query->redirect("/cgi-bin/koha/circ/circulation.pl?borrowernumber=$borrower[0]");
|
|
} else {
|
|
my $url = "/cgi-bin/koha/reserve/request.pl?";
|
|
if (@biblionumber > 1) {
|
|
$url .= "multi_hold=1&biblionumbers=$biblionumbers";
|
|
} else {
|
|
$url .= "biblionumber=$biblionumber[0]";
|
|
}
|
|
print $query->redirect($url);
|
|
}
|