#
# 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.
+# You should have received a copy of the GNU General Public License
+# along with Koha; if not, see <http://www.gnu.org/licenses>.
-use strict;
-use warnings;
-use CGI;
+use Modern::Perl;
+use CGI qw ( -utf8 );
use C4::Context;
-use C4::Output;
-use C4::Branch; # GetBranchName
-use C4::Auth;
-use C4::Dates qw/format_date/;
-use C4::Circulation;
-use C4::Members;
-use C4::Biblio;
-use C4::Items;
-
-use Date::Calc qw(
- Today
- Add_Delta_Days
- Date_to_Days
-);
-use C4::Reserves;
-use C4::Koha;
-
-my $input = new CGI;
+use C4::Output qw( output_html_with_http_headers );
+use C4::Auth qw( get_template_and_user );
+use C4::Items qw( ModItemTransfer );
+use Date::Calc qw( Date_to_Days Today );
+use C4::Reserves qw( ModReserve ModReserveCancelAll );
+use Koha::DateUtils qw( dt_from_string );
+use Koha::BiblioFrameworks;
+use Koha::Items;
+use Koha::ItemTypes;
+use Koha::Patrons;
+use Koha::BackgroundJob::BatchCancelHold;
+
+my $input = CGI->new;
my $item = $input->param('itemnumber');
my $borrowernumber = $input->param('borrowernumber');
my $tbr = $input->param('tbr') || '';
my $all_branches = $input->param('allbranches') || '';
my $cancelall = $input->param('cancelall');
+my $tab = $input->param('tab');
+my $cancelBulk = $input->param('cancelBulk');
-my $cancel;
-
-my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
+my ( $template, $loggedinuser, $cookie, $flags ) = get_template_and_user(
{
- template_name => "circ/waitingreserves.tmpl",
+ template_name => "circ/waitingreserves.tt",
query => $input,
type => "intranet",
- authnotrequired => 0,
flagsrequired => { circulate => "circulate_remaining_permissions" },
- debug => 1,
}
);
$template->param( TransferWhenCancelAllWaitingHolds => 1 ) if $transfer_when_cancel_all;
my @cancel_result;
-# if we have a return from the form we launch the subroutine CancelReserve
+# if we have a return from the form we cancel the holds
if ($item) {
my $res = cancel( $item, $borrowernumber, $fbr, $tbr );
push @cancel_result, $res if $res;
}
-if ( C4::Context->preference('IndependantBranches') ) {
+if ( C4::Context->preference('IndependentBranches') ) {
undef $all_branches;
} else {
- $template->param( all_branches_link => $input->url . '?allbranches=1' )
+ $template->param( all_branches_link => '/cgi-bin/koha/circ/waitingreserves.pl' . '?allbranches=1' )
unless $all_branches;
}
$template->param( all_branches => 1 ) if $all_branches;
-my (@reservloop, @overloop);
-my ($reservcount, $overcount);
-my @getreserves = $all_branches ? GetReservesForBranch() : GetReservesForBranch($default);
+if ($cancelBulk) {
+ my $reason = $input->param("cancellation-reason");
+ my @hold_ids = split( ',', scalar $input->param("ids") );
+ my $params = {
+ reason => $reason,
+ hold_ids => \@hold_ids,
+ };
+ my $job_id = Koha::BackgroundJob::BatchCancelHold->new->enqueue($params);
+
+ $template->param(
+ enqueued => 1,
+ job_id => $job_id
+ );
+}
+
+my (@reserve_loop, @over_loop);
+# FIXME - Is priority => 0 useful? If yes it must be moved to waiting, otherwise we need to remove it from here.
+my $holds = Koha::Holds->waiting->search({ priority => 0, ( $all_branches ? () : ( branchcode => $default ) ) }, { order_by => ['waitingdate'] });
+
# get reserves for the branch we are logged into, or for all branches
my $today = Date_to_Days(&Today);
-foreach my $num (@getreserves) {
- next unless ($num->{'waitingdate'} && $num->{'waitingdate'} ne '0000-00-00');
-
- my $itemnumber = $num->{'itemnumber'};
- my $gettitle = GetBiblioFromItemNumber( $itemnumber );
- my $borrowernum = $num->{'borrowernumber'};
- my $holdingbranch = $gettitle->{'holdingbranch'};
- my $homebranch = $gettitle->{'homebranch'};
-
- my %getreserv = (
- itemnumber => $itemnumber,
- borrowernum => $borrowernum,
- );
- # fix up item type for display
- $gettitle->{'itemtype'} = C4::Context->preference('item-level_itypes') ? $gettitle->{'itype'} : $gettitle->{'itemtype'};
- my $getborrower = GetMember(borrowernumber => $num->{'borrowernumber'});
- my $itemtypeinfo = getitemtypeinfo( $gettitle->{'itemtype'} ); # using the fixed up itype/itemtype
- $getreserv{'waitingdate'} = format_date( $num->{'waitingdate'} );
- my ( $waiting_year, $waiting_month, $waiting_day ) = split (/-/, $num->{'waitingdate'});
- ( $waiting_year, $waiting_month, $waiting_day ) =
- Add_Delta_Days( $waiting_year, $waiting_month, $waiting_day,
- C4::Context->preference('ReservesMaxPickUpDelay'));
- my $calcDate = Date_to_Days( $waiting_year, $waiting_month, $waiting_day );
-
- $getreserv{'itemtype'} = $itemtypeinfo->{'description'};
- $getreserv{'title'} = $gettitle->{'title'};
- $getreserv{'biblionumber'} = $gettitle->{'biblionumber'};
- $getreserv{'barcode'} = $gettitle->{'barcode'};
- $getreserv{'branchname'} = GetBranchName($gettitle->{'homebranch'});
- $getreserv{'homebranch'} = $gettitle->{'homebranch'};
- $getreserv{'holdingbranch'} = $gettitle->{'holdingbranch'};
- $getreserv{'itemcallnumber'} = $gettitle->{'itemcallnumber'};
- $getreserv{'enumchron'} = $gettitle->{'enumchron'};
- $getreserv{'copynumber'} = $gettitle->{'copynumber'};
- if ( $homebranch ne $holdingbranch ) {
- $getreserv{'dotransfer'} = 1;
- }
- $getreserv{'borrowername'} = $getborrower->{'surname'};
- $getreserv{'borrowerfirstname'} = $getborrower->{'firstname'};
- $getreserv{'borrowerphone'} = $getborrower->{'phone'};
- if ( $getborrower->{'emailaddress'} ) {
- $getreserv{'borrowermail'} = $getborrower->{'emailaddress'};
- }
-
+while ( my $hold = $holds->next ) {
+ next unless $hold->waitingdate;
+
+ my ( $expire_year, $expire_month, $expire_day ) = split (/-/, $hold->expirationdate);
+ my $calcDate = Date_to_Days( $expire_year, $expire_month, $expire_day );
+
if ($today > $calcDate) {
if ($cancelall) {
- my $res = cancel( $itemnumber, $borrowernum, $holdingbranch, $homebranch, !$transfer_when_cancel_all );
+ my $res = cancel( $hold->item->itemnumber, $hold->borrowernumber, $hold->item->holdingbranch, $hold->item->homebranch, !$transfer_when_cancel_all );
push @cancel_result, $res if $res;
next;
} else {
- push @overloop, \%getreserv;
- $overcount++;
+ push @over_loop, $hold;
}
}else{
- push @reservloop, \%getreserv;
- $reservcount++;
+ push @reserve_loop, $hold;
}
}
+my $holds_with_cancellation_requests = Koha::Holds->waiting->filter_by_has_cancellation_requests;
+
$template->param(cancel_result => \@cancel_result) if @cancel_result;
+
$template->param(
- reserveloop => \@reservloop,
- reservecount => $reservcount,
- overloop => \@overloop,
- overcount => $overcount,
- show_date => format_date(C4::Dates->today('iso')),
- dateformat => C4::Context->preference("dateformat"),
- ReservesMaxPickUpDelay => C4::Context->preference('ReservesMaxPickUpDelay')
+ reserveloop => \@reserve_loop,
+ reservecount => scalar @reserve_loop,
+ overloop => \@over_loop,
+ overcount => scalar @over_loop,
+ cancel_reqs_count => $holds_with_cancellation_requests->count,
+ cancel_reqs => $holds_with_cancellation_requests,
+ show_date => dt_from_string,
+ tab => $tab,
);
-if ($cancelall) {
+# Checking if there is a Fast Cataloging Framework
+$template->param( fast_cataloging => 1 ) if Koha::BiblioFrameworks->find( 'FA' );
+
+if ($item && $tab eq 'holdsover' && !@cancel_result) {
+ print $input->redirect("/cgi-bin/koha/circ/waitingreserves.pl#holdsover");
+} elsif ($cancelall) {
print $input->redirect("/cgi-bin/koha/circ/waitingreserves.pl");
} else {
output_html_with_http_headers $input, $cookie, $template->output;
# if the document is not in his homebranch location and there is not reservation after, we transfer it
if ($transfer && !$nextreservinfo) {
- ModItemTransfer( $item, $fbr, $tbr );
+ ModItemTransfer( $item, $fbr, $tbr, 'CancelReserve' );
}
# if we have a result
if ($nextreservinfo) {
my %res;
- my $borrowerinfo = GetMemberDetails( $nextreservinfo );
- my $iteminfo = GetBiblioFromItemNumber($item);
+ my $patron = Koha::Patrons->find( $nextreservinfo );
+ my $title = Koha::Items->find( $item )->biblio->title;
if ( $messages->{'transfert'} ) {
$res{messagetransfert} = $messages->{'transfert'};
- $res{branchname} = GetBranchName($messages->{'transfert'});
+ $res{branchcode} = $messages->{'transfert'};
}
$res{message} = 1;
$res{nextreservnumber} = $nextreservinfo;
- $res{nextreservsurname} = $borrowerinfo->{'surname'};
- $res{nextreservfirstname} = $borrowerinfo->{'firstname'};
+ $res{nextreservsurname} = $patron->surname;
+ $res{nextreservfirstname} = $patron->firstname;
$res{nextreservitem} = $item;
- $res{nextreservtitle} = $iteminfo->{'title'};
+ $res{nextreservtitle} = $title;
$res{waiting} = $messages->{'waiting'} ? 1 : 0;
return \%res;