<input id="hold_form_biblios" type="hidden" name="biblionumbers" value="" />
<input type="hidden" name="findborrower" id="holdFor" value="" />
<input type="hidden" name="club" id="holdForClub" value="" />
- <input type="hidden" name="multi_hold" value="1"/>
</form>
<form id="list_form" method="get" action="/cgi-bin/koha/reserve/request.pl">
<input type="text" size="40" id="patron" class="focus" name="findborrower" autocomplete="off" />
<input type="submit" value="Search" />
[% IF multi_hold %]
- <input type="hidden" name="multi_hold" value="[% multi_hold | html %]"/>
<input type="hidden" name="biblionumbers" value="[% biblionumbers | html %]"/>
[% ELSE %]
<input type="hidden" name="biblionumber" value="[% biblionumber | html %]" />
[% END %]
- [% IF ( multi_hold ) %]
- <input type="hidden" name="multi_hold" value="[% multi_hold | html %]"/>
- <input type="hidden" name="biblionumbers" value="[% biblionumbers | html %]"/>
- [% END %]
</form> <!-- /#holds_patronsearch -->
[% IF borrowers %]
[% INCLUDE 'circ-patron-search-results.inc' destination = "holds" %]
<input type="text" size="40" id="club" class="focus" name="findclub" autocomplete="off" />
<input type="submit" value="Search" />
[% IF multi_hold %]
- <input type="hidden" name="multi_hold" value="[% multi_hold | html %]"/>
<input type="hidden" name="biblionumbers" value="[% biblionumbers | html %]"/>
[% ELSE %]
<input type="hidden" name="biblionumber" value="[% biblionumber | html %]" />
[% END %]
- [% IF ( multi_hold ) %]
- <input type="hidden" name="multi_hold" value="[% multi_hold | html %]"/>
- <input type="hidden" name="biblionumbers" value="[% biblionumbers | html %]"/>
- [% END %]
</form> <!-- /#holds_patronsearch -->
[% IF clubs %]
[% INCLUDE 'clubs-table.inc' destination = "holds" %]
<form action="/api/v1/clubs/[% club.id | html %]/holds" method="post" name="form" id="club-request-form">
[% IF ( multi_hold ) %]
- <input type="hidden" name="multi_hold" value="[% multi_hold | html %]"/>
<input type="hidden" name="biblionumbers" id="multi_hold_bibs" value="[% biblionumbers | html %]"/>
<input type="hidden" name="bad_bibs" id="bad_bibs" value=""/>
<input type="hidden" name="request" value="any"/>
<input type="hidden" name="type" value="str8" />
[% IF ( multi_hold ) %]
- <input type="hidden" name="multi_hold" value="[% multi_hold | html %]"/>
<input type="hidden" name="biblionumbers" id="multi_hold_bibs" value="[% biblionumbers | html %]"/>
<input type="hidden" name="bad_bibs" id="bad_bibs" value=""/>
<input type="hidden" name="request" value="any"/>
<form id="hold_form" method="get" action="/cgi-bin/koha/reserve/request.pl">
<!-- Value will be set here by placeHold() -->
<input id="hold_form_biblios" type="hidden" name="biblionumbers" value="" />
- <input type="hidden" name="multi_hold" value="1"/>
</form>
</main>
my @expirationdates = $query->multi_param('expirationdate');
my @branch = $query->multi_param('pickup');
my @itemnumber = $query->multi_param('itemnumber');
-my $multi_hold = $query->param('multi_hold');
my $biblionumbers = $query->param('biblionumbers');
my $count=@rank;
print $query->redirect("/cgi-bin/koha/circ/circulation.pl?borrowernumber=$borrower[0]");
} else {
my $url = "/cgi-bin/koha/reserve/request.pl?";
- if ($multi_hold) {
+ if (@biblionumber > 1) {
$url .= "multi_hold=1&biblionumbers=$biblionumbers";
} else {
$url .= "biblionumber=$biblionumber[0]";
my $borrower = Koha::Patrons->find( $borrowernumber );
$borrower = $borrower->unblessed if $borrower;
-my $multi_hold = $input->param('multi_hold');
-my $biblionumbers = $multi_hold ? $input->param('biblionumbers') : ($biblionumber . '/');
+my $biblionumbers = $input->param('biblionumbers');
+$biblionumbers ||= $biblionumber . '/';
+
my $bad_bibs = $input->param('bad_bibs');
my $holds_to_place_count = $input->param('holds_to_place_count') || 1;
}
- } elsif ($multi_hold) {
+ } elsif (@biblionumbers > 1) {
my $bibinfo = $bibinfos{$biblionumber};
if ( $can_override || CanBookBeReserved($borrower->{'borrowernumber'}, $biblionumber)->{status} eq 'OK' ) {
AddReserve(
}
}
- if ($multi_hold) {
- if ($bad_bibs) {
- $biblionumbers .= $bad_bibs;
- }
- print $input->redirect("request.pl?biblionumbers=$biblionumbers&multi_hold=1");
- }
- else {
- print $input->redirect("request.pl?biblionumber=$biblionumber");
+ if ($bad_bibs) {
+ $biblionumbers .= $bad_bibs;
}
+ print $input->redirect("request.pl?biblionumber=$biblionumber");
}
elsif ( $borrowernumber eq '' ) {
print $input->header();
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'));
+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.
$do_check = $patron->do_check_for_previous_checkout($item) if $wants_check;
if ( $do_check && $wants_check ) {
$item->{checked_previously} = $do_check;
- if ( $multihold ) {
+ if ( $multi_hold ) {
$biblioloopiter{checked_previously} = $do_check;
} else {
$template->param( checked_previously => $do_check );