[Koha-patches] [PATCH][3.0.x] [bug #2811] OpacRenewalAllowed improve.(replace previously send)

[Nahuel Angelinetti]

There is a patch to disallow Renew link in opac-user when opacrenewalAllowed is off.
And an improve in opac-renew.pl to check if opacrenewalallowed is on before renew the document, because a user can renew the
document if he know the url.
---
 koha-tmpl/opac-tmpl/prog/en/modules/opac-user.tmpl |    9 ++++++++-
 opac/opac-renew.pl                                 |    4 +++-
 2 files changed, 11 insertions(+), 2 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..f4b65f5 100644
--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-user.tmpl
+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-user.tmpl
@@ -259,7 +259,14 @@ $.tablesorter.addParser({
 <td>
 <!-- TMPL_IF NAME="debarred" -->Account Frozen
 <!-- TMPL_ELSIF NAME="status" -->
-<a href="/cgi-bin/koha/opac-renew.pl?from=opac_user&amp;item=<!-- TMPL_VAR NAME="itemnumber" -->&amp;bornum=<!-- TMPL_VAR NAME="borrowernumber" -->">Renew</a> <span class="renewals">(<!-- TMPL_VAR NAME="renewsleft" --> of <!-- TMPL_VAR NAME="renewsallowed" --> renewals remaining)</span>
+    <!-- TMPL_IF NAME="OpacRenewalAllowed" -->
+        <a href="/cgi-bin/koha/opac-renew.pl?from=opac_user&amp;item=<!-- TMPL_VAR NAME="itemnumber" -->&amp;bornum=<!-- TMPL_VAR NAME="borrowernumber" -->">Renew</a>
+        <span class="renewals">
+                (<!-- TMPL_VAR NAME="renewsleft" --> of <!-- TMPL_VAR NAME="renewsallowed" --> renewals remaining)
+        </span>
+    <!-- TMPL_ELSE -->
+        Renewal not allowed
+    <!-- /TMPL_IF -->
 <!-- TMPL_ELSIF NAME="onreserve" -->On Reserve
 <!-- TMPL_ELSE -->No renewals left
 <!-- /TMPL_IF -->
diff --git a/opac/opac-renew.pl b/opac/opac-renew.pl
index 195e827..ae96f90 100755
--- a/opac/opac-renew.pl
+++ b/opac/opac-renew.pl
@@ -23,8 +23,10 @@ my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
 my $itemnumber     = $query->param('item');
 my $borrowernumber = $query->param('borrowernumber') || $query->param('bornum');
 
+my $opacrenew = C4::Context->preference("OpacRenewalAllowed");
 my ($status,$error) = CanBookBeRenewed( $borrowernumber, $itemnumber );
-if ( $status == 1 ) {
+
+if ( $status == 1 && $opacrenew == 1 ) {
     AddRenewal( $borrowernumber, $itemnumber );
 }
 # FIXME: else return ERROR to user!!
-- 
1.5.6.3