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>

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 decb867..2ad85d5 100644 (file)
--- 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 195e827..5eb9761 100755 (executable)
--- 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!!