@@ -350,10 +355,14 @@
Not holdable
[% ELSIF itemloo.not_holdable == 'cannotReserveFromOtherBranches' %]
Patron is from different library
+ [% ELSIF itemloo.not_holdable == 'branchNotInHoldGroup' %]
+ Cannot place hold from patrons's library
[% ELSIF itemloo.not_holdable == 'itemAlreadyOnHold' %]
Patron already has hold for this item
[% ELSIF itemloo.not_holdable == 'cannotBeTransferred' %]
Cannot be transferred to pickup library
+ [% ELSIF itemloo.not_holdable == 'pickupNotInHoldGroup' %]
+ Pickup library is not in hold group
[% ELSE %]
[% itemloo.not_holdable | html %]
[% END %]
@@ -437,6 +446,9 @@
Damaged
[% END %]
+
+ [% itemloo.pickup_locations | html %]
+
[% END %]
[% END %]
--- a/reserve/request.pl
+++ a/reserve/request.pl
@@ -227,7 +227,7 @@ foreach my $biblionumber (@biblionumbers) {
my $force_hold_level;
if ( $patron ) {
{ # CanBookBeReserved
- my $canReserve = CanBookBeReserved( $patron->borrowernumber, $biblionumber, $pickup );
+ my $canReserve = CanBookBeReserved( $patron->borrowernumber, $biblionumber );
if ( $canReserve->{status} eq 'OK' ) {
#All is OK and we can continue
@@ -489,7 +489,7 @@ foreach my $biblionumber (@biblionumbers) {
$item->{'holdallowed'} = $branchitemrule->{'holdallowed'};
- my $can_item_be_reserved = CanItemBeReserved( $patron->borrowernumber, $itemnumber, $pickup );
+ my $can_item_be_reserved = CanItemBeReserved( $patron->borrowernumber, $itemnumber );
$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 } );
@@ -503,6 +503,11 @@ foreach my $biblionumber (@biblionumbers) {
{
$item->{available} = 1;
$num_available++;
+ if($branchitemrule->{'hold_fulfillment_policy'} eq 'any' ) {
+ $item->{pickup_locations} = 'Any library';
+ } else {
+ $item->{pickup_locations} = join (', ', map { $_->{branchname} } Koha::Items->find($itemnumber)->pickup_locations());
+ }
push( @available_itemtypes, $item->{itype} );
}
--