[Koha-patches] [PATCH] fix for bug 1101: Renew All for the OPAC
Jesse Weaver
jesse.weaver at liblime.com
Thu Aug 21 18:52:11 CEST 2008
This adds a Renew All form to the OPAC, and updates opac-renew.pl to take multiple item= parameters.
---
koha-tmpl/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 decb867..2ad85d5 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 195e827..5eb9761 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!!
--
1.5.5.GIT
More information about the Koha-patches
mailing list