Attachment #77900 for bug #7614

View | Details | Raw Unified | Return to bug 7614
Collapse All | Expand All

Lines 111-116 Link Here

(-)a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-reserve.tt (+5 lines)
111	</li>	111	</li>
112	[% END %]	112	[% END %]
113		113
		114	[% IF ( no_pickup_locations ) %]
		115	<li id="no_pickup_locations">
		116	None of the libraries are available for pickup location.
		117	</li>
		118	[% END %]
114	[% ELSE %]	119	[% ELSE %]
115		120
116	[% IF ( none_available && multi_hold ) %]	121	[% IF ( none_available && multi_hold ) %]




my $itemdata_enumchron = 0;
my $anyholdable = 0;
my $itemLevelTypes = C4::Context->preference('item-level_itypes');
my $pickup_locations = Koha::Libraries->search({ pickup_location => 1 });
$template->param('item_level_itypes' => $itemLevelTypes);

foreach my $biblioNum (@biblionumbers) {

        }
    }

    $biblioLoopIter{holdable} &&= CanBookBeReserved($borrowernumber,$biblioNum) eq 'OK';

    # 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

    $anyholdable = 1 if $biblioLoopIter{holdable};
}

unless ($pickup_locations->count) {
    $numBibsAvailable = 0;
    $anyholdable = 0;
    $template->param(
        message => 1,
        no_pickup_locations => 1
    );
}

if ( $numBibsAvailable == 0 || $anyholdable == 0) {
    $template->param( none_available => 1 );
- 

Return to bug 7614

Lines 389-394 my $numBibsAvailable = 0; Link Here

(-)a/opac/opac-reserve.pl (-2 / +10 lines)
389	my $itemdata_enumchron = 0;	389	my $itemdata_enumchron = 0;
390	my $anyholdable = 0;	390	my $anyholdable = 0;
391	my $itemLevelTypes = C4::Context->preference('item-level_itypes');	391	my $itemLevelTypes = C4::Context->preference('item-level_itypes');
		392	my $pickup_locations = Koha::Libraries->search({ pickup_location => 1 });
392	$template->param('item_level_itypes' => $itemLevelTypes);	393	$template->param('item_level_itypes' => $itemLevelTypes);
393		394
394	foreach my $biblioNum (@biblionumbers) {	395	foreach my $biblioNum (@biblionumbers) {
Lines 585-591 foreach my $biblioNum (@biblionumbers) { Link Here
585	}	586	}
586	}	587	}
587		588
588	$biblioLoopIter{holdable} &&= CanBookBeReserved($borrowernumber,$biblioNum,$branch) eq 'OK';	589	$biblioLoopIter{holdable} &&= CanBookBeReserved($borrowernumber,$biblioNum) eq 'OK';
589		590
590	# For multiple holds per record, if a patron has previously placed a hold,	591	# For multiple holds per record, if a patron has previously placed a hold,
591	# the patron can only place more holds of the same type. That is, if the	592	# the patron can only place more holds of the same type. That is, if the
Lines 610-615 foreach my $biblioNum (@biblionumbers) { Link Here
610	$anyholdable = 1 if $biblioLoopIter{holdable};	611	$anyholdable = 1 if $biblioLoopIter{holdable};
611	}	612	}
612		613
		614	unless ($pickup_locations->count) {
		615	$numBibsAvailable = 0;
		616	$anyholdable = 0;
		617	$template->param(
		618	message => 1,
		619	no_pickup_locations => 1
		620	);
		621	}
613		622
614	if ( $numBibsAvailable == 0 \|\| $anyholdable == 0) {	623	if ( $numBibsAvailable == 0 \|\| $anyholdable == 0) {
615	$template->param( none_available => 1 );	624	$template->param( none_available => 1 );
616	-