Kyle M Hall
2c5e9bed7d
A library appears to be able to place a second item level hold on an item a patron already has on hold if A) AllowHoldPolicyOverride is enabled and B) the circ rule allow for multple item level holds. Once the patron submits the hold requests though, the hold does not get stored in the database. Because allowing two item level holds for the same item makes no sense, we should not allow this attempt to take place, even if AllowHoldPolicyOverride is enabled. Test Plan: 1) Enable AllowHoldPolicyOverride 2) Set up circ rules to allow for multiple item level holds on one record 3) Place an item level hold on a record 4) Note you can force placing a second item hold on that reocrd 5) Attempt to do so, it will not actually work 6) Apply this patch 7) Note you can no longer place another item level hold for the same item you just placed an item-level hold on 8) Note you can still force holds that contravene the circ rules for any and all other reasons 9) Test with record level holds 10) Test by placing multiple holds from search results Signed-off-by: Martha Fuerst <mfurest@hmcpl.org> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Simple fix for a regression. Works as expected. Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
690 lines
26 KiB
Perl
Executable file
690 lines
26 KiB
Perl
Executable file
#!/usr/bin/perl
|
|
|
|
|
|
#written 2/1/00 by chris@katipo.oc.nz
|
|
# Copyright 2000-2002 Katipo Communications
|
|
# Parts Copyright 2011 Catalyst IT
|
|
#
|
|
# 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>.
|
|
|
|
=head1 request.pl
|
|
|
|
script to place reserves/requests
|
|
|
|
=cut
|
|
|
|
use Modern::Perl;
|
|
|
|
use CGI qw ( -utf8 );
|
|
use List::MoreUtils qw/uniq/;
|
|
use Date::Calc qw/Date_to_Days/;
|
|
use C4::Output;
|
|
use C4::Auth;
|
|
use C4::Reserves;
|
|
use C4::Biblio;
|
|
use C4::Items;
|
|
use C4::Koha;
|
|
use C4::Serials;
|
|
use C4::Circulation;
|
|
use Koha::DateUtils;
|
|
use C4::Utils::DataTables::Members;
|
|
use C4::Members;
|
|
use C4::Search; # enabled_staff_search_views
|
|
|
|
use Koha::Biblios;
|
|
use Koha::DateUtils;
|
|
use Koha::Checkouts;
|
|
use Koha::Holds;
|
|
use Koha::IssuingRules;
|
|
use Koha::Items;
|
|
use Koha::ItemTypes;
|
|
use Koha::Libraries;
|
|
use Koha::Patrons;
|
|
|
|
my $dbh = C4::Context->dbh;
|
|
my $input = new CGI;
|
|
my ( $template, $borrowernumber, $cookie, $flags ) = get_template_and_user(
|
|
{
|
|
template_name => "reserve/request.tt",
|
|
query => $input,
|
|
type => "intranet",
|
|
authnotrequired => 0,
|
|
flagsrequired => { reserveforothers => 'place_holds' },
|
|
}
|
|
);
|
|
|
|
my $showallitems = $input->param('showallitems');
|
|
my $pickup = $input->param('pickup') || C4::Context->userenv->{'branch'};
|
|
|
|
my $itemtypes = { map { $_->{itemtype} => $_ } @{ Koha::ItemTypes->search_with_localization->unblessed } };
|
|
|
|
# Select borrowers infos
|
|
my $findborrower = $input->param('findborrower');
|
|
$findborrower = '' unless defined $findborrower;
|
|
$findborrower =~ s|,| |g;
|
|
my $borrowernumber_hold = $input->param('borrowernumber') || '';
|
|
my $messageborrower;
|
|
my $warnings;
|
|
my $messages;
|
|
my $exceeded_maxreserves;
|
|
my $exceeded_holds_per_record;
|
|
|
|
my $date = output_pref({ dt => dt_from_string, dateformat => 'iso', dateonly => 1 });
|
|
my $action = $input->param('action');
|
|
$action ||= q{};
|
|
|
|
if ( $action eq 'move' ) {
|
|
my $where = $input->param('where');
|
|
my $reserve_id = $input->param('reserve_id');
|
|
my $prev_priority = $input->param('prev_priority');
|
|
my $next_priority = $input->param('next_priority');
|
|
my $first_priority = $input->param('first_priority');
|
|
my $last_priority = $input->param('last_priority');
|
|
my $hold_itemnumber = $input->param('itemnumber');
|
|
if ( $prev_priority == 0 && $next_priority == 1 ){
|
|
C4::Reserves::RevertWaitingStatus({ itemnumber => $hold_itemnumber });
|
|
} else {
|
|
AlterPriority( $where, $reserve_id, $prev_priority, $next_priority, $first_priority, $last_priority );
|
|
}
|
|
} elsif ( $action eq 'cancel' ) {
|
|
my $reserve_id = $input->param('reserve_id');
|
|
my $hold = Koha::Holds->find( $reserve_id );
|
|
$hold->cancel if $hold;
|
|
} elsif ( $action eq 'setLowestPriority' ) {
|
|
my $reserve_id = $input->param('reserve_id');
|
|
ToggleLowestPriority( $reserve_id );
|
|
} elsif ( $action eq 'toggleSuspend' ) {
|
|
my $reserve_id = $input->param('reserve_id');
|
|
my $suspend_until = $input->param('suspend_until');
|
|
ToggleSuspend( $reserve_id, $suspend_until );
|
|
}
|
|
|
|
if ($findborrower) {
|
|
my $patron = Koha::Patrons->find( { cardnumber => $findborrower } );
|
|
if ( $patron ) {
|
|
$borrowernumber_hold = $patron->borrowernumber;
|
|
} else {
|
|
my $dt_params = { iDisplayLength => -1 };
|
|
my $results = C4::Utils::DataTables::Members::search(
|
|
{
|
|
searchmember => $findborrower,
|
|
dt_params => $dt_params,
|
|
}
|
|
);
|
|
my $borrowers = $results->{patrons};
|
|
if ( scalar @$borrowers == 1 ) {
|
|
$borrowernumber_hold = $borrowers->[0]->{borrowernumber};
|
|
} elsif ( @$borrowers ) {
|
|
$template->param( borrowers => $borrowers );
|
|
} else {
|
|
$messageborrower = "'$findborrower'";
|
|
}
|
|
}
|
|
}
|
|
|
|
my @biblionumbers = ();
|
|
my $biblionumber = $input->param('biblionumber');
|
|
my $biblionumbers = $input->param('biblionumbers');
|
|
if ( $biblionumbers ) {
|
|
@biblionumbers = split '/', $biblionumbers;
|
|
} else {
|
|
push @biblionumbers, $input->multi_param('biblionumber');
|
|
}
|
|
|
|
my $multihold = scalar $input->param('multi_hold');
|
|
# FIXME multi_hold should not be a variable but depends on the number of elements in @biblionumbers
|
|
$template->param(multi_hold => scalar $input->param('multi_hold'));
|
|
|
|
# If we have the borrowernumber because we've performed an action, then we
|
|
# don't want to try to place another reserve.
|
|
if ($borrowernumber_hold && !$action) {
|
|
my $patron = Koha::Patrons->find( $borrowernumber_hold );
|
|
my $diffbranch;
|
|
|
|
# we check the reserves of the user, and if they can reserve a document
|
|
# FIXME At this time we have a simple count of reservs, but, later, we could improve the infos "title" ...
|
|
|
|
my $reserves_count = $patron->holds->count;
|
|
|
|
my $new_reserves_count = scalar( @biblionumbers );
|
|
|
|
my $maxreserves = C4::Context->preference('maxreserves');
|
|
if ( $maxreserves
|
|
&& ( $reserves_count + $new_reserves_count > $maxreserves ) )
|
|
{
|
|
my $new_reserves_allowed =
|
|
$maxreserves - $reserves_count > 0
|
|
? $maxreserves - $reserves_count
|
|
: 0;
|
|
$warnings = 1;
|
|
$exceeded_maxreserves = 1;
|
|
$template->param(
|
|
new_reserves_allowed => $new_reserves_allowed,
|
|
new_reserves_count => $new_reserves_count,
|
|
reserves_count => $reserves_count,
|
|
maxreserves => $maxreserves,
|
|
);
|
|
}
|
|
|
|
# we check the date expiry of the borrower (only if there is an expiry date, otherwise, set to 1 (warn)
|
|
my $expiry_date = $patron->dateexpiry;
|
|
my $expiry = 0; # flag set if patron account has expired
|
|
if ($expiry_date and $expiry_date ne '0000-00-00' and
|
|
Date_to_Days(split /-/,$date) > Date_to_Days(split /-/,$expiry_date)) {
|
|
$expiry = 1;
|
|
}
|
|
|
|
# check if the borrower make the reserv in a different branch
|
|
if ( $patron->branchcode ne C4::Context->userenv->{'branch'} ) {
|
|
$diffbranch = 1;
|
|
}
|
|
|
|
my $amount_outstanding = $patron->account->balance;
|
|
$template->param(
|
|
patron => $patron,
|
|
expiry => $expiry,
|
|
diffbranch => $diffbranch,
|
|
messages => $messages,
|
|
warnings => $warnings,
|
|
amount_outstanding => $amount_outstanding,
|
|
);
|
|
}
|
|
|
|
$template->param( messageborrower => $messageborrower );
|
|
|
|
# FIXME launch another time GetMember perhaps until (Joubu: Why?)
|
|
my $patron = Koha::Patrons->find( $borrowernumber_hold );
|
|
|
|
my $logged_in_patron = Koha::Patrons->find( $borrowernumber );
|
|
|
|
my $wants_check;
|
|
if ($patron) {
|
|
$wants_check = $patron->wants_check_for_previous_checkout;
|
|
}
|
|
my $itemdata_enumchron = 0;
|
|
my $itemdata_ccode = 0;
|
|
my @biblioloop = ();
|
|
foreach my $biblionumber (@biblionumbers) {
|
|
next unless $biblionumber =~ m|^\d+$|;
|
|
|
|
my %biblioloopiter = ();
|
|
|
|
my $biblio = Koha::Biblios->find( $biblionumber );
|
|
|
|
my $force_hold_level;
|
|
if ( $patron ) {
|
|
{ # CanBookBeReserved
|
|
my $canReserve = CanBookBeReserved( $patron->borrowernumber, $biblionumber, $pickup );
|
|
if ( $canReserve->{status} eq 'OK' ) {
|
|
|
|
#All is OK and we can continue
|
|
}
|
|
elsif ( $canReserve->{status} eq 'tooManyReserves' ) {
|
|
$exceeded_maxreserves = 1;
|
|
$template->param( maxreserves => $canReserve->{limit} );
|
|
}
|
|
elsif ( $canReserve->{status} eq 'tooManyHoldsForThisRecord' ) {
|
|
$exceeded_holds_per_record = 1;
|
|
$biblioloopiter{ $canReserve->{status} } = 1;
|
|
}
|
|
elsif ( $canReserve->{status} eq 'ageRestricted' ) {
|
|
$template->param( $canReserve->{status} => 1 );
|
|
$biblioloopiter{ $canReserve->{status} } = 1;
|
|
}
|
|
else {
|
|
$biblioloopiter{ $canReserve->{status} } = 1;
|
|
}
|
|
}
|
|
|
|
# For multiple holds per record, if a patron has previously placed a hold,
|
|
# the patron can only place more holds of the same type. That is, if the
|
|
# patron placed a record level hold, all the holds the patron places must
|
|
# be record level. If the patron placed an item level hold, all holds
|
|
# the patron places must be item level
|
|
my $holds = Koha::Holds->search(
|
|
{
|
|
borrowernumber => $patron->borrowernumber,
|
|
biblionumber => $biblionumber,
|
|
found => undef,
|
|
}
|
|
);
|
|
$force_hold_level = $holds->forced_hold_level();
|
|
$biblioloopiter{force_hold_level} = $force_hold_level;
|
|
$template->param( force_hold_level => $force_hold_level );
|
|
|
|
# For a librarian to be able to place multiple record holds for a patron for a record,
|
|
# we must find out what the maximum number of holds they can place for the patron is
|
|
my $max_holds_for_record = GetMaxPatronHoldsForRecord( $patron->borrowernumber, $biblionumber );
|
|
my $remaining_holds_for_record = $max_holds_for_record - $holds->count();
|
|
$biblioloopiter{remaining_holds_for_record} = $max_holds_for_record;
|
|
$template->param( max_holds_for_record => $max_holds_for_record );
|
|
$template->param( remaining_holds_for_record => $remaining_holds_for_record );
|
|
|
|
{ # alreadypossession
|
|
# Check to see if patron is allowed to place holds on records where the
|
|
# patron already has an item from that record checked out
|
|
if ( !C4::Context->preference('AllowHoldsOnPatronsPossessions')
|
|
&& CheckIfIssuedToPatron( $patron->borrowernumber, $biblionumber ) )
|
|
{
|
|
$template->param( alreadypossession => 1, );
|
|
}
|
|
}
|
|
}
|
|
|
|
|
|
my $count = Koha::Holds->search( { biblionumber => $biblionumber } )->count();
|
|
my $totalcount = $count;
|
|
|
|
# FIXME think @optionloop, is maybe obsolete, or must be switchable by a systeme preference fixed rank or not
|
|
# make priorities options
|
|
|
|
my @optionloop;
|
|
for ( 1 .. $count + 1 ) {
|
|
push(
|
|
@optionloop,
|
|
{
|
|
num => $_,
|
|
selected => ( $_ == $count + 1 ),
|
|
}
|
|
);
|
|
}
|
|
# adding a fixed value for priority options
|
|
my $fixedRank = $count+1;
|
|
|
|
my %itemnumbers_of_biblioitem;
|
|
|
|
my @hostitems = get_hostitemnumbers_of($biblionumber);
|
|
my @itemnumbers;
|
|
if (@hostitems){
|
|
$template->param('hostitemsflag' => 1);
|
|
push(@itemnumbers, @hostitems);
|
|
}
|
|
|
|
my $items = Koha::Items->search({ -or => { biblionumber => $biblionumber, itemnumber => { in => \@itemnumbers } } });
|
|
|
|
unless ( $items->count ) {
|
|
# FIXME Then why do we continue?
|
|
$template->param('noitems' => 1);
|
|
$biblioloopiter{noitems} = 1;
|
|
}
|
|
|
|
## Here we go backwards again to create hash of biblioitemnumber to itemnumbers,
|
|
## when by definition all of the itemnumber have the same biblioitemnumber
|
|
my ( $iteminfos_of );
|
|
while ( my $item = $items->next ) {
|
|
$item = $item->unblessed;
|
|
my $biblioitemnumber = $item->{biblioitemnumber};
|
|
my $itemnumber = $item->{itemnumber};
|
|
push( @{ $itemnumbers_of_biblioitem{$biblioitemnumber} }, $itemnumber );
|
|
$iteminfos_of->{$itemnumber} = $item;
|
|
}
|
|
|
|
## Should be same as biblionumber
|
|
my @biblioitemnumbers = keys %itemnumbers_of_biblioitem;
|
|
|
|
my $biblioiteminfos_of = {
|
|
map {
|
|
my $biblioitem = $_;
|
|
( $biblioitem->{biblioitemnumber} => $biblioitem )
|
|
} @{ Koha::Biblioitems->search(
|
|
{ biblioitemnumber => { -in => \@biblioitemnumbers } },
|
|
{ select => ['biblioitemnumber', 'publicationyear', 'itemtype']}
|
|
)->unblessed
|
|
}
|
|
};
|
|
|
|
my $frameworkcode = GetFrameworkCode( $biblionumber );
|
|
my @notforloan_avs = Koha::AuthorisedValues->search_by_koha_field({ kohafield => 'items.notforloan', frameworkcode => $frameworkcode });
|
|
my $notforloan_label_of = { map { $_->authorised_value => $_->lib } @notforloan_avs };
|
|
|
|
my @bibitemloop;
|
|
|
|
my @available_itemtypes;
|
|
foreach my $biblioitemnumber (@biblioitemnumbers) {
|
|
my $biblioitem = $biblioiteminfos_of->{$biblioitemnumber};
|
|
my $num_available = 0;
|
|
my $num_override = 0;
|
|
my $hiddencount = 0;
|
|
|
|
$biblioitem->{force_hold_level} = $force_hold_level;
|
|
|
|
if ( $biblioitem->{biblioitemnumber} ne $biblionumber ) {
|
|
$biblioitem->{hostitemsflag} = 1;
|
|
}
|
|
|
|
$biblioloopiter{description} = $biblioitem->{description};
|
|
$biblioloopiter{itypename} = $biblioitem->{description};
|
|
if ( $biblioitem->{itemtype} ) {
|
|
|
|
$biblioitem->{description} =
|
|
$itemtypes->{ $biblioitem->{itemtype} }{description};
|
|
|
|
$biblioloopiter{imageurl} =
|
|
getitemtypeimagelocation( 'intranet',
|
|
$itemtypes->{ $biblioitem->{itemtype} }{imageurl} );
|
|
}
|
|
|
|
foreach my $itemnumber ( @{ $itemnumbers_of_biblioitem{$biblioitemnumber} } ) {
|
|
my $item = $iteminfos_of->{$itemnumber};
|
|
my $do_check;
|
|
if ( $patron ) {
|
|
$do_check = $patron->do_check_for_previous_checkout($item) if $wants_check;
|
|
if ( $do_check && $wants_check ) {
|
|
$item->{checked_previously} = $do_check;
|
|
if ( $multihold ) {
|
|
$biblioloopiter{checked_previously} = $do_check;
|
|
} else {
|
|
$template->param( checked_previously => $do_check );
|
|
}
|
|
}
|
|
}
|
|
$item->{force_hold_level} = $force_hold_level;
|
|
|
|
unless (C4::Context->preference('item-level_itypes')) {
|
|
$item->{itype} = $biblioitem->{itemtype};
|
|
}
|
|
|
|
$item->{itypename} = $itemtypes->{ $item->{itype} }{description};
|
|
$item->{imageurl} = getitemtypeimagelocation( 'intranet', $itemtypes->{ $item->{itype} }{imageurl} );
|
|
$item->{homebranch} = $item->{homebranch};
|
|
|
|
# if the holdingbranch is different than the homebranch, we show the
|
|
# holdingbranch of the document too
|
|
if ( $item->{homebranch} ne $item->{holdingbranch} ) {
|
|
$item->{holdingbranch} = $item->{holdingbranch};
|
|
}
|
|
|
|
if($item->{biblionumber} ne $biblionumber){
|
|
$item->{hostitemsflag} = 1;
|
|
$item->{hosttitle} = Koha::Biblios->find( $item->{biblionumber} )->title;
|
|
}
|
|
|
|
# if the item is currently on loan, we display its return date and
|
|
# change the background color
|
|
my $issue = Koha::Checkouts->find( { itemnumber => $itemnumber } );
|
|
if ( $issue ) {
|
|
$item->{date_due} = $issue->date_due;
|
|
$item->{backgroundcolor} = 'onloan';
|
|
}
|
|
|
|
# checking reserve
|
|
my $item_object = Koha::Items->find( $itemnumber );
|
|
my $holds = $item_object->current_holds;
|
|
if ( my $first_hold = $holds->next ) {
|
|
my $p = Koha::Patrons->find( $first_hold->borrowernumber );
|
|
|
|
$item->{backgroundcolor} = 'reserved';
|
|
$item->{reservedate} = output_pref({ dt => dt_from_string( $first_hold->reservedate ), dateonly => 1 }); # FIXME Should be formatted in the template
|
|
$item->{ReservedFor} = $p;
|
|
$item->{ExpectedAtLibrary} = $first_hold->branchcode;
|
|
$item->{waitingdate} = $first_hold->waitingdate;
|
|
}
|
|
|
|
# Management of the notforloan document
|
|
if ( $item->{notforloan} ) {
|
|
$item->{backgroundcolor} = 'other';
|
|
$item->{notforloanvalue} =
|
|
$notforloan_label_of->{ $item->{notforloan} };
|
|
}
|
|
|
|
# Management of lost or long overdue items
|
|
if ( $item->{itemlost} ) {
|
|
|
|
# FIXME localized strings should never be in Perl code
|
|
$item->{message} =
|
|
$item->{itemlost} == 1 ? "(lost)"
|
|
: $item->{itemlost} == 2 ? "(long overdue)"
|
|
: "";
|
|
$item->{backgroundcolor} = 'other';
|
|
if ($logged_in_patron->category->hidelostitems && !$showallitems) {
|
|
$item->{hide} = 1;
|
|
$hiddencount++;
|
|
}
|
|
}
|
|
|
|
# Check the transit status
|
|
my ( $transfertwhen, $transfertfrom, $transfertto ) =
|
|
GetTransfers($itemnumber);
|
|
|
|
if ( defined $transfertwhen && $transfertwhen ne '' ) {
|
|
$item->{transfertwhen} = output_pref({ dt => dt_from_string( $transfertwhen ), dateonly => 1 });
|
|
$item->{transfertfrom} = $transfertfrom;
|
|
$item->{transfertto} = $transfertto;
|
|
$item->{nocancel} = 1;
|
|
}
|
|
|
|
# If there is no loan, return and transfer, we show a checkbox.
|
|
$item->{notforloan} ||= 0;
|
|
|
|
# if independent branches is on we need to check if the person can reserve
|
|
# for branches they arent logged in to
|
|
if ( C4::Context->preference("IndependentBranches") ) {
|
|
if (! C4::Context->preference("canreservefromotherbranches")){
|
|
# can't reserve items so need to check if item homebranch and userenv branch match if not we can't reserve
|
|
my $userenv = C4::Context->userenv;
|
|
unless ( C4::Context->IsSuperLibrarian ) {
|
|
$item->{cantreserve} = 1 if ( $item->{homebranch} ne $userenv->{branch} );
|
|
}
|
|
}
|
|
}
|
|
|
|
if ( $patron ) {
|
|
my $patron_unblessed = $patron->unblessed;
|
|
my $branch = C4::Circulation::_GetCircControlBranch($item, $patron_unblessed);
|
|
|
|
my $branchitemrule = GetBranchItemRule( $branch, $item->{'itype'} );
|
|
|
|
$item->{'holdallowed'} = $branchitemrule->{'holdallowed'};
|
|
|
|
my $can_item_be_reserved = CanItemBeReserved( $patron->borrowernumber, $itemnumber, $pickup );
|
|
$item->{not_holdable} = $can_item_be_reserved->{status} unless ( $can_item_be_reserved->{status} eq 'OK' );
|
|
|
|
$item->{item_level_holds} = Koha::IssuingRules->get_opacitemholds_policy( { item => $item_object, patron => $patron } );
|
|
|
|
if (
|
|
!$item->{cantreserve}
|
|
&& !$exceeded_maxreserves
|
|
&& IsAvailableForItemLevelRequest($item_object, $patron)
|
|
&& $can_item_be_reserved->{status} eq 'OK'
|
|
)
|
|
{
|
|
$item->{available} = 1;
|
|
$num_available++;
|
|
|
|
push( @available_itemtypes, $item->{itype} );
|
|
}
|
|
elsif ( C4::Context->preference('AllowHoldPolicyOverride') ) {
|
|
# If AllowHoldPolicyOverride is set, it should override EVERY restriction, not just branch item rules
|
|
# with the exception of itemAlreadyOnHold because, you know, the item is already on hold
|
|
if ( $can_item_be_reserved ne 'itemAlreadyOnHold' ) {
|
|
$item->{override} = 1;
|
|
$num_override++;
|
|
}
|
|
|
|
push( @available_itemtypes, $item->{itype} );
|
|
}
|
|
|
|
# If none of the conditions hold true, then neither override nor available is set and the item cannot be checked
|
|
|
|
# Show serial enumeration when needed
|
|
if ($item->{enumchron}) {
|
|
$itemdata_enumchron = 1;
|
|
}
|
|
# Show collection when needed
|
|
if ($item->{ccode}) {
|
|
$itemdata_ccode = 1;
|
|
}
|
|
}
|
|
|
|
push @{ $biblioitem->{itemloop} }, $item;
|
|
}
|
|
|
|
if ( $num_override == scalar( @{ $biblioitem->{itemloop} } ) ) { # That is, if all items require an override
|
|
$template->param( override_required => 1 );
|
|
} elsif ( $num_available == 0 ) {
|
|
$template->param( none_available => 1 );
|
|
$biblioloopiter{warn} = 1;
|
|
$biblioloopiter{none_avail} = 1;
|
|
}
|
|
$template->param( hiddencount => $hiddencount);
|
|
|
|
push @bibitemloop, $biblioitem;
|
|
}
|
|
|
|
@available_itemtypes = uniq( @available_itemtypes );
|
|
$template->param( available_itemtypes => \@available_itemtypes );
|
|
|
|
# existingreserves building
|
|
my @reserveloop;
|
|
my @reserves = Koha::Holds->search( { biblionumber => $biblionumber }, { order_by => 'priority' } );
|
|
foreach my $res (
|
|
sort {
|
|
my $a_found = $a->found() || '';
|
|
my $b_found = $a->found() || '';
|
|
$a_found cmp $b_found;
|
|
} @reserves
|
|
)
|
|
{
|
|
my $priority = $res->priority();
|
|
my %reserve;
|
|
my @optionloop;
|
|
for ( my $i = 1 ; $i <= $totalcount ; $i++ ) {
|
|
push(
|
|
@optionloop,
|
|
{
|
|
num => $i,
|
|
selected => ( $i == $priority ),
|
|
}
|
|
);
|
|
}
|
|
|
|
if ( $res->is_found() ) {
|
|
$reserve{'holdingbranch'} = $res->item()->holdingbranch();
|
|
$reserve{'biblionumber'} = $res->item()->biblionumber();
|
|
$reserve{'barcodenumber'} = $res->item()->barcode();
|
|
$reserve{'wbrcode'} = $res->branchcode();
|
|
$reserve{'itemnumber'} = $res->itemnumber();
|
|
$reserve{'wbrname'} = $res->branch()->branchname();
|
|
|
|
if ( $reserve{'holdingbranch'} eq $reserve{'wbrcode'} ) {
|
|
|
|
# Just because the holdingbranch matches the reserve branch doesn't mean the item
|
|
# has arrived at the destination, check for an open transfer for the item as well
|
|
my ( $transfertwhen, $transfertfrom, $transferto ) =
|
|
C4::Circulation::GetTransfers( $res->itemnumber() );
|
|
if ( not $transferto or $transferto ne $res->branchcode() ) {
|
|
$reserve{'atdestination'} = 1;
|
|
}
|
|
}
|
|
|
|
# set found to 1 if reserve is waiting for patron pickup
|
|
$reserve{'found'} = $res->is_found();
|
|
$reserve{'intransit'} = $res->is_in_transit();
|
|
}
|
|
elsif ( $res->priority() > 0 ) {
|
|
if ( my $item = $res->item() ) {
|
|
$reserve{'itemnumber'} = $item->id();
|
|
$reserve{'barcodenumber'} = $item->barcode();
|
|
$reserve{'item_level_hold'} = 1;
|
|
}
|
|
}
|
|
|
|
$reserve{'expirationdate'} = output_pref( { dt => dt_from_string( $res->expirationdate ), dateonly => 1 } )
|
|
unless ( !defined( $res->expirationdate ) || $res->expirationdate eq '0000-00-00' );
|
|
$reserve{'date'} = output_pref( { dt => dt_from_string( $res->reservedate ), dateonly => 1 } );
|
|
$reserve{'borrowernumber'} = $res->borrowernumber();
|
|
$reserve{'biblionumber'} = $res->biblionumber();
|
|
$reserve{'patron'} = $res->borrower;
|
|
$reserve{'notes'} = $res->reservenotes();
|
|
$reserve{'waiting_date'} = $res->waitingdate();
|
|
$reserve{'ccode'} = $res->item() ? $res->item()->ccode() : undef;
|
|
$reserve{'barcode'} = $res->item() ? $res->item()->barcode() : undef;
|
|
$reserve{'priority'} = $res->priority();
|
|
$reserve{'lowestPriority'} = $res->lowestPriority();
|
|
$reserve{'optionloop'} = \@optionloop;
|
|
$reserve{'suspend'} = $res->suspend();
|
|
$reserve{'suspend_until'} = $res->suspend_until();
|
|
$reserve{'reserve_id'} = $res->reserve_id();
|
|
$reserve{itemtype} = $res->itemtype();
|
|
$reserve{branchcode} = $res->branchcode();
|
|
$reserve{object} = $res;
|
|
|
|
push( @reserveloop, \%reserve );
|
|
}
|
|
|
|
# get the time for the form name...
|
|
my $time = time();
|
|
|
|
$template->param(
|
|
time => $time,
|
|
fixedRank => $fixedRank,
|
|
);
|
|
|
|
# display infos
|
|
$template->param(
|
|
optionloop => \@optionloop,
|
|
bibitemloop => \@bibitemloop,
|
|
itemdata_enumchron => $itemdata_enumchron,
|
|
itemdata_ccode => $itemdata_ccode,
|
|
date => $date,
|
|
biblionumber => $biblionumber,
|
|
findborrower => $findborrower,
|
|
title => $biblio->title,
|
|
author => $biblio->author,
|
|
holdsview => 1,
|
|
C4::Search::enabled_staff_search_views,
|
|
);
|
|
|
|
$biblioloopiter{biblionumber} = $biblionumber;
|
|
$biblioloopiter{title} = $biblio->title;
|
|
$biblioloopiter{rank} = $fixedRank;
|
|
$biblioloopiter{reserveloop} = \@reserveloop;
|
|
|
|
if (@reserveloop) {
|
|
$template->param( reserveloop => \@reserveloop );
|
|
}
|
|
|
|
push @biblioloop, \%biblioloopiter;
|
|
}
|
|
|
|
$template->param( biblioloop => \@biblioloop );
|
|
$template->param( biblionumbers => $biblionumbers );
|
|
$template->param( exceeded_maxreserves => $exceeded_maxreserves );
|
|
$template->param( exceeded_holds_per_record => $exceeded_holds_per_record );
|
|
$template->param( subscriptionsnumber => CountSubscriptionFromBiblionumber($biblionumber));
|
|
$template->param( pickup => $pickup );
|
|
|
|
if ( C4::Context->preference( 'AllowHoldDateInFuture' ) ) {
|
|
$template->param( reserve_in_future => 1 );
|
|
}
|
|
|
|
$template->param(
|
|
SuspendHoldsIntranet => C4::Context->preference('SuspendHoldsIntranet'),
|
|
AutoResumeSuspendedHolds => C4::Context->preference('AutoResumeSuspendedHolds'),
|
|
);
|
|
|
|
# printout the page
|
|
output_html_with_http_headers $input, $cookie, $template->output;
|
|
|
|
sub sort_borrowerlist {
|
|
my $borrowerslist = shift;
|
|
my $ref = [];
|
|
push @{$ref}, sort {
|
|
uc( $a->{surname} . $a->{firstname} ) cmp
|
|
uc( $b->{surname} . $b->{firstname} )
|
|
} @{$borrowerslist};
|
|
return $ref;
|
|
}
|