#!/usr/bin/perl
#script to place reserves/requests
-#writen 2/1/00 by chris@katipo.oc.nz
+#written 2/1/00 by chris@katipo.oc.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 2 of the License, or (at your option) any later
-# version.
+# 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.
+# 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, write to the Free Software Foundation, Inc.,
-# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
-
-use strict;
-use warnings;
-
-use CGI;
-use C4::Biblio;
-use C4::Items;
-use C4::Output;
-use C4::Reserves;
-use C4::Circulation;
-use C4::Members;
-use C4::Auth qw/checkauth/;
+# 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::Reserves qw( CanItemBeReserved AddReserve CanBookBeReserved );
+use C4::Auth qw( checkauth );
+
+use Koha::Items;
+use Koha::Patrons;
my $input = CGI->new();
checkauth($input, 0, { reserveforothers => 'place_holds' }, 'intranet');
-my @bibitems=$input->param('biblioitem');
-# FIXME I think reqbib does not exist anymore, it's used in line 82, to AddReserve of contraint type 'o'
-# I bet it's a 2.x feature, reserving a given biblioitem, that is useless in Koha 3.0
-# we can remove this line, the AddReserve of constrainttype 'o',
-# and probably remove the reserveconstraint table as well, I never could fill anything in this table.
-my @reqbib=$input->param('reqbib');
-my $biblionumber=$input->param('biblionumber');
-my $borrowernumber=$input->param('borrowernumber');
-my $notes=$input->param('notes');
-my $branch=$input->param('pickup');
-my $startdate=$input->param('reserve_date') || '';
-my @rank=$input->param('rank-request');
-my $type=$input->param('type');
-my $title=$input->param('title');
-my $borrower=GetMember('borrowernumber'=>$borrowernumber);
-my $checkitem=$input->param('checkitem');
+my @bibitems = $input->multi_param('biblioitem');
+my @reqbib = $input->multi_param('reqbib');
+my $biblionumber = $input->param('biblionumber');
+my $borrowernumber = $input->param('borrowernumber');
+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');
+my $itemtype = $input->param('itemtype') || undef;
+my $non_priority = $input->param('non_priority');
+
+my $borrower = Koha::Patrons->find( $borrowernumber );
+$borrower = $borrower->unblessed if $borrower;
+
+my $biblionumbers = $input->param('biblionumbers');
+$biblionumbers ||= $biblionumber . '/';
-my $multi_hold = $input->param('multi_hold');
-my $biblionumbers = $multi_hold ? $input->param('biblionumbers') : ($biblionumber . '/');
my $bad_bibs = $input->param('bad_bibs');
+my $holds_to_place_count = $input->param('holds_to_place_count') || 1;
my %bibinfos = ();
my @biblionumbers = split '/', $biblionumbers;
foreach my $bibnum (@biblionumbers) {
my %bibinfo = ();
- $bibinfo{title} = $input->param("title_$bibnum");
- $bibinfo{rank} = $input->param("rank_$bibnum");
+ $bibinfo{title} = $input->param("title_$bibnum");
+ $bibinfo{rank} = $input->param("rank_$bibnum");
+ $bibinfo{pickup} = $input->param("pickup_$bibnum");
$bibinfos{$bibnum} = \%bibinfo;
}
my $found;
-# if we have an item selectionned, and the pickup branch is the same as the holdingbranch
-# of the document, we force the value $rank and $found .
-if ($checkitem ne ''){
- $rank[0] = '0' unless C4::Context->preference('ReservesNeedReturns');
- my $item = $checkitem;
- $item = GetItem($item);
- if ( $item->{'holdingbranch'} eq $branch ){
- $found = 'W' unless C4::Context->preference('ReservesNeedReturns');
- }
-}
+if ( $type eq 'str8' && $borrower ) {
-if ($type eq 'str8' && $borrower){
-
- foreach my $biblionumber (keys %bibinfos) {
- my $count=@bibitems;
- @bibitems=sort @bibitems;
- my $i2=1;
+ foreach my $biblionumber ( keys %bibinfos ) {
+ my $count = @bibitems;
+ @bibitems = sort @bibitems;
+ my $i2 = 1;
my @realbi;
- $realbi[0]=$bibitems[0];
- for (my $i=1;$i<$count;$i++) {
- my $i3=$i2-1;
- if ($realbi[$i3] ne $bibitems[$i]) {
- $realbi[$i2]=$bibitems[$i];
+ $realbi[0] = $bibitems[0];
+ for ( my $i = 1 ; $i < $count ; $i++ ) {
+ my $i3 = $i2 - 1;
+ if ( $realbi[$i3] ne $bibitems[$i] ) {
+ $realbi[$i2] = $bibitems[$i];
$i2++;
}
}
- my $const;
- if ($checkitem ne ''){
- my $item = GetItem($checkitem);
- if ($item->{'biblionumber'} ne $biblionumber) {
- $biblionumber = $item->{'biblionumber'};
- }
- }
+ my $can_override = C4::Context->preference('AllowHoldPolicyOverride');
+ if ( defined $checkitem && $checkitem ne '' ) {
+ my $item_pickup_location = $input->param("item_pickup_$checkitem");
+ my $item = Koha::Items->find($checkitem);
- if ($multi_hold) {
+ if ( $item->biblionumber ne $biblionumber ) {
+ $biblionumber = $item->biblionumber;
+ }
+
+ my $can_item_be_reserved = CanItemBeReserved($borrower->{'borrowernumber'}, $item->itemnumber, $item_pickup_location)->{status};
+
+ if ( $can_item_be_reserved eq 'OK' || ( $can_item_be_reserved ne 'itemAlreadyOnHold' && $can_override ) ) {
+ AddReserve(
+ {
+ branchcode => $item_pickup_location,
+ borrowernumber => $borrower->{'borrowernumber'},
+ biblionumber => $biblionumber,
+ priority => $rank[0],
+ reservation_date => $startdate,
+ expiration_date => $expirationdate,
+ notes => $notes,
+ title => $title,
+ itemnumber => $checkitem,
+ found => $found,
+ itemtype => $itemtype,
+ non_priority => $non_priority,
+ }
+ );
+
+ }
+
+ } elsif (@biblionumbers > 1) {
my $bibinfo = $bibinfos{$biblionumber};
- AddReserve($branch,$borrower->{'borrowernumber'},$biblionumber,'a',[$biblionumber],
- $bibinfo->{rank},$startdate,$expirationdate,$notes,$bibinfo->{title},$checkitem,$found);
+ if ( $can_override || CanBookBeReserved($borrower->{'borrowernumber'}, $biblionumber)->{status} eq 'OK' ) {
+ AddReserve(
+ {
+ branchcode => $bibinfo->{pickup},
+ borrowernumber => $borrower->{'borrowernumber'},
+ biblionumber => $biblionumber,
+ priority => $bibinfo->{rank},
+ reservation_date => $startdate,
+ expiration_date => $expirationdate,
+ notes => $notes,
+ title => $bibinfo->{title},
+ itemnumber => $checkitem,
+ found => $found,
+ itemtype => $itemtype,
+ non_priority => $non_priority,
+ }
+ );
+ }
} else {
- if ($input->param('request') eq 'any'){
- # place a request on 1st available
- AddReserve($branch,$borrower->{'borrowernumber'},$biblionumber,'a',\@realbi,$rank[0],$startdate,$expirationdate,$notes,$title,$checkitem,$found);
- } elsif ($reqbib[0] ne ''){
- # FIXME : elsif probably never reached, (see top of the script)
- # place a request on a given item
- AddReserve($branch,$borrower->{'borrowernumber'},$biblionumber,'o',\@reqbib,$rank[0],$startdate,$expirationdate,$notes,$title,$checkitem, $found);
- } else {
- AddReserve($branch,$borrower->{'borrowernumber'},$biblionumber,'a',\@realbi,$rank[0],$startdate,$expirationdate,$notes,$title,$checkitem, $found);
+ # place a request on 1st available
+ for ( my $i = 0 ; $i < $holds_to_place_count ; $i++ ) {
+ if ( $can_override || CanBookBeReserved($borrower->{'borrowernumber'}, $biblionumber)->{status} eq 'OK' ) {
+ AddReserve(
+ {
+ branchcode => $branch,
+ borrowernumber => $borrower->{'borrowernumber'},
+ biblionumber => $biblionumber,
+ priority => $rank[0],
+ reservation_date => $startdate,
+ expiration_date => $expirationdate,
+ notes => $notes,
+ title => $title,
+ itemnumber => $checkitem,
+ found => $found,
+ itemtype => $itemtype,
+ non_priority => $non_priority,
+ }
+ );
+ }
}
}
}
- if ($multi_hold) {
- if ($bad_bibs) {
- $biblionumbers .= $bad_bibs;
- }
- print $input->redirect("request.pl?biblionumbers=$biblionumbers&multi_hold=1");
- } else {
- print $input->redirect("request.pl?biblionumber=$biblionumber");
+ if ($bad_bibs) {
+ $biblionumbers .= $bad_bibs;
}
-} elsif ($borrower eq ''){
- print $input->header();
- print "Invalid borrower number please try again";
-# Not sure that Dump() does HTML escaping. Use firebug or something to trace
-# instead.
-# print $input->Dump;
+ print $input->redirect("request.pl?biblionumbers=$biblionumbers");
+}
+elsif ( $borrowernumber eq '' ) {
+ print $input->header();
+ print "Invalid borrower number please try again";
+
+ # Not sure that Dump() does HTML escaping. Use firebug or something to trace
+ # instead.
+ #print $input->Dump;
}