@@ -, +, @@ I don't see nay reason we needed this Never passed in code Assigned and never referenced All references in coide were in template - never set or passed --- .../prog/en/modules/reserve/request.tt | 54 +++---------------- reserve/request.pl | 19 +------ 2 files changed, 7 insertions(+), 66 deletions(-) --- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt @@ -105,17 +105,6 @@ [% END %] - [% IF ( messagetransfert ) %] -
-

Hold found for ([% nextreservtitle | html %]), please transfer

-

Hold placed by : [% nextreservsurname | html %] [% nextreservfirstname | html %] at : [% branchname | html %] , Please transfer this item. -

-
- -
-
- [% END %] - [% UNLESS ( multi_hold ) %]

Place a hold on [% INCLUDE 'biblio-title.inc' link = 1 %]

[% ELSE %] @@ -325,30 +314,16 @@ [% IF ( biblioloo.checked_previously ) %] Patron has previously checked out this title
[% END %] - [% IF ( biblioloo.alreadyres ) %] + [% IF ( biblioloo.none_avail || biblioloo.noitems ) %] - [% END %] [% END %] @@ -367,7 +342,7 @@ [% END %] - [% IF ( no_reserves_allowed || exceeded_maxreserves || exceeded_holds_per_record || alreadyreserved || none_available || alreadypossession || ageRestricted ) %] + [% IF ( no_reserves_allowed || exceeded_maxreserves || exceeded_holds_per_record || none_available || alreadypossession || ageRestricted ) %]
[% UNLESS ( multi_hold ) %] @@ -381,8 +356,6 @@
  • Too many holds for this record: [% patron.firstname | html %] [% patron.surname | html %] can only place a maximum of [% max_holds_for_record | html %] hold(s) on this record.
  • [% ELSIF ( alreadypossession ) %]
  • [% patron.firstname | html %] [% patron.surname | html %] is already in possession of one item.
  • - [% ELSIF ( alreadyreserved ) %] -
  • [% patron.firstname | html %] [% patron.surname | html %] already has a hold on this item.
  • [% ELSIF ( ageRestricted ) %]
  • Age restricted
  • [% ELSIF ( none_available ) %] @@ -539,7 +512,6 @@ [% END %] - [% IF remaining_holds_for_record > 1 %] @@ -838,30 +810,16 @@ [% IF ( biblioloo.checked_previously ) %] Patron has previously checked out this title
    [% END %] - [% IF ( biblioloo.alreadyres ) %] + [% IF ( biblioloo.none_avail || biblioloo.noitems ) %] - [% END %] [% END %] @@ -905,7 +863,7 @@ [% UNLESS ( patron ) %] [% IF ( reserveloop ) %] -
    + [% IF ( multi_hold ) %] [% END %] --- a/reserve/request.pl +++ a/reserve/request.pl @@ -322,7 +322,6 @@ foreach my $biblionumber (@biblionumbers) { } my $count = Koha::Holds->search( { biblionumber => $biblionumber } )->count(); - my $totalcount = $count; # adding a fixed value for priority options my $fixedRank = $count+1; @@ -348,13 +347,7 @@ foreach my $biblionumber (@biblionumbers) { $biblioloopiter{title} = $biblio->title; $biblioloopiter{rank} = $fixedRank; - # get the time for the form name... - my $time = time(); - - $template->param( - time => $time, - fixedRank => $fixedRank, - ); + $template->param( fixedRank => $fixedRank ); unless( $patron ){ @@ -769,13 +762,3 @@ $template->param( # printout the page output_html_with_http_headers $input, $cookie, $template->output; - -sub sort_borrowerlist { - my $borrowerslist = shift; - my $ref = []; - push @{$ref}, sort { - uc( $a->{surname} . $a->{firstname} ) cmp - uc( $b->{surname} . $b->{firstname} ) - } @{$borrowerslist}; - return $ref; -} --