Bug 10572: Add phone to message_transport_types table for new installs

[koha.git] / opac / opac-renew.pl

diff --git a/opac/opac-renew.pl b/opac/opac-renew.pl
index 195e827..c699b35 100755 (executable)
--- a/opac/opac-renew.pl
+++ b/opac/opac-renew.pl
@@ -2,12 +2,33 @@
 
 #written 18/1/2000 by chris@katipo.co.nz
 # adapted for use in the hlt opac by finlay@katipo.co.nz 29/11/2002
-#script to renew items from the web
+# script to renew items from the web
+# Parts Copyright 2010,2011 Catalyst IT
+
+# 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.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+
+
+use strict;
+use warnings;
 
 use CGI;
 use C4::Circulation;
 use C4::Auth;
-
+use C4::Items;
+use C4::Members;
 my $query = new CGI;
 
 my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
@@ -20,16 +41,40 @@ my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
                  debug           => 1,
        }
 ); 
-my $itemnumber     = $query->param('item');
-my $borrowernumber = $query->param('borrowernumber') || $query->param('bornum');
+my @items          = $query->param('item');
+
+my $opacrenew = C4::Context->preference("OpacRenewalAllowed");
 
-my ($status,$error) = CanBookBeRenewed( $borrowernumber, $itemnumber );
-if ( $status == 1 ) {
-    AddRenewal( $borrowernumber, $itemnumber );
+my $errorstring='';
+for my $itemnumber ( @items ) {
+    my ($status,$error) = CanBookBeRenewed( $borrowernumber, $itemnumber );
+    if ( $status == 1 && $opacrenew == 1 ) {
+       my $renewalbranch = C4::Context->preference('OpacRenewalBranch');
+       my $branchcode;
+       if ($renewalbranch eq 'itemhomebranch'){
+           my $item = GetItem($itemnumber);
+           $branchcode=$item->{'homebranch'};
+       }
+       elsif ($renewalbranch eq 'patronhomebranch'){
+           my $borrower = GetMemberDetails($borrowernumber);
+           $branchcode = $borrower->{'branchcode'};
+       }
+       elsif ($renewalbranch eq 'checkoutbranch'){
+           my $issue = GetOpenIssue($itemnumber);
+           $branchcode = $issue->{'branchcode'};
+       }
+       elsif ($renewalbranch eq 'NULL'){
+           $branchcode='';
+       }
+       else {
+           $branchcode='OPACRenew'
+       }
+        AddRenewal( $borrowernumber, $itemnumber, $branchcode);
+    }
+    else {
+       $errorstring .= $error ."|";
+    }
 }
-# FIXME: else return ERROR to user!!
 
-if ( $query->param('from') eq 'opac_user' ) {
-    print $query->redirect("/cgi-bin/koha/opac-user.pl");
-} 
-# FIXME: ELSE WHAT?  No response at all.  Not very robust.
+print $query->redirect("/cgi-bin/koha/opac-user.pl?renew_error=$errorstring");
+