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::CirculationRules;
use Koha::Items;
use Koha::ItemTypes;
use Koha::Libraries;
use Koha::Patrons;
+use Koha::Clubs;
my $dbh = C4::Context->dbh;
my $input = new CGI;
}
);
-my $multihold = $input->param('multi_hold');
-$template->param(multi_hold => $multihold);
my $showallitems = $input->param('showallitems');
+my $pickup = $input->param('pickup') || C4::Context->userenv->{'branch'};
my $itemtypes = { map { $_->{itemtype} => $_ } @{ Koha::ItemTypes->search_with_localization->unblessed } };
my $findborrower = $input->param('findborrower');
$findborrower = '' unless defined $findborrower;
$findborrower =~ s|,| |g;
+my $findclub = $input->param('findclub');
+$findclub = '' unless defined $findclub && !$findborrower;
my $borrowernumber_hold = $input->param('borrowernumber') || '';
+my $club_hold = $input->param('club')||'';
my $messageborrower;
+my $messageclub;
my $warnings;
my $messages;
my $exceeded_maxreserves;
$action ||= q{};
if ( $action eq 'move' ) {
- my $where = $input->param('where');
- my $reserve_id = $input->param('reserve_id');
- AlterPriority( $where, $reserve_id );
+ 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');
- CancelReserve({ reserve_id => $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 );
}
}
+if($findclub) {
+ my $club = Koha::Clubs->find( { name => $findclub } );
+ if( $club ) {
+ $club_hold = $club->id;
+ } else {
+ my @clubs = Koha::Clubs->search( [
+ { name => { like => '%'.$findclub.'%' } },
+ { description => { like => '%'.$findclub.'%' } }
+ ] );
+ if( scalar @clubs == 1 ) {
+ $club_hold = $clubs[0]->id;
+ } elsif ( @clubs ) {
+ $template->param( clubs => \@clubs );
+ } else {
+ $messageclub = "'$findclub'";
+ }
+ }
+}
+
my @biblionumbers = ();
+my $biblionumber = $input->param('biblionumber');
my $biblionumbers = $input->param('biblionumbers');
-if ($multihold) {
+if ( $biblionumbers ) {
@biblionumbers = split '/', $biblionumbers;
} else {
push @biblionumbers, $input->multi_param('biblionumber');
}
+my $multi_hold = @biblionumbers > 1;
+$template->param(multi_hold => $multi_hold);
# If we have the borrowernumber because we've performed an action, then we
# don't want to try to place another reserve.
# 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 =
- GetReserveCount( $patron->borrowernumber );
+ my $reserves_count = $patron->holds->count;
my $new_reserves_count = scalar( @biblionumbers );
$diffbranch = 1;
}
- my $is_debarred = $patron->is_debarred;
+ my $amount_outstanding = $patron->account->balance;
$template->param(
- borrowernumber => $patron->borrowernumber,
- borrowersurname => $patron->surname,
- borrowerfirstname => $patron->firstname,
- borrowerstreetaddress => $patron->address,
- borrowercity => $patron->city,
- borrowerphone => $patron->phone,
- borrowermobile => $patron->mobile,
- borrowerfax => $patron->fax,
- borrowerphonepro => $patron->phonepro,
- borroweremail => $patron->email,
- borroweremailpro => $patron->emailpro,
- cardnumber => $patron->cardnumber,
+ patron => $patron,
expiry => $expiry,
diffbranch => $diffbranch,
messages => $messages,
warnings => $warnings,
- restricted => $is_debarred,
- amount_outstanding => GetMemberAccountRecords($patron->borrowernumber),
+ amount_outstanding => $amount_outstanding,
);
}
-$template->param( messageborrower => $messageborrower );
+if ($club_hold && !$borrowernumber_hold && !$action) {
+ my $club = Koha::Clubs->find($club_hold);
+
+ my $enrollments = $club->club_enrollments;
+
+ my $maxreserves = C4::Context->preference('maxreserves');
+ my $new_reserves_count = scalar( @biblionumbers );
+
+ my @members;
+
+ while(my $enrollment = $enrollments->next) {
+ next if $enrollment->is_canceled;
+ my $member = { patron => $enrollment->patron->unblessed };
+ my $reserves_count = $enrollment->patron->holds->count;
+ if ( $maxreserves
+ && ( $reserves_count + $new_reserves_count > $maxreserves ) )
+ {
+ $member->{new_reserves_allowed} = $maxreserves - $reserves_count > 0
+ ? $maxreserves - $reserves_count
+ : 0;
+ $member->{exceeded_maxreserves} = 1;
+ }
+ my $expiry_date = $enrollment->patron->dateexpiry;
+ $member->{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)) {
+ $member->{expiry} = 1;
+ }
+ $member->{amount_outstanding} = $enrollment->patron->account->balance;
+ if ( $enrollment->patron->branchcode ne C4::Context->userenv->{'branch'} ) {
+ $member->{diffbranch} = 1;
+ }
+
+ push @members, $member;
+ }
+
+ $template->param(
+ club => $club,
+ members => \@members,
+ maxreserves => $maxreserves,
+ new_reserves_count => $new_reserves_count
+ );
+}
+
+$template->param(
+ messageborrower => $messageborrower,
+ messageclub => $messageclub
+);
# 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 $dat = GetBiblioData($biblionumber);
+ my $biblio = Koha::Biblios->find( $biblionumber );
- my $canReserve = CanBookBeReserved( $patron->borrowernumber, $biblionumber );
- $canReserve //= '';
- if ( $canReserve eq 'OK' ) {
+ my $force_hold_level;
+ if ( $patron ) {
+ { # CanBookBeReserved
+ my $canReserve = CanBookBeReserved( $patron->borrowernumber, $biblionumber );
+ if ( $canReserve->{status} eq 'OK' ) {
- #All is OK and we can continue
- }
- elsif ( $canReserve eq 'tooManyReserves' ) {
- $exceeded_maxreserves = 1;
- }
- elsif ( $canReserve eq 'tooManyHoldsForThisRecord' ) {
- $exceeded_holds_per_record = 1;
- $biblioloopiter{$canReserve} = 1;
- }
- elsif ( $canReserve eq 'ageRestricted' ) {
- $template->param( $canReserve => 1 );
- $biblioloopiter{$canReserve} = 1;
- }
- else {
- $biblioloopiter{$canReserve} = 1;
- }
+ #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;
+ }
+ }
- my $force_hold_level;
- if ( $patron->borrowernumber ) {
# 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
$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 );
- }
- # Check to see if patron is allowed to place holds on records where the
- # patron already has an item from that record checked out
- my $alreadypossession;
- if ( !C4::Context->preference('AllowHoldsOnPatronsPossessions')
- && CheckIfIssuedToPatron( $patron->borrowernumber, $biblionumber ) )
- {
- $template->param( alreadypossession => $alreadypossession, );
+ { # 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, );
+ }
+ }
}
## Should be same as biblionumber
my @biblioitemnumbers = keys %itemnumbers_of_biblioitem;
- ## Hash of biblioitemnumber to 'biblioitem' table records
- my $biblioiteminfos_of = GetBiblioItemInfosOf(@biblioitemnumbers);
-
- 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 $biblioiteminfos_of = {
+ map {
+ my $biblioitem = $_;
+ ( $biblioitem->{biblioitemnumber} => $biblioitem )
+ } @{ Koha::Biblioitems->search(
+ { biblioitemnumber => { -in => \@biblioitemnumbers } },
+ { select => ['biblioitemnumber', 'publicationyear', 'itemtype']}
+ )->unblessed
+ }
+ };
my @bibitemloop;
my $num_available = 0;
my $num_override = 0;
my $hiddencount = 0;
+ my $num_alreadyheld = 0;
$biblioitem->{force_hold_level} = $force_hold_level;
$itemtypes->{ $biblioitem->{itemtype} }{imageurl} );
}
+ # iterating through all items first to check if any of them available
+ # to pass this value further inside down to IsAvailableForItemLevelRequest to
+ # it's complicated logic to analyse.
+ # (before this loop was inside that sub loop so it was O(n^2) )
+ my $items_any_available;
+
+ $items_any_available = ItemsAnyAvailableAndNotRestricted( { biblionumber => $biblioitemnumber, patron => $patron })
+ if $patron;
+
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 ( $multi_hold ) {
+ $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->{holdingbranch} = $item->{holdingbranch};
}
- if($item->{biblionumber} ne $biblionumber){
- $item->{hostitemsflag}=1;
- $item->{hosttitle} = GetBiblioData($item->{biblionumber})->{title};
- }
-
+ 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 } );
}
# checking reserve
- my $holds = Koha::Items->find( $itemnumber )->current_holds;
+ my $item_object = Koha::Items->find( $itemnumber );
+ my $holds = $item_object->current_holds;
if ( my $first_hold = $holds->next ) {
- my $patron = Koha::Patrons->find( $first_hold->borrowernumber );
+ 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->{ReservedForBorrowernumber} = $first_hold->borrowernumber;
- $item->{ReservedForSurname} = $patron->surname;
- $item->{ReservedForFirstname} = $patron->firstname;
+ $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;
# for branches they arent logged in to
if ( C4::Context->preference("IndependentBranches") ) {
if (! C4::Context->preference("canreservefromotherbranches")){
- # cant reserve items so need to check if item homebranch and userenv branch match if not we cant reserve
+ # 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} );
}
}
- my $patron_unblessed = $patron->unblessed;
- my $branch = C4::Circulation::_GetCircControlBranch($item, $patron_unblessed);
+ if ( $patron ) {
+ my $patron_unblessed = $patron->unblessed;
+ my $branch = C4::Circulation::_GetCircControlBranch($item, $patron_unblessed);
- my $branchitemrule = GetBranchItemRule( $branch, $item->{'itype'} );
+ my $branchitemrule = GetBranchItemRule( $branch, $item->{'itype'} );
- $item->{'holdallowed'} = $branchitemrule->{'holdallowed'};
+ $item->{'holdallowed'} = $branchitemrule->{'holdallowed'};
- my $can_item_be_reserved = CanItemBeReserved( $patron->borrowernumber, $itemnumber );
- $item->{not_holdable} = $can_item_be_reserved unless ( $can_item_be_reserved eq 'OK' );
+ my $can_item_be_reserved = CanItemBeReserved( $patron->borrowernumber, $itemnumber )->{status};
+ $item->{not_holdable} = $can_item_be_reserved unless ( $can_item_be_reserved eq 'OK' );
- $item->{item_level_holds} = OPACItemHoldsAllowed( $item, $patron_unblessed);
+ $item->{item_level_holds} = Koha::CirculationRules->get_opacitemholds_policy( { item => $item_object, patron => $patron } );
- if (
- !$item->{cantreserve}
- && !$exceeded_maxreserves
- && IsAvailableForItemLevelRequest($item, $patron_unblessed)
- && $can_item_be_reserved eq 'OK'
- )
- {
- $item->{available} = 1;
- $num_available++;
+ if (
+ !$item->{cantreserve}
+ && !$exceeded_maxreserves
+ && $can_item_be_reserved eq 'OK'
+ # items_any_available defined outside of the current loop,
+ # so we avoiding loop inside IsAvailableForItemLevelRequest:
+ && IsAvailableForItemLevelRequest($item_object, $patron, undef, $items_any_available)
+ )
+ {
+ $item->{available} = 1;
+ $num_available++;
+ if($branchitemrule->{'hold_fulfillment_policy'} eq 'any' ) {
+ $item->{pickup_locations} = 'Any library';
+ $item->{pickup_locations_code} = 'all';
+ } else {
+ my $arr_locations = Koha::Items->find($itemnumber)
+ ->pickup_locations( { patron => $patron } );
+
+ $item->{pickup_locations} = join( ', ',
+ map { $_->unblessed->{branchname} } @$arr_locations);
+ $item->{pickup_locations_code} = join( ',',
+ map { $_->unblessed->{branchcode} } @$arr_locations);
+ }
- push( @available_itemtypes, $item->{itype} );
- }
- elsif ( C4::Context->preference('AllowHoldPolicyOverride') ) {
- # If AllowHoldPolicyOverride is set, it should override EVERY restriction, not just branch item rules
- $item->{override} = 1;
- $num_override++;
- }
+ 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++;
+ } else { $num_alreadyheld++ }
+
+ 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
+ # 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 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
+ # While we can't override an alreay held item, we should be able to override the others
+ # Unless all items are already held
+ if ( $num_override > 0 && ($num_override + $num_alreadyheld) == 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 );
}
}
- # get borrowers reserve info
- if ( C4::Context->preference('HidePatronName') ) {
- $reserve{'hidename'} = 1;
- $reserve{'cardnumber'} = $res->borrower()->cardnumber();
- }
- $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{'expirationdate'} = $res->expirationdate;
+ $reserve{'date'} = $res->reservedate;
$reserve{'borrowernumber'} = $res->borrowernumber();
$reserve{'biblionumber'} = $res->biblionumber();
- $reserve{'borrowernumber'} = $res->borrowernumber();
- $reserve{'firstname'} = $res->borrower()->firstname();
- $reserve{'surname'} = $res->borrower()->surname();
+ $reserve{'patron'} = $res->borrower;
$reserve{'notes'} = $res->reservenotes();
$reserve{'waiting_date'} = $res->waitingdate();
$reserve{'ccode'} = $res->item() ? $res->item()->ccode() : undef;
$reserve{'reserve_id'} = $res->reserve_id();
$reserve{itemtype} = $res->itemtype();
$reserve{branchcode} = $res->branchcode();
+ $reserve{object} = $res;
push( @reserveloop, \%reserve );
}
optionloop => \@optionloop,
bibitemloop => \@bibitemloop,
itemdata_enumchron => $itemdata_enumchron,
+ itemdata_ccode => $itemdata_ccode,
date => $date,
biblionumber => $biblionumber,
findborrower => $findborrower,
- title => $dat->{title},
- author => $dat->{author},
- holdsview => 1,
+ biblio => $biblio,
+ holdsview => 1,
C4::Search::enabled_staff_search_views,
);
- if ( $patron ) { # FIXME This test seems very useless
- $template->param( borrower_branchcode => $patron->branchcode );
- }
$biblioloopiter{biblionumber} = $biblionumber;
- $biblioloopiter{title} = $dat->{title};
+ $biblioloopiter{title} = $biblio->title;
$biblioloopiter{rank} = $fixedRank;
$biblioloopiter{reserveloop} = \@reserveloop;
$template->param( biblionumbers => $biblionumbers );
$template->param( exceeded_maxreserves => $exceeded_maxreserves );
$template->param( exceeded_holds_per_record => $exceeded_holds_per_record );
-
-if ($multihold) {
- $template->param( multi_hold => 1 );
-}
+$template->param( subscriptionsnumber => CountSubscriptionFromBiblionumber($biblionumber));
+$template->param( pickup => $pickup );
if ( C4::Context->preference( 'AllowHoldDateInFuture' ) ) {
$template->param( reserve_in_future => 1 );
projects / koha-equinox.git / blobdiff