From a4ee974e040e363c0f70c352e88ee8490c81858c Mon Sep 17 00:00:00 2001
From: Jesse Weaver <jesse.weaver@liblime.com>
Date: Thu, 21 Aug 2008 11:52:11 -0500
Subject: [PATCH] fix for bug 1101: Renew All for the OPAC

This adds a Renew All form to the OPAC, and updates opac-renew.pl to take multiple item= parameters.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
---
 .../opac-tmpl/prog/en/modules/opac-user.tmpl  | 20 +++++++++----------
 opac/opac-renew.pl                            | 10 ++++++----
 2 files changed, 16 insertions(+), 14 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-user.tmpl b/koha-tmpl/opac-tmpl/prog/en/modules/opac-user.tmpl
index decb867dc7..2ad85d5740 100644
--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-user.tmpl
+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-user.tmpl
@@ -209,16 +209,16 @@ $.tablesorter.addParser({
             </tr>
         <!-- /TMPL_LOOP --></tbody>
         </table>
-        <!-- FIXME: See bug 1101 <!--TMPL_IF Name="OpacRenewalAllowed"-->
-            <form action="/cgi-bin/koha/opac-renew.pl" method="post">
-                <input type="hidden" name="from" value="opac_user" />
-                <input type="hidden" name="borrowernumber" value="<!-- TMPL_VAR NAME="borrowernumber" -->" />
-                <!-- TMPL_LOOP NAME="ISSUES" -->
-                <input type="hidden" name="item" value="<!-- TMPL_VAR NAME="itemnumber" -->" />
-                <!-- /TMPL_LOOP -->
-                <input type="submit" value="Renew All" />
-            </form>
-        <!--/TMPL_IF--> -->
+		<!-- TMPL_IF name="OpacRenewalAllowed" -->
+		<form action="/cgi-bin/koha/opac-renew.pl" method="post">
+			<input type="hidden" name="from" value="opac_user" />
+			<input type="hidden" name="borrowernumber" value="<!-- TMPL_VAR NAME="borrowernumber" -->" />
+			<!-- TMPL_LOOP NAME="ISSUES" -->
+			<input type="hidden" name="item" value="<!-- TMPL_VAR NAME="itemnumber" -->" />
+			<!-- /TMPL_LOOP -->
+			<input type="submit" value="Renew All" />
+		</form>
+		<!-- /TMPL_IF -->
     <!-- TMPL_ELSE -->
         <table>
             <tr><td>You have nothing checked out</td></tr>
diff --git a/opac/opac-renew.pl b/opac/opac-renew.pl
index 195e8274fe..5eb97614c8 100755
--- a/opac/opac-renew.pl
+++ b/opac/opac-renew.pl
@@ -20,12 +20,14 @@ my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
 		  debug           => 1,
 	}
 ); 
-my $itemnumber     = $query->param('item');
+my @items          = $query->param('item');
 my $borrowernumber = $query->param('borrowernumber') || $query->param('bornum');
 
-my ($status,$error) = CanBookBeRenewed( $borrowernumber, $itemnumber );
-if ( $status == 1 ) {
-    AddRenewal( $borrowernumber, $itemnumber );
+for my $itemnumber ( @items ) {
+    my ($status,$error) = CanBookBeRenewed( $borrowernumber, $itemnumber );
+    if ( $status == 1 ) {
+        AddRenewal( $borrowernumber, $itemnumber );
+    }
 }
 # FIXME: else return ERROR to user!!