From ec9ff6ea2b07236066b2bd65333d93430b67171d Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Wed, 1 Apr 2009 11:57:11 -0500 Subject: [PATCH] Hiding renew link in the OPAC for items which cannot be renewed. Fixes bug 3083. Content-Type: text/plain; charset="utf-8" --- koha-tmpl/opac-tmpl/prog/en/modules/opac-user.tmpl | 2 +- opac/opac-user.pl | 5 +++-- 2 files changed, 4 insertions(+), 3 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 8cfef7d..06f3c69 100644 --- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-user.tmpl +++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-user.tmpl @@ -224,7 +224,7 @@ $.tablesorter.addParser({ &borrowernumber=">Renew ( of renewals remaining) - Not renewable + Not renewable ( of renewals remaining) (On hold) diff --git a/opac/opac-user.pl b/opac/opac-user.pl index 777c659..b353486 100755 --- a/opac/opac-user.pl +++ b/opac/opac-user.pl @@ -127,8 +127,9 @@ foreach my $issue ( @issue_list ) { # check if item is renewable my ($status,$renewerror) = CanBookBeRenewed( $borrowernumber, $issue->{'itemnumber'} ); ($issue->{'renewcount'},$issue->{'renewsallowed'},$issue->{'renewsleft'}) = GetRenewCount($borrowernumber, $issue->{'itemnumber'}); - - $issue->{'status'} = $status || C4::Context->preference("OpacRenewalAllowed"); + $issue->{'status'} = $status && C4::Context->preference("OpacRenewalAllowed"); + if($renewerror eq "too_many"){ $issue->{'too_many'} = 1 }; + if($renewerror eq "on_reserve"){ $issue->{'on_reserve'} = 1 }; if ( $issue->{'overdue'} ) { push @overdues, $issue; -- 1.5.6.5