rel_3_0 moved to HEAD
[koha.git] / opac / opac-reserve.pl
index 356f658..1626fdc 100755 (executable)
 #!/usr/bin/perl
-# NOTE: This file uses standard 8-character tabs
+
+# This file is part of Koha.
+#
+# Koha is free software; you can redistribute it and/or modify it under the
+# terms of the GNU General Public License as published by the Free Software
+# Foundation; either version 2 of the License, or (at your option) any later
+# version.
+#
+# Koha is distributed in the hope that it will be useful, but WITHOUT ANY
+# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
+# A PARTICULAR PURPOSE.  See the GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License along with
+# Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place,
+# Suite 330, Boston, MA  02111-1307 USA
 
 use strict;
 require Exporter;
 use CGI;
-
 use C4::Biblio;
-use C4::Auth;         # checkauth, getborrowernumber.
+use C4::Auth;    # checkauth, getborrowernumber.
 use C4::Koha;
 use C4::Circulation::Circ2;
 use C4::Reserves2;
 use C4::Interface::CGI::Output;
-use HTML::Template;
 use C4::Date;
 use C4::Context;
+use C4::Branch; # GetBranches
 
 my $MAXIMUM_NUMBER_OF_RESERVES = C4::Context->preference("maxreserves");
 
 my $query = new CGI;
-my ($template, $borrowernumber, $cookie)
-    = get_template_and_user({template_name => "opac-reserve.tmpl",
-                            query => $query,
-                            type => "opac",
-                            authnotrequired => 0,
-                            flagsrequired => {borrow => 1},
-                            debug => 1,
-                            });
+my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
+    {
+        template_name   => "opac-reserve.tmpl",
+        query           => $query,
+        type            => "opac",
+        authnotrequired => 0,
+        flagsrequired   => { borrow => 1 },
+        debug           => 1,
+    }
+);
 
 # get borrower information ....
-my ($borr, $flags) = getpatroninformation(undef, $borrowernumber);
-# my @bordat;
-# $bordat[0] = $borr;
+my ( $borr, $flags ) = getpatroninformation( undef, $borrowernumber );
 
 # get biblionumber.....
-my $biblionumber = $query->param('bib');
+my $biblionumber = $query->param('biblionumber');
 
-my $bibdata = bibdata($biblionumber);
- $template->param($bibdata);
- $template->param(biblionumber => $biblionumber);
+my $bibdata = GetBiblioData($biblionumber);
+$template->param($bibdata);
+$template->param( biblionumber => $biblionumber );
 
 # get the rank number....
-my ($rank,$reserves) = FindReserves($biblionumber,'');
-$template->param(reservecount => $rank);
+my ( $rank, $reserves ) = FindReserves( $biblionumber, '' );
+$template->param( reservecount => $rank );
 
 foreach my $res (@$reserves) {
-    if ($res->{'found'} eq 'W') {
-       $rank--;
+    if ( $res->{'found'} eq 'W' ) {
+        $rank--;
     }
 }
 
 $rank++;
-$template->param(rank => $rank);
+$template->param( rank => $rank );
 
 # pass the pickup branch along....
 my $branch = $query->param('branch');
-$template->param(branch => $branch);
+$template->param( branch => $branch );
 
 my $branches = GetBranches();
+
 # make sure it's a real branch
-if (!$branches->{$branch}) {
-$branch='';
+if ( !$branches->{$branch} ) {
+    $branch = '';
 }
-$template->param(branchname => $branches->{$branch}->{'branchname'});
+$template->param( branchname => $branches->{$branch}->{'branchname'} );
 
 # make branch selection options...
-#my $branchoptions = '';
 my @branches;
 my @select_branch;
 my %select_branches;
 
 my @CGIbranchlooparray;
 
-foreach my $branch (keys %$branches) {
-       if ($branch) {
-               my %line;
-               $line{branch} = $branches->{$branch}->{'branchname'};
-               $line{value} = $branch;
-               push @CGIbranchlooparray, \%line;
-       }
+foreach my $branch ( keys %$branches ) {
+    if ($branch) {
+        my %line;
+        $line{branch} = $branches->{$branch}->{'branchname'};
+        $line{value}  = $branch;
+        push @CGIbranchlooparray, \%line;
+    }
 }
-@CGIbranchlooparray = sort {$a->{branch} cmp $b->{branch}} @CGIbranchlooparray;
+@CGIbranchlooparray =
+  sort { $a->{branch} cmp $b->{branch} } @CGIbranchlooparray;
 my $CGIbranchloop = \@CGIbranchlooparray;
-$template->param( CGIbranch => $CGIbranchloop);
+$template->param( CGIbranch => $CGIbranchloop );
 
 #### THIS IS A BIT OF A HACK BECAUSE THE BIBLIOITEMS DATA IS A LITTLE MESSED UP!
 # get the itemtype data....
-my @items = ItemInfo(undef, $biblionumber, 'opac');
+my @items = GetItemInfosOf($biblionumber);
 
 #######################################################
 # old version, add so that old templates still work
 my %types_old;
 foreach my $itm (@items) {
     my $ity = $itm->{'itemtype'};
-    unless ($types_old {$ity}) {
-       $types_old{$ity}->{'itemtype'} = $ity;
-       $types_old{$ity}->{'branchinfo'}->{$itm->{'branchcode'}} = 1;
-       $types_old{$ity}->{'description'} = $itm->{'description'};
-    } else {
-       $types_old{$ity}->{'branchinfo'}->{$itm->{'branchcode'}} ++;
+    unless ( $types_old{$ity} ) {
+        $types_old{$ity}->{'itemtype'} = $ity;
+        $types_old{$ity}->{'branchinfo'}->{ $itm->{'branchcode'} } = 1;
+        $types_old{$ity}->{'description'} = $itm->{'description'};
+    }
+    else {
+        $types_old{$ity}->{'branchinfo'}->{ $itm->{'branchcode'} }++;
     }
 }
 
-foreach my $type (values %types_old) {
+foreach my $type ( values %types_old ) {
     my $copies = "";
-    foreach my $bc (keys %{$type->{'branchinfo'}}) {
-       $copies .= $branches->{$bc}->{'branchname'}."(".$type->{'branchinfo'}->{$bc}.")";
+    foreach my $bc ( keys %{ $type->{'branchinfo'} } ) {
+        $copies .=
+            $branches->{$bc}->{'branchname'} . "("
+          . $type->{'branchinfo'}->{$bc} . ")";
     }
     $type->{'copies'} = $copies;
 }
@@ -120,165 +137,184 @@ foreach my $itm (@items) {
 }
 @items = @temp;
 my $itemcount = @items;
-$template->param(itemcount => $itemcount);
+$template->param( itemcount => $itemcount );
 
 my %types;
 my %itemtypes;
 my @duedates;
 foreach my $itm (@items) {
-    push @duedates, {date_due => format_date($itm->{'date_due'})} if defined $itm->{'date_due'};
-    $itm->{$itm->{'publictype'}} = 1;
+    push @duedates, { date_due => format_date( $itm->{'date_due'} ) }
+      if defined $itm->{'date_due'};
+    $itm->{ $itm->{'publictype'} } = 1;
+
     # FIXME CalcReserveFee is supposed to be internal-use-only
-    my $fee  = CalcReserveFee(undef, $borrowernumber, $itm->{'biblionumber'},'a',($itm->{'biblioitemnumber'}));
+    my $fee = CalcReserveFee( undef, $borrowernumber, $itm->{'biblionumber'},
+        'a', ( $itm->{'biblioitemnumber'} ) );
     $fee = sprintf "%.02f", $fee;
     $itm->{'reservefee'} = $fee;
     my $pty = $itm->{'publictype'};
-    $itemtypes{$itm->{'itemtype'}} = $itm;
-    unless ($types {$pty}) {
-       $types{$pty}->{'count'} = 1;
-       $types{$pty}->{$itm->{'itemtype'}} = 1;
-       push @{$types{$pty}->{'items'}}, $itm;
-    } else {
-       unless ($types{$pty}->{$itm->{'itemtype'}}) {
-           $types{$pty}->{'count'}++;
-           $types{$pty}->{$itm->{'itemtype'}} = 1;
-           push @{$types{$pty}->{'items'}}, $itm;
-       }
+    $itemtypes{ $itm->{'itemtype'} } = $itm;
+    unless ( $types{$pty} ) {
+        $types{$pty}->{'count'} = 1;
+        $types{$pty}->{ $itm->{'itemtype'} } = 1;
+        push @{ $types{$pty}->{'items'} }, $itm;
+    }
+    else {
+        unless ( $types{$pty}->{ $itm->{'itemtype'} } ) {
+            $types{$pty}->{'count'}++;
+            $types{$pty}->{ $itm->{'itemtype'} } = 1;
+            push @{ $types{$pty}->{'items'} }, $itm;
+        }
     }
 }
 
-
-$template->param(ITEMS => \@duedates);
+$template->param( ITEMS => \@duedates );
 
 my $width = keys %types;
-my @publictypes = sort {$b->{'count'} <=> $a->{'count'}} values %types;
+my @publictypes = sort { $b->{'count'} <=> $a->{'count'} } values %types;
 my $typecount;
 foreach my $pt (@publictypes) {
     $typecount += $pt->{'count'};
 }
-$template->param(onlyone => 1) if $typecount == 1;
+$template->param( onlyone => 1 ) if $typecount == 1;
 
 my @typerows;
-for (my $rownum=0;$rownum<$publictypes[0]->{'count'} ;$rownum++) {
+for ( my $rownum = 0 ; $rownum < $publictypes[0]->{'count'} ; $rownum++ ) {
     my @row;
     foreach my $pty (@publictypes) {
-       my @items = @{$pty->{'items'}};
-       push @row, $items[$rownum] if defined $items[$rownum];
+        my @items = @{ $pty->{'items'} };
+        push @row, $items[$rownum] if defined $items[$rownum];
     }
     my $last = @row;
-    $row[$last-1]->{'last'} =1 if $last == $width;
-    my $fill = ($width - $last)*2;
+    $row[ $last - 1 ]->{'last'} = 1 if $last == $width;
+    my $fill = ( $width - $last ) * 2;
     $fill-- if $fill;
-    push @typerows, {ROW => \@row, fill => $fill};
+    push @typerows, { ROW => \@row, fill => $fill };
 }
-$template->param(TYPE_ROWS => \@typerows);
-$width = 2*$width -1;
-$template->param(totalwidth => 2*$width-1,
-);
+$template->param( TYPE_ROWS => \@typerows );
+$width = 2 * $width - 1;
+$template->param( totalwidth => 2 * $width - 1, );
 
-if ($query->param('item_types_selected')) {
-       # this is what happens after the itemtypes have been selected. Stage 2
-       my @itemtypes = $query->param('itemtype');
-       my $fee = 0;
-       my $proceed = 0;
-       if (@itemtypes) {
-               my %newtypes;
-               foreach my $itmtype (@itemtypes) {
-               $newtypes{$itmtype} = $itemtypes{$itmtype};
-               }
-               my @types = values %newtypes;
-               $template->param(TYPES => \@types);
-               foreach my $type (@itemtypes) {
-               my @reqbibs;
-               foreach my $item (@items) {
-                       if ($item->{'itemtype'} eq $type) {
-                       push @reqbibs, $item->{'biblioitemnumber'};
-                       }
-               }
-               $fee += CalcReserveFee(undef,$borrowernumber,$biblionumber,'o',\@reqbibs);
-               }
-               $proceed = 1;
-       } elsif ($query->param('all')) {
-               $template->param(all => 1);
-               $fee = 1;
-               $proceed = 1;
-       }
-       if ($proceed && $branch) {
-               $fee = sprintf "%.02f", $fee;
-               $template->param(fee => $fee,istherefee => $fee>0?1:0);
-               $template->param(item_types_selected => 1);
-               warn "Branch is ==$branch==";
-               $template->param(no_branch_selected => 1) unless ($branch!='');
-       } else {
-               $template->param(message => 1);
-               $template->param(no_items_selected => 1) unless ($proceed);
-               $template->param(no_branch_selected => 1) unless ($branch);
-               warn "Branch is ==$branch==";
-       }
-} elsif ($query->param('place_reserve')) {
-       # here we actually do the reserveration. Stage 3.
-       my $title = $bibdata->{'title'};
-       my @itemtypes = $query->param('itemtype');
-       foreach my $type (@itemtypes) {
-               my @reqbibs;
-               foreach my $item (@items) {
-               if ($item->{'itemtype'} eq $type) {
-                       push @reqbibs, $item->{'biblioitemnumber'};
-               }
-               }
-               CreateReserve(undef,$branch,$borrowernumber,$biblionumber,'o',\@reqbibs,$rank,'',$title);
-       }
-       if ($query->param('all')) {
-               CreateReserve(undef,$branch,$borrowernumber,$biblionumber,'a', undef, $rank,'',$title);
-       }
-       print $query->redirect("/cgi-bin/koha/opac-user.pl");
-} else {
-       # Here we check that the borrower can actually make reserves Stage 1.
-       my $noreserves = 0;
-       my $maxoutstanding = C4::Context->preference("maxoutstanding");
-       $template->param(noreserve => 1) unless $maxoutstanding;
-       if ($borr->{'amountoutstanding'} > $maxoutstanding) {
-               my $amount = sprintf "\$%.02f", $borr->{'amountoutstanding'};
-               $template->param(message => 1);
-               $noreserves = 1;
-               $template->param(too_much_oweing => $amount);
-       }
-       if ($borr->{gonenoaddress} eq 1) {
-               $noreserves = 1;
-               $template->param(message => 1,
-                                               GNA => 1);
-       }
-       if ($borr->{lost} eq 1) {
-               $noreserves = 1;
-               $template->param(message => 1,
-                                               lost => 1);
-       }
-       if ($borr->{debarred} eq 1) {
-               $noreserves = 1;
-               $template->param(message => 1,
-                                               debarred => 1);
-       }
-       my ($resnum, $reserves) = FindReserves('', $borrowernumber);
-       $template->param(RESERVES => $reserves);
-       if ($resnum >= $MAXIMUM_NUMBER_OF_RESERVES) {
-               $template->param(message => 1);
-               $noreserves = 1;
-               $template->param(too_many_reserves => $resnum);
-       }
-       foreach my $res (@$reserves) {
-               if ($res->{'biblionumber'} == $biblionumber) {
-               $template->param(message => 1);
-               $noreserves = 1;
-               $template->param(already_reserved => 1);
+if ( $query->param('item_types_selected') ) {
+
+    # this is what happens after the itemtypes have been selected. Stage 2
+    my @itemtypes = $query->param('itemtype');
+    my $fee       = 0;
+    my $proceed   = 0;
+    if (@itemtypes) {
+        my %newtypes;
+        foreach my $itmtype (@itemtypes) {
+            $newtypes{$itmtype} = $itemtypes{$itmtype};
+        }
+        my @types = values %newtypes;
+        $template->param( TYPES => \@types );
+        foreach my $type (@itemtypes) {
+            my @reqbibs;
+            foreach my $item (@items) {
+                if ( $item->{'itemtype'} eq $type ) {
+                    push @reqbibs, $item->{'biblioitemnumber'};
+                }
+            }
+            $fee +=
+              CalcReserveFee( undef, $borrowernumber, $biblionumber, 'o',
+                \@reqbibs );
+        }
+        $proceed = 1;
+    }
+    elsif ( $query->param('all') ) {
+        $template->param( all => 1 );
+        # No idea why fee would be set to 1 ... it's supposed to be a monetary value, not a flag
+               # -- JF
+               #$fee = 1;
+               $proceed = 1;
+    }
+    if ( $proceed && $branch ) {
+        $fee = sprintf "%.02f", $fee;
+               if ($fee > 1) {
+        $template->param( fee => $fee, istherefee => $fee > 0 ? 1 : 0 );
                }
-       }
-       unless ($noreserves) {
-               $template->param(TYPES => \@types_old);
-               $template->param(select_item_types => 1);
-       }
+        $template->param( item_types_selected => 1 );
+        $template->param( no_branch_selected => 1 ) unless ( $branch != '' );
+    }
+    else {
+        $template->param( message            => 1 );
+        $template->param( no_items_selected  => 1 ) unless ($proceed);
+        $template->param( no_branch_selected => 1 ) unless ($branch);
+    }
+}
+elsif ( $query->param('place_reserve') ) {
+
+    # here we actually do the reserveration. Stage 3.
+    my $title     = $bibdata->{'title'};
+    my @itemtypes = $query->param('itemtype');
+    foreach my $type (@itemtypes) {
+        my @reqbibs;
+        foreach my $item (@items) {
+            if ( $item->{'itemtype'} eq $type ) {
+                push @reqbibs, $item->{'biblioitemnumber'};
+            }
+        }
+        CreateReserve( undef, $branch, $borrowernumber, $biblionumber, 'o',
+            \@reqbibs, $rank, '', $title );
+    }
+    if ( $query->param('all') ) {
+        CreateReserve( undef, $branch, $borrowernumber, $biblionumber, 'a',
+            undef, $rank, '', $title );
+    }
+    print $query->redirect("/cgi-bin/koha/opac-user.pl");
 }
+else {
 
-# check that you can actually make the reserve.
+    # Here we check that the borrower can actually make reserves Stage 1.
+    my $noreserves     = 0;
+    my $maxoutstanding = C4::Context->preference("maxoutstanding");
+    $template->param( noreserve => 1 ) unless $maxoutstanding;
+    if ( $borr->{'amountoutstanding'} > $maxoutstanding ) {
+        my $amount = sprintf "\$%.02f", $borr->{'amountoutstanding'};
+        $template->param( message => 1 );
+        $noreserves = 1;
+        $template->param( too_much_oweing => $amount );
+    }
+    if ( $borr->{gonenoaddress} eq 1 ) {
+        $noreserves = 1;
+        $template->param(
+            message => 1,
+            GNA     => 1
+        );
+    }
+    if ( $borr->{lost} eq 1 ) {
+        $noreserves = 1;
+        $template->param(
+            message => 1,
+            lost    => 1
+        );
+    }
+    if ( $borr->{debarred} eq 1 ) {
+        $noreserves = 1;
+        $template->param(
+            message  => 1,
+            debarred => 1
+        );
+    }
+    my ( $resnum, $reserves ) = FindReserves( '', $borrowernumber );
+    $template->param( RESERVES => $reserves );
+    if ( $resnum >= $MAXIMUM_NUMBER_OF_RESERVES ) {
+        $template->param( message => 1 );
+        $noreserves = 1;
+        $template->param( too_many_reserves => $resnum );
+    }
+    foreach my $res (@$reserves) {
+        if ( $res->{'biblionumber'} == $biblionumber ) {
+            $template->param( message => 1 );
+            $noreserves = 1;
+            $template->param( already_reserved => 1 );
+        }
+    }
+    unless ($noreserves) {
+        $template->param( TYPES             => \@types_old );
+        $template->param( select_item_types => 1 );
+    }
+}
 
 output_html_with_http_headers $query, $cookie, $template->output;